-
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 #44053 #44313 #44315
Automated cherry pick of #44053 #44313 #44315
Conversation
This brings in the rest of #44053 |
/lgtm |
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
The failures seem to be introduced by commit 2a6083c, which is not in 1.6.
20adb85
to
03102ec
Compare
Sorry, the commits seem a bit weird to me...Why we have two federation commits in the middle? |
Need to clean those federation commits. This is generally caused by reverts in HEAD. |
The existing healthcheck lib was pretty complicated and was hiding some bugs (like the count always being 1), This is a reboot of the interface and implementation to be significantly simpler and better tested.
03102ec
to
ef39e80
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: MrHohn, enisoc, thockin
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
Automatic merge from submit-queue |
Commit found in the "release-1.6" branch appears to be this PR. Removing the "cherrypick-candidate" label. If this is an error find help to get your PR picked. |
@thockin: The following test(s) failed:
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. 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. |
Automatic merge from submit-queue Manual cherry pick of part of #44053 to release-1.5 This cherry pick has the same purpose as #44246 and #44315. The first commit is a manual application of git commit 7664b97 to release-1.5 branch. The unit test part is dropped as it has too many dependencies that shouldn't be cherry picked. The second commit is a surgical fix that does not exist on 1.6+, as we overhauled healthcheck package completely. It fixes the health check deletion logic in OnServiceUpdate for the same bug. @thockin @freehan **Release note**: ```release-note Fix for kube-proxy healthcheck handling an update that simultaneously removes one port and adds another. ```
…k-of-#44053-kubernetes#44313-upstream-release-1.6 Automatic merge from submit-queue Automated cherry pick of kubernetes#44053 kubernetes#44313 Cherry pick of kubernetes#44053 kubernetes#44313 on release-1.6. kubernetes#44053: Add tests for kube-proxy healthcheck, fix bug kubernetes#44313: Zero-out healthchecks when no endpoints ```release-note Fix corner-case with OnlyLocal Service healthchecks ```
Cherry pick of #44053 #44313 on release-1.6.
#44053: Add tests for kube-proxy healthcheck, fix bug
#44313: Zero-out healthchecks when no endpoints