-
Notifications
You must be signed in to change notification settings - Fork 4k
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
azure labels to skip in nodegroupset #6634
azure labels to skip in nodegroupset #6634
Conversation
Change itself LGTM, though WDYT about moving this code to /assign |
cb3c22f
to
1df29be
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes themselves look good (may be a good idea to normalize capitalization of constants), but the code currently does not build, likely as a result of the package move; additional post-move work needed.
/approve cancel |
3f82ebf
to
832017c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we are keeping these outside of provider-specific space temporarily (for change sequencing reasons), and plan to move it the near future - let's add a comment to this effect in the description.
/test ls |
@gandhipr: The specified target(s) for
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test pull-cluster-autoscaler-e2e-azure |
/lgtm @MaciekPytel @gjtempleton @x13n would it make sense to schedule a follow-up item to move all of the cloud-provider-specific stuff in |
Yeah, that makes a lot of sense to me. There's no point having cloudprovider-specific processors in core CA dirs. |
Cool, once this gets and approval and merge I'll do that. |
Sounds good, approving then! /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gandhipr, rakechill, tallaxes, x13n 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 |
/cherry-pick cluster-autoscaler-release-1.30 |
@comtalyst: new pull request created: #7262 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
/cherry-pick cluster-autoscaler-release-1.29 |
@comtalyst: new pull request created: #7263 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
/cherry-pick cluster-autoscaler-release-1.28 |
@comtalyst: new pull request created: #7264 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
/cherry-pick cluster-autoscaler-release-1.27 |
@comtalyst: new pull request created: #7265 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
What type of PR is this?
feature - skip a few azure specific labels in nodegroupset
What this PR does / why we need it:
This PR add a few azure specific labels that needs to be skipped when checking for nodegroup similarity.
This change doesn't include moving azure specific lables under /azure directory - that chore will be covered in another change
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: