Skip to content

ipsec: fix privileged tests#41006

Merged
joestringer merged 1 commit intomainfrom
pr/smagnani96/fix-ipsec-privileged-tests
Aug 8, 2025
Merged

ipsec: fix privileged tests#41006
joestringer merged 1 commit intomainfrom
pr/smagnani96/fix-ipsec-privileged-tests

Conversation

@smagnani96
Copy link
Copy Markdown
Contributor

@smagnani96 smagnani96 commented Aug 7, 2025

This commit fixed some test name in the ipsec suite by adding the "TestPrivileged" prefix, to make sure they're executed and not skipped.

@smagnani96
ipsec: fix privileged tests
1b9dab5 
This commit fixed some test name in the ipsec suite by adding the
"TestPrivileged" prefix, to make sure they're executed and not skipped.

Signed-off-by: Simone Magnani <simone.magnani@isovalent.com>
@smagnani96 smagnani96 self-assigned this Aug 7, 2025
@smagnani96 smagnani96 added kind/bug This is a bug in the Cilium logic. 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 labels Aug 7, 2025
@smagnani96
Copy link
Copy Markdown
Contributor Author

smagnani96 commented Aug 7, 2025

/test

@smagnani96 smagnani96 marked this pull request as ready for review August 7, 2025 18:51
@smagnani96 smagnani96 requested a review from a team as a code owner August 7, 2025 18:51
@smagnani96 smagnani96 requested a review from ldelossa August 7, 2025 18:51
@smagnani96 smagnani96 added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch labels Aug 8, 2025
@smagnani96
Copy link
Copy Markdown
Contributor Author

smagnani96 commented Aug 8, 2025

@cilium/committers This can be merged. PTAL at labels: just added the needs-backport/1.18, not sure about other ones.

@joestringer joestringer added this pull request to the merge queue Aug 8, 2025
Merged via the queue into main with commit 5198d28 Aug 8, 2025
156 of 157 checks passed
@joestringer joestringer deleted the pr/smagnani96/fix-ipsec-privileged-tests branch August 8, 2025 23:30
@YutaroHayakawa YutaroHayakawa mentioned this pull request Aug 11, 2025
Merged
10 tasks
@YutaroHayakawa YutaroHayakawa 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 11, 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 Aug 12, 2025
smagnani96 added a commit that referenced this pull request Aug 19, 2025
@smagnani96
ipsec: fix xfrm privileged tests
d605803 
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 mentioned this pull request Aug 19, 2025
Merged
github-merge-queue bot pushed a commit that referenced this pull request Aug 25, 2025
@smagnani96 @pchaigno
ipsec: fix xfrm privileged tests
56a0504 
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>
pippolo84 pushed a commit that referenced this pull request Aug 25, 2025
@smagnani96 @pippolo84
ipsec: fix xfrm privileged tests
2c3fc2e 
[ upstream commit 56a0504 ]

[ backporter's notes: Fixes minor conflicts due to different signature
for setupIPSecSuitePrivileged in stable branch. ]

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>
Signed-off-by: Fabio Falzoi <fabio.falzoi@isovalent.com>
pippolo84 pushed a commit that referenced this pull request Aug 25, 2025
@smagnani96 @pippolo84
ipsec: fix xfrm privileged tests
67c0e9e 
[ upstream commit 56a0504 ]

[ backporter's notes: Fixes minor conflicts due to different signature
for setupIPSecSuitePrivileged in stable branch. ]

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>
Signed-off-by: Fabio Falzoi <fabio.falzoi@isovalent.com>
pippolo84 pushed a commit that referenced this pull request Aug 26, 2025
@smagnani96 @pippolo84
ipsec: fix xfrm privileged tests
b677930 
[ upstream commit 56a0504 ]

[ backporter's notes: Fixes minor conflicts due to different signature
for setupIPSecSuitePrivileged in stable branch. ]

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>
Signed-off-by: Fabio Falzoi <fabio.falzoi@isovalent.com>
pippolo84 pushed a commit that referenced this pull request Aug 26, 2025
@smagnani96 @pippolo84
ipsec: fix xfrm privileged tests
5d7dd54 
[ upstream commit 56a0504 ]

[ backporter's notes: Fixes minor conflicts due to different signature
for setupIPSecSuitePrivileged in stable branch. ]

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>
Signed-off-by: Fabio Falzoi <fabio.falzoi@isovalent.com>
pippolo84 pushed a commit that referenced this pull request Aug 26, 2025
@smagnani96 @pippolo84
ipsec: fix xfrm privileged tests
1888655 
[ upstream commit 56a0504 ]

[ backporter's notes: Fixes minor conflicts due to different signature
for setupIPSecSuitePrivileged in stable branch. ]

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>
Signed-off-by: Fabio Falzoi <fabio.falzoi@isovalent.com>
github-merge-queue bot pushed a commit that referenced this pull request Sep 1, 2025
@smagnani96 @pippolo84
ipsec: fix xfrm privileged tests
fbc250b 
[ upstream commit 56a0504 ]

[ backporter's notes: Fixes minor conflicts due to different signature
for setupIPSecSuitePrivileged in stable branch. ]

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>
Signed-off-by: Fabio Falzoi <fabio.falzoi@isovalent.com>
@cilium-release-bot cilium-release-bot bot moved this to Released in cilium v1.19.0 Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ldelossa ldelossa ldelossa approved these changes

Assignees

@smagnani96 smagnani96

Labels

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

No open projects
cilium v1.19.0
Status: Released

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@smagnani96 @ldelossa @joestringer @msune @YutaroHayakawa