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.

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

bpf/test: Adjust mock function to reflect changes in tail_ipvX_policy #31738

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

jschwinger233
Copy link
Member

@jschwinger233 jschwinger233 commented Apr 3, 2024

0cd9780 (bpf: lxc: fine-tune from_tunnel path in ingress tail-call) removed define(TUNNEL_MODE) in tail_ipvX_policy(), this commit makes sure the bpf mock function for that has the same behavior.

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 3, 2024
cilium@0cd9780 (bpf: lxc: fine-tune from_tunnel path in ingress tail-call)
removed `define(TUNNEL_MODE)` in tail_ipvX_policy(), this commit makes
sure the bpf mock function for that has the same behavior.

Signed-off-by: Zhichuan Liang <gray.liang@isovalent.com>
@jschwinger233 jschwinger233 added the release-note/misc This PR makes changes that have no direct user impact. label Apr 3, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 3, 2024
@jschwinger233 jschwinger233 added 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 Apr 3, 2024
@jschwinger233 jschwinger233 marked this pull request as ready for review April 3, 2024 09:11
@jschwinger233 jschwinger233 requested a review from a team as a code owner April 3, 2024 09:11
Copy link
Member

@julianwiedmann julianwiedmann left a comment

Choose a reason for hiding this comment

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

thank you!

@julianwiedmann
Copy link
Member

/test

@julianwiedmann julianwiedmann added this pull request to the merge queue Apr 3, 2024
Merged via the queue into cilium:main with commit b6ca995 Apr 3, 2024
62 checks passed
@jschwinger233 jschwinger233 deleted the gray/fix-bpf-ipsec-test branch April 3, 2024 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. feature/ipsec Relates to Cilium's IPsec feature release-note/ci This PR makes changes to the CI. release-note/misc This PR makes changes that have no direct user impact. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants