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

CI: K8sPolicyTest varied flakes #10760

Closed
raybejjani opened this issue Mar 30, 2020 · 2 comments · Fixed by #10779
Closed

CI: K8sPolicyTest varied flakes #10760

raybejjani opened this issue Mar 30, 2020 · 2 comments · Fixed by #10779
Assignees
Labels
area/CI Continuous Integration testing issue or flake

Comments

@raybejjani
Copy link
Contributor

raybejjani commented Mar 30, 2020

This seems like a race between adding this policy and regenerating. My current guess is that the way we wait for k8s netpols to regenerate is not as robust as what we do for CNPs.

These are all of the form:

Expected command: kubectl exec -n 202003300621k8spolicytestbasictestchecksallkindofkubernetespoli app2-64975875bc-j29hw -- curl --path-as-is -s -D /dev/stderr --fail --connect-timeout 5 --max-time 8 http://1.1.1.1/ -w "time-> DNS: '%{time_namelookup}(%{remote_ip})', Connect: '%{time_connect}',Transfer '%{time_starttransfer}', total '%{time_total}'"
To have failed, but it was successful:

https://jenkins.cilium.io/job/Ginkgo-CI-Tests-Pipeline/10979/testReport/junit/Suite-k8s-1/11/K8sPolicyTest_Basic_Test_Denies_traffic_with_k8s_default_deny_ingress_egress_policy/

https://jenkins.cilium.io/job/Ginkgo-CI-Tests-Pipeline/10976/testReport/junit/Suite-k8s-1/18/K8sPolicyTest_Basic_Test_Denies_traffic_with_k8s_default_deny_egress_policy/

https://jenkins.cilium.io/job/Ginkgo-CI-Tests-Pipeline/10970/

https://jenkins.cilium.io/job/Ginkgo-CI-Tests-Pipeline/10967/

There may be cross-test effects. In one case a number of tests failed later with cilium preflight failures (no socket found) https://jenkins.cilium.io/job/Ginkgo-CI-Tests-Pipeline/10971/

@raybejjani raybejjani added the area/CI Continuous Integration testing issue or flake label Mar 30, 2020
@raybejjani raybejjani added this to Daily Flakes in 1.8 CI Mar 30, 2020
@joestringer
Copy link
Member

From weekly meeting: Maybe related to CEP status changes?

@tgraf tgraf removed this from Daily Flakes in 1.8 CI May 4, 2020
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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants