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

Revert ".github/workflows: run datapath complexity tests directly in VM" #24535

Merged
merged 1 commit into from Mar 23, 2023

Conversation

tklauser
Copy link
Member

@tklauser tklauser commented Mar 23, 2023

This reverts commit 4ca9c8e.

Reason for revert: the complexity-test image was broken, lacking clang and llc binaries. This was fixed in
cilium/little-vm-helper-images#51 but for some reason the image build failed so no new image version was built. Revert for now to unbreak the ci-verifier workflow.

Successful run with test commit: https://github.com/cilium/cilium/actions/runs/4497492997?pr=24535

This reverts commit 4ca9c8e.

Reason for revert: the complexity-test image was broken, lacking clang
and llc binaries. This was fixed in
cilium/little-vm-helper-images#51 but for some
reason the image build failed so no new image version was built. Revert
for now to unbreak the ci-verifier workflow.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser tklauser added the release-note/ci This PR makes changes to the CI. label Mar 23, 2023
@tklauser tklauser requested review from a team as code owners March 23, 2023 05:46
@tklauser
Copy link
Member Author

Successful run with test commit: https://github.com/cilium/cilium/actions/runs/4497492997?pr=24535. Removing test commit.

@tklauser tklauser force-pushed the pr/tklauser/revert-verifier-vm-run branch from c1d705e to 98cd4b2 Compare March 23, 2023 05:56
@brlbil brlbil removed the request for review from nbusseneau March 23, 2023 11:25
@tklauser tklauser added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Mar 23, 2023
@borkmann borkmann merged commit 5e1ffa8 into master Mar 23, 2023
54 of 83 checks passed
@borkmann borkmann deleted the pr/tklauser/revert-verifier-vm-run branch March 23, 2023 13:21
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