Skip to content

ci: unquarantine failing test on net-next#20310

Merged
aanm merged 1 commit intocilium:masterfrom
ysksuzuki:unquarantine-all-policy-test
Jul 8, 2022
Merged

ci: unquarantine failing test on net-next#20310
aanm merged 1 commit intocilium:masterfrom
ysksuzuki:unquarantine-all-policy-test

Conversation

@ysksuzuki
Copy link
Copy Markdown
Member

K8sPolicyTest Multi-node policy test validates fromEntities policies Validates fromEntities all policy has been stable with kernel 5.18 (VM rev 145). Now we can unquarantine it.

Fixes: #18520

Signed-off-by: Yusuke Suzuki yusuke-suzuki@cybozu.co.jp

Please ensure your pull request adheres to the following guidelines:

  • For first time contributors, read Submitting a pull request
  • All code is covered by unit and/or runtime tests where feasible.
  • All commits contain a well written commit description including a title,
    description and a Fixes: #XXX line if the commit addresses a particular
    GitHub issue.
  • All commits are signed off. See the section Developer’s Certificate of Origin
  • Provide a title or release-note blurb suitable for the release notes.
  • Thanks for contributing!

`K8sPolicyTest Multi-node policy test validates fromEntities policies
Validates fromEntities all policy` has been stable with kernel 5.18
(VM rev 145). Now we can unquarantine it.

Fixes: cilium#18520

Signed-off-by: Yusuke Suzuki <yusuke-suzuki@cybozu.co.jp>
@ysksuzuki ysksuzuki requested a review from a team June 27, 2022 07:00
@ysksuzuki ysksuzuki requested a review from a team as a code owner June 27, 2022 07:00
@maintainer-s-little-helper maintainer-s-little-helper Bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jun 27, 2022
@aanm
Copy link
Copy Markdown
Member

aanm commented Jun 27, 2022

/test

Job 'Cilium-PR-K8s-GKE' failed:

Click to show.

Test Name

K8sServicesTest Checks E/W loadbalancing (ClusterIP, NodePort from inside cluster, etc) Checks service on same node

Failure Output

FAIL: Unable to restart unmanaged pods with 'kubectl -n kube-system delete pods event-exporter-gke-5479fd58c8-l79b2 konnectivity-agent-5958c7487b-52x7z konnectivity-agent-5958c7487b-xltp5 konnectivity-agent-autoscaler-555f599d94-vtwcf kube-dns-autoscaler-f4d55555-pwklw l7-default-backend-69fb9fd9f9-qgnrw metrics-server-v0.4.5-bbb794dcc-jm7w5': Exitcode: -1 

If it is a flake and a GitHub issue doesn't already exist to track it, comment /mlh new-flake Cilium-PR-K8s-GKE so I can create one.

@ysksuzuki
Copy link
Copy Markdown
Member Author

ysksuzuki commented Jun 27, 2022

k8s-1.24-kernel-net-next is green.

The two failing tests, gke-stable and test-runtime hit flakes, perhaps. I don't think this change can affect them. I think test-runtime hits #19598.

Also, Travis CI failed Automatic restarts limited: Please try restarting this job later or contact support@travis-ci.com. Something was wrong with the CI environment.
https://app.travis-ci.com/github/cilium/cilium/jobs/574777352

@tklauser tklauser added the release-note/ci This PR makes changes to the CI. label Jun 27, 2022
@maintainer-s-little-helper maintainer-s-little-helper Bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jun 27, 2022
Copy link
Copy Markdown
Member

@jrajahalme jrajahalme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI fails seem unrelated to this change.

@aanm aanm merged commit 63f7f69 into cilium:master Jul 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/ci This PR makes changes to the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: net-next K8sPolicyTest Multi-node policy test validates fromEntities policies Validates fromEntities all policy

5 participants