Skip to content

build: Add bpf to PATH for privileged tests#11515

Merged
aanm merged 1 commit intomasterfrom
pr/jrajahalme/tests-path-bpf
May 13, 2020
Merged

build: Add bpf to PATH for privileged tests#11515
aanm merged 1 commit intomasterfrom
pr/jrajahalme/tests-path-bpf

Conversation

@jrajahalme
Copy link
Copy Markdown
Member

@jrajahalme jrajahalme commented May 13, 2020

Privileged unit tests need 'cilium-map-migrate' which is built for them,
but the tests can't find it as it is not in the PATH. Add 'bpf' to the
PATH to fix this.

Fix #10838
Fix #11512

Privileged unit tests need 'cilium-map-migrate' which is built for them,
but the tests can't find it as it is not in the PATH. Add 'bpf' to the
PATH to fix this.

Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
@jrajahalme jrajahalme added kind/bug/CI This is a bug in the testing code. area/CI-improvement Topic or proposal to improve the Continuous Integration workflow release-note/ci This PR makes changes to the CI. labels May 13, 2020
@jrajahalme jrajahalme requested a review from a team as a code owner May 13, 2020 04:23
@jrajahalme
Copy link
Copy Markdown
Member Author

test-me-please

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.01%) to 37.843% when pulling c7c33b1 on pr/jrajahalme/tests-path-bpf into d147e15 on master.

Copy link
Copy Markdown
Member

@tklauser tklauser left a comment

Choose a reason for hiding this comment

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

Nice find! I assume this will fix #10838 and #11512.

@tklauser tklauser added the ci/flake This is a known failure that occurs in the tree. Please investigate me! label May 13, 2020
@aanm aanm merged commit 8542a03 into master May 13, 2020
@aanm aanm deleted the pr/jrajahalme/tests-path-bpf branch May 13, 2020 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/CI-improvement Topic or proposal to improve the Continuous Integration workflow ci/flake This is a known failure that occurs in the tree. Please investigate me! kind/bug/CI This is a bug in the testing code. release-note/ci This PR makes changes to the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: 4.9 Ginkgo-CI-Tests-runtime-Pipeline broken CI: RuntimePrivilegedUnitTests Run Tests fails with exit code 1

4 participants