Skip to content

Conversation

smagnani96
Copy link
Contributor

As in #41006, this commits adds the TestPrivileged prefix to some xfrm tests we missed to modify in the latest PR. With this, all the unparallel tests should be executed properly in CI.

As in #41006, this commits adds the TestPrivileged prefix to some xfrm
tests we missed to modify in the latest PR. With this, all the
unparallel tests should be executed properly in CI.

Signed-off-by: Simone Magnani <simone.magnani@isovalent.com>
@smagnani96 smagnani96 self-assigned this Aug 19, 2025
@smagnani96 smagnani96 added kind/bug This is a bug in the Cilium logic. area/CI Continuous Integration testing issue or flake area/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. release-note/ci This PR makes changes to the CI. feature/ipsec Relates to Cilium's IPsec feature needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch labels Aug 19, 2025
smagnani96 added a commit that referenced this pull request Aug 19, 2025
This commit adds the TestPrivileged prefix to the node and ipsec linux tests.
In main this is not needed as it is already running from within a TestPrivileged
suite. While PRs related to similar fixes being backported from main #41078
and #41267 do sync up some tests, in v1.18 there were others that
needed the TestPrivileged prefix to be added. Here's the fix.

The last remaining bits will be the backport of #41279, which have not
been added to this PR for consistency with history.

Signed-off-by: Simone Magnani <simone.magnani@isovalent.com>
@smagnani96
Copy link
Contributor Author

/test

@smagnani96 smagnani96 marked this pull request as ready for review August 19, 2025 16:42
@smagnani96 smagnani96 requested a review from a team as a code owner August 19, 2025 16:42
@smagnani96 smagnani96 requested a review from ldelossa August 19, 2025 16:42
github-merge-queue bot pushed a commit that referenced this pull request Aug 19, 2025
This commit adds the TestPrivileged prefix to the node and ipsec linux tests.
In main this is not needed as it is already running from within a TestPrivileged
suite. While PRs related to similar fixes being backported from main #41078
and #41267 do sync up some tests, in v1.18 there were others that
needed the TestPrivileged prefix to be added. Here's the fix.

The last remaining bits will be the backport of #41279, which have not
been added to this PR for consistency with history.

Signed-off-by: Simone Magnani <simone.magnani@isovalent.com>
@smagnani96 smagnani96 requested review from pchaigno and removed request for ldelossa August 25, 2025 09:03
@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 Aug 25, 2025
@pchaigno pchaigno added this pull request to the merge queue Aug 25, 2025
Merged via the queue into main with commit 56a0504 Aug 25, 2025
158 checks passed
@pchaigno pchaigno deleted the pr/smagnani96/fix-privileged branch August 25, 2025 09:29
@pippolo84 pippolo84 mentioned this pull request Aug 25, 2025
17 tasks
@pippolo84 pippolo84 added backport-pending/1.18 The backport for Cilium 1.18.x for this PR is in progress. and removed needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch labels Aug 25, 2025
@github-actions github-actions bot added backport-done/1.18 The backport for Cilium 1.18.x for this PR is done. and removed backport-pending/1.18 The backport for Cilium 1.18.x for this PR is in progress. labels Sep 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake area/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. backport-done/1.18 The backport for Cilium 1.18.x for this PR is done. feature/ipsec Relates to Cilium's IPsec feature kind/bug This is a bug in the Cilium logic. 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