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: don't use ./contrib/scripts/kind.sh --xdp in 1.13 workflow #24611

Merged
merged 1 commit into from Mar 30, 2023

Conversation

tklauser
Copy link
Member

The corresponding changes introducing the --xdp flag haven't been backported to v1.13. Don't use it in 1.13 workflows for now.

Partially reverts commit 4310b5e ("contrib/kind: enable XDP_TX from pod veth").

See https://github.com/cilium/cilium/actions/runs/4544793510 for a failure on a 1.13 backport PR.

The corresponding changes introducing the --xdp flag haven't been
backported to v1.13. Don't use it in 1.13 workflows for now.

Partially reverts commit 4310b5e ("contrib/kind: enable XDP_TX from
pod veth").

Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser tklauser added the release-note/ci This PR makes changes to the CI. label Mar 28, 2023
@tklauser tklauser requested review from brb and lmb March 28, 2023 16:34
@tklauser tklauser requested review from a team as code owners March 28, 2023 16:34
@tklauser tklauser mentioned this pull request Mar 29, 2023
9 tasks
@tklauser
Copy link
Member Author

Marking as ready to merge given Lorenz as the original author approved. This will unblock 1.13 backports, namely #24607.

@tklauser tklauser added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Mar 30, 2023
@NikAleksandrov NikAleksandrov mentioned this pull request Mar 30, 2023
29 tasks
@julianwiedmann julianwiedmann merged commit 83bf477 into master Mar 30, 2023
43 checks passed
@julianwiedmann julianwiedmann deleted the pr/tklauser/revert-xdp-1.13 branch March 30, 2023 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants