-
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
Cherry pick label vmss #2877
Cherry pick label vmss #2877
Conversation
feat: enable min/max on Azure VMSS autodiscovery
Fix azure autodiscovery when max tags aren't specified
Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA. It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.
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. I understand the commands that are listed here. |
Welcome @hichemaichour! |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
hey @hichemaichour, thanks for the contribution. Given the size of the changes and that it touches multiple cloud providers and also the nature of it being a feature), I don't think we will be cherry-picking this to earlier releases. |
b8ec7cf
to
884d97d
Compare
…-configurable-ttl-1.17
884d97d
to
e94ebf3
Compare
@marwanad , I understand I was just looking forward to this feature, and with this issue (#2628) present for 1.17, I was hoping that the feature is introduced as well in 1.16 |
@hichemaichour yup, sorry about that. I think you can safely ignore the CSI node log errors, but regardless 1.17 is meant to be run on K8s 1.17 only. /close |
@marwanad: Closed this PR. 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. |
Cherry picking:
/area provider/azure