Automated cherry pick of #125675: Fix endpoints status out-of-sync when the pod state changes#125859
Conversation
|
This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The DetailsInstructions 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. |
ecdb76d to
c2a9dc8
Compare
When Pod state changes rapidly, endpoints controller may use outdated informer cache to sync Service. If the outdated endpoints appear to be expected by the controller, it skips updating it. The commit fixes it by checking if endpoints informer cache is outdated when processing a service. If the endpoints is stale, it returns an error and retries later. Signed-off-by: Quan Tian <quan.tian@broadcom.com>
c2a9dc8 to
98d8d98
Compare
|
@chymy thanks for reviewing. I have fixed all code conflicts. |
|
/assign @aojea |
|
/kind bug |
|
LGTM label has been added. DetailsGit tree hash: e8e9747d3a7e2205a791bfb99136387dd404bc83 |
|
/cc kubernetes/release-managers |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aojea, saschagrunert, tnqn The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@tnqn: The following test failed, say
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. DetailsInstructions 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. I understand the commands that are listed here. |
|
The Kubernetes project has merge-blocking tests that are currently too flaky to consistently pass. This bot retests PRs for certain kubernetes repos according to the following rules:
You can:
/retest |
Cherry pick of #125675 on release-1.28.
#125675: Fix endpoints status out-of-sync when the pod state changes
For details on the cherry pick process, see the cherry pick requests page.