Skip to content

loadbalancer: Fix privileged testing#44733

Merged
dylandreimerink merged 1 commit intocilium:mainfrom
joamaki:pr/joamaki/lb-fix-privileged-tests
Mar 13, 2026
Merged

loadbalancer: Fix privileged testing#44733
dylandreimerink merged 1 commit intocilium:mainfrom
joamaki:pr/joamaki/lb-fix-privileged-tests

Conversation

@joamaki
Copy link
Copy Markdown
Contributor

@joamaki joamaki commented Mar 11, 2026

In commit 4951b2c ("run only privileged tests on conformance-runtime") the privileged testing CI setup changed to run only TestPrivileged* tests instead of all tests. Adapt the loadbalancer script test runner to this to again run the loadbalancer script tests as privileged (e.g. test against real BPF maps instead of in-memory fakes).

In commit 4951b2c ("run only privileged tests on conformance-runtime")
the privileged testing CI setup changed to run only TestPrivileged* tests
instead of all tests. Adapt the loadbalancer script test runner to this
to again run the loadbalancer script tests as privileged (e.g. test
against real BPF maps instead of in-memory fakes).

Signed-off-by: Jussi Maki <jussi@isovalent.com>
@joamaki joamaki added the release-note/ci This PR makes changes to the CI. label Mar 11, 2026
@joamaki
Copy link
Copy Markdown
Contributor Author

joamaki commented Mar 11, 2026

/test

1 similar comment
@joamaki
Copy link
Copy Markdown
Contributor Author

joamaki commented Mar 11, 2026

/test

@joamaki joamaki marked this pull request as ready for review March 13, 2026 10:25
@joamaki joamaki requested a review from a team as a code owner March 13, 2026 10:25
@joamaki joamaki requested a review from dylandreimerink March 13, 2026 10:25
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Mar 13, 2026
@dylandreimerink dylandreimerink added this pull request to the merge queue Mar 13, 2026
Merged via the queue into cilium:main with commit 0c56817 Mar 13, 2026
85 checks passed
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.

3 participants