-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
gh/workflows: enable egress gateway in ci-datapath #24210
Conversation
5e774e5
to
02c6a9a
Compare
/test |
@lmb MBOI - to trigger only the ci-datapath GHA you can type |
02c6a9a
to
8042484
Compare
8042484
to
9750d4b
Compare
/ci-datapath |
9750d4b
to
c641ea8
Compare
/ci-datapath |
Enable the egress gateway in some datapath workflows. Doing this is a bit tricky, since EGW relies on BPF masquerading to function. The latter has been disabled to work around #23283. Instead, we can force legacy host routing which has a similar effect. Unfortunately, BPF masquerading doesn't work for IPv6 so enabling it breaks a bunch of testcases! The solution is to run half of the tests with BPF masq and EGW disabled, and the other half with EGW on, BPF masq on and IPv6 masq disabled. Updates #24151. Signed-off-by: Lorenz Bauer <lmb@isovalent.com>
3592550
to
d1240bb
Compare
/ci-datapath |
Actually working now: https://github.com/cilium/cilium/actions/runs/4363798670 |
d1240bb
to
a26eeda
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Enable the egress gateway in some datapath workflows. Doing this is
a bit tricky, since EGW relies on BPF masquerading to function. The
latter has been disabled to work around #23283. Instead, we can
force legacy host routing which has a similar effect.
Unfortunately, BPF masquerading doesn't work for IPv6 so enabling
it breaks a bunch of testcases! The solution is to run half of
the tests with BPF masq and EGW disabled, and the other half with
EGW on, BPF masq on and IPv6 masq disabled.
Updates #24151.