fix: Discover default storage class when none specified #293
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes aws/karpenter-provider-aws#3647
Description
Previously, when the following
volumeClaimTemplate
was used forDeployments
, we would not discover thestorageClassName
that is being used as the default and fail to find the driver associated with a volumeThis meant that Karpenter and the kube-scheduler would disagree. Karpenter would think that more pods could schedule on the node and the kube-scheduler would reject this due to the volume limits on the node. This change fixes this discovery by finding the default storage class (if one is specified) and using that if no
storageClassName
is specifiedHow was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.