-
Notifications
You must be signed in to change notification settings - Fork 419
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CI] Bump/add cluster versions #3510
Conversation
Signed-off-by: chipzoller <chipzoller@gmail.com>
An open question here: Should we add into the EKS values file the equivalent value to set |
Signed-off-by: chipzoller <chipzoller@gmail.com>
I like the idea of defaulting to a storageClass that "just works" if one is not already defined. As you've mentioned I think it can get quite complicated for existing users if they are already on a different storageClass. Maybe for now it's best to doc as a troubleshooting step? |
I did at least document it here: https://docs.kubecost.com/install-and-configure/install/provider-installations/aws-eks-cost-monitoring#prerequisites-1 |
Perfect! I'd be open to attempting a gp2 default. My biggest concern would b that swapping storageClasses may be a destructive operation on some people's PVs? Not 100% sure. |
Adding a default value sounds good to me. Does that work for you, @jessegoodier ? |
Signed-off-by: chipzoller chipzoller@gmail.com
What does this PR change?
gp2
StorageClass for all EKS installations.Does this PR rely on any other PRs?
No
How does this PR impact users? (This is the kind of thing that goes in release notes!)
None, CI only.
Links to Issues or tickets this PR addresses or fixes
Closes #3493
What risks are associated with merging this PR? What is required to fully test this PR?
None, CI only.
How was this PR tested?
Will be tested manually after PR is merged. Testing of this change will not be reflected in the Actions run as part of this PR (known limitation).
Have you made an update to documentation? If so, please provide the corresponding PR.
N/A