-
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
Automated cherry pick of #76656: Switch to instance-level update APIs for Azure VMSS loadbalancer operations #76992
Automated cherry pick of #76656: Switch to instance-level update APIs for Azure VMSS loadbalancer operations #76992
Conversation
53a3568
to
0c624db
Compare
/assign @tpepper @justaugustus @andyzhangx |
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.
/lgtm
/test pull-kubernetes-e2e-aks-engine-azure |
/retest |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cblecker, feiskyer, justaugustus 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 |
/retest |
/test pull-kubernetes-kubemark-e2e-gce-big |
Commented on the flake issue opened here: #73105 (comment) |
/test pull-kubernetes-kubemark-e2e-gce-big |
1 similar comment
/test pull-kubernetes-kubemark-e2e-gce-big |
ping @tpepper |
I'm worried about the state of azure/storage patch backports. And this one is a good example of why I'm not happy about this. This is a size/XL patch that feels like a new feature enabling as much or more than a critical bug fix. But it was set to cherry pick to all branches 1.1[1234] and is merged in master (1.15) and 1.11 and 1.12, so it would be really odd to not merge in 1.13 and 1.14. :( |
0c624db
to
0bbf72c
Compare
Rebased because of conflicts. The cherry pick fixes a critical bug of VMSS: all nodes may be rebooted at the same time on some conditions (see details in #71594). @tpepper The change is big because of api-version updates. We need this change to ensure that doesn't' happen. Could you help to get this approved for 1.14? |
@justaugustus @andyzhangx Need lgtm again because of rebase. |
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.
/lgtm
Do you know if this will be release in a 1.14 patch version? |
@jgsqware yep, this cherry pick should be included in v1.14.2 |
Nice and any tip when it will go out? |
@jgsqware -- you can find up-to-date information on patch releases here: https://github.com/kubernetes/sig-release/blob/master/releases/patch-releases.md |
Excellent thank you! |
Cherry pick of #76656 on release-1.14.
Depends on #76753.
#76656: Switch to instance-level update APIs for Azure VMSS loadbalancer operations
Previously, VMSS API doesn't support configurations for each instance. Hence, Azure cloud provider upgrades manually for all instances whenever loadbalancer backend pools need to be updated. This would cause all instances restarted if there're also some other VMSS model changes.
To fix this issue, the PR switches to instance-level update APIs for Azure VMSS loadbalancer operations. It also upgrades Azure SDK to v21.4.0.
/kind bug
/priority critical-urgent
/sig azure