Skip to content
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

Storage: Amend default choice for StorageClass management to honor a specified OpenStack-related value #11002

Conversation

seh
Copy link
Contributor

@seh seh commented Mar 9, 2021

When choosing a default value for the Cluster spec's "cloudConfig.manageStorageClasses" field, first check whether a user specified a concrete value for the related OpenStack "blockStorage.createStorageClass" field. If so, use that value as the effective default value for the former field as well, so as to avoid an unnecessary conflict between the field values on the second validation pass.

See #10733, @zetaab's findings there, and the ensuing discussion in the "kops-dev" channel in the "Kubernetes" Slack workspace for context.


This change is Reviewable

seh added 2 commits March 9, 2021 09:10
When choosing a default value for the Cluster spec's
"cloudConfig.manageStorageClasses" field, first check whether a user
specified a concrete value for the related OpenStack
"blockStorage.createStorageClass" field. If so, use that value as the
effective default value for the former field as well, so as to avoid
an unnecessary conflict between the field values on the second
validation pass.
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Mar 9, 2021
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: zetaab

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added lgtm "Looks good to me", indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Mar 9, 2021
@zetaab
Copy link
Member

zetaab commented Mar 9, 2021

this needs to be backported to release-1.20 also

@seh
Copy link
Contributor Author

seh commented Mar 9, 2021

this needs to be backported to release-1.20 also

See #11003.

@k8s-ci-robot k8s-ci-robot merged commit 59d3fb4 into kubernetes:master Mar 9, 2021
@k8s-ci-robot k8s-ci-robot added this to the v1.21 milestone Mar 9, 2021
k8s-ci-robot added a commit that referenced this pull request Mar 9, 2021
…igin-release-1.20

Automated cherry pick of #11002: Honor an OpenStack StorageClass management choice
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/api cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants