Skip to content

test: Wait for IPCache entries in testSessionAffinity#11771

Merged
aanm merged 1 commit intomasterfrom
pr/brb/fix-session-affinity-flake
May 29, 2020
Merged

test: Wait for IPCache entries in testSessionAffinity#11771
aanm merged 1 commit intomasterfrom
pr/brb/fix-session-affinity-flake

Conversation

@brb
Copy link
Copy Markdown
Member

@brb brb commented May 29, 2020

We need to wait for an IPCache entry of the restarted pod. Otherwise, a next testSessionAffinity test invokation (with fromOutside = true) might fail, as it could take up to 10s for the entry to appear. See [1] for more details.

This is a temporary fix to get rid of the session affinity flakes.

[1]: #11751

Fix #11698

We need to wait for an IPCache entry of the restarted pod. Otherwise, a
next testSessionAffinity test (with fromOutside = true) might fail, as
it could take up to 10s for the entry to appear. See [1] for more
details.

This is a temporary fix to get rid of the session affinity flakes.

[1]: #11751

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb brb added pending-review area/CI Continuous Integration testing issue or flake labels May 29, 2020
@brb brb requested a review from a team as a code owner May 29, 2020 10:06
@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

3 similar comments
@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

@brb
Copy link
Copy Markdown
Member Author

brb commented May 29, 2020

retest-net-next

@tklauser tklauser added the release-note/ci This PR makes changes to the CI. label May 29, 2020
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 36.85% when pulling b04ec74 on pr/brb/fix-session-affinity-flake into 05daeef on master.

@brb
Copy link
Copy Markdown
Member Author

brb commented May 29, 2020

CI provisioning timeout.

@brb
Copy link
Copy Markdown
Member Author

brb commented May 29, 2020

retest-net-next

@brb
Copy link
Copy Markdown
Member Author

brb commented May 29, 2020

CI net-next hit #11741 flake. We can merge this PR.

@aanm aanm merged commit 9accb13 into master May 29, 2020
@aanm aanm deleted the pr/brb/fix-session-affinity-flake branch May 29, 2020 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

K8sServicesTest Checks service across nodes Tests NodePort BPF Tests with vxlan Tests NodePort with sessionAffinity from outside

6 participants