-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
move GCERegionalPersistentDisk feature from cloud-provider into kube_features.go #77564
move GCERegionalPersistentDisk feature from cloud-provider into kube_features.go #77564
Conversation
/priority backlog |
75f4225
to
2aa0d82
Compare
/approve |
/area code-organization |
/assign @dims |
/test pull-kubernetes-dependencies pull-kubernetes-kubemark-e2e-gce-big |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andrewsykim, lavalamp, logicalhan 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 |
2aa0d82
to
c65c5d2
Compare
thanks for squashing! /lgtm |
/test pull-kubernetes-node-e2e |
/test pull-kubernetes-kubemark-e2e-gce-big |
/test pull-kubernetes-verify |
/test pull-kubernetes-e2e-gce-100-performance |
… to pkg/features since it is no longer used in cloud-provider. This change prevents cloud-provider from bringing in apiserver and component-base (and csi-translation-lib from bringing those two things in transitively)
c65c5d2
to
33522a7
Compare
/lgtm |
reported integration flake in #77573 /retest |
/test pull-kubernetes-integration |
2 similar comments
/test pull-kubernetes-integration |
/test pull-kubernetes-integration |
/retest Review the full test history for this PR. Silence the bot with an |
As of #77412, GCERegionalPersistentDisk is no longer used in cloud-provider. This PR removes the string declaration from cloud-provider and declares it directly in kube_features.go.
This change prevents cloud-provider from bringing in apiserver and component-base (and csi-translation-lib from bringing those two things in transitively).
What type of PR is this?
/kind cleanup
/sig cloud-provider