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.

Sign up for GitHub

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

Jump to bottom

bpf: Cover IPsec+KPR in complexity and compile tests #32316

Merged
merged 2 commits into from
May 6, 2024
Merged

bpf: Cover IPsec+KPR in complexity and compile tests #32316

merged 2 commits into from
May 6, 2024

Conversation

pchaigno
Copy link
Member

@pchaigno pchaigno commented May 2, 2024

No description provided.

@pchaigno pchaigno added area/CI Continuous Integration testing issue or flake sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. 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 May 2, 2024
@pchaigno pchaigno force-pushed the compile-complex-tests-ipsec-kpr branch 2 times, most recently from ce46fb0 to 399b6dc Compare May 2, 2024 15:24
@pchaigno pchaigno marked this pull request as ready for review May 2, 2024 15:25
@pchaigno pchaigno requested review from a team as code owners May 2, 2024 15:25
@pchaigno pchaigno requested review from rgo3 and jibi May 2, 2024 15:25
@pchaigno pchaigno enabled auto-merge May 2, 2024 16:04
pchaigno added 2 commits May 5, 2024 08:39
@pchaigno
bpf: Cover IPsec+KPR in compile tests
1a8b5cd 
This commit ensures we cover ENABLE_IPSEC=1 with the more advanced
options of KPR in compile tests.

Like TUNNEL_MODE=1, ENABLE_IPSEC=1 cannot be used in bpf_xdp. We
therefore need to define it for each MAX_*_OPTIONS instead of simply
adding it to MAX_BASE_OPTIONS.

Signed-off-by: Paul Chaignon <[email protected]>
@pchaigno
bpf: Cover IPsec+KPR in complexity tests
e166908 
Signed-off-by: Paul Chaignon <[email protected]>
@pchaigno pchaigno force-pushed the compile-complex-tests-ipsec-kpr branch from 399b6dc to e166908 Compare May 5, 2024 06:39
@pchaigno
Copy link
Member Author

pchaigno commented May 5, 2024

/test

@pchaigno pchaigno requested review from jibi and removed request for jibi May 5, 2024 11:13
@pchaigno pchaigno added this pull request to the merge queue May 6, 2024
@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 May 6, 2024
Merged via the queue into cilium:main with commit f331c04 May 6, 2024
64 checks passed
@pchaigno pchaigno deleted the compile-complex-tests-ipsec-kpr branch May 6, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rgo3 rgo3 rgo3 approved these changes

@jibi jibi jibi approved these changes

Assignees
No one assigned
Labels
area/CI Continuous Integration testing issue or flake area/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. feature/ipsec Relates to Cilium's IPsec feature 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. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pchaigno @jibi @rgo3