Skip to content
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

v1.9 backports 2021-07-14 #16875

Closed
wants to merge 2 commits into from
Closed

v1.9 backports 2021-07-14 #16875

wants to merge 2 commits into from

Conversation

jibi
Copy link
Member

@jibi jibi commented Jul 14, 2021

Once this PR is merged, you can update the PR labels via:

$ for pr in 16381; do contrib/backporting/set-labels.py $pr done 1.9; done

jibi added 2 commits July 14, 2021 13:11
[ upstream commit c61d02f ]

Signed-off-by: Gilberto Bertin <gilberto@isovalent.com>
[ upstream commit 9e08627 ]

When an endpoint's identity is updated, Cilium does not sync immediately
the new state with k8s, but rather waits up to 10 seconds for the
sync-to-k8s-ciliumendpoint controller to run, meaning that the the new
identity can remain unannounced for up to 10 seconds.

This commit fixes this by explicitly triggering the k8s sync controller
whenever an endpoint's identity is updated.

Fixes: #15097
Suggested-by: Sebastian Wicki <sebastian@isovalent.com>
Signed-off-by: Gilberto Bertin <gilberto@isovalent.com>
@jibi jibi added kind/backports This PR provides functionality previously merged into master. backport/1.9 labels Jul 14, 2021
@jibi jibi requested a review from aanm July 14, 2021 11:14
@jibi jibi requested a review from a team as a code owner July 14, 2021 11:14
@jibi
Copy link
Member Author

jibi commented Jul 14, 2021

test-backport-v1.9

@jibi jibi closed this Jul 14, 2021
@jibi jibi deleted the pr/v1.9-backport-2021-07-14 branch August 2, 2021 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/backports This PR provides functionality previously merged into master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants