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: compile test ENABLE_EGRESS_GATEWAY_COMMON #27515

Merged
merged 1 commit into from Aug 16, 2023

Conversation

lmb
Copy link
Contributor

@lmb lmb commented Aug 15, 2023

Ensure that only defining ENABLE_EGRESS_GATEWAY_COMMON actually works. Also fix the issues that this uncovered.

Fixes: 6cad3dc ("bpf: gate egressgw datapath on separate defines")

@lmb lmb added release-note/minor This PR changes functionality that users may find relevant to operating Cilium. feature/egress-gateway Impacts the egress IP gateway feature. area/modularization labels Aug 15, 2023
@lmb
Copy link
Contributor Author

lmb commented Aug 15, 2023

/test

@lmb lmb marked this pull request as ready for review August 15, 2023 17:52
@lmb lmb requested review from a team as code owners August 15, 2023 17:52
Ensure that only defining ENABLE_EGRESS_GATEWAY_COMMON actually
works. Also fix the issues that this uncovered.

Fixes: 6cad3dc ("bpf: gate egressgw datapath on separate defines")
Signed-off-by: Lorenz Bauer <lmb@isovalent.com>
@lmb
Copy link
Contributor Author

lmb commented Aug 16, 2023

/test

@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 16, 2023
@lmb lmb merged commit b1e9788 into cilium:main Aug 16, 2023
59 checks passed
@lmb lmb deleted the bpf-compile-egw-common branch August 16, 2023 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/modularization feature/egress-gateway Impacts the egress IP gateway feature. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/minor This PR changes functionality that users may find relevant to operating Cilium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants