Skip to content

Conversation

julianwiedmann
Copy link
Member

When the EGW policy lookup returns the ifindex of the egress interface, skip the FIB-lookup based selection and redirect straight to the desired interface.

This currently only covers the minimal datapath part. To make full use of this enhancement we'll also need CEGP control plane improvements (and migrate to an EGW policy entry with space to store the ifindex).

When the EGW policy lookup returns the ifindex of the egress interface,
skip the FIB-lookup based selection and redirect straight to the desired
interface.

This currently only covers the minimal datapath part. To make full use of
this enhancement we'll also need CEGP control plane improvements
(and migrate to an EGW policy entry with space to store the ifindex).

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
@julianwiedmann julianwiedmann added area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. release-note/misc This PR makes changes that have no direct user impact. feature/egress-gateway Impacts the egress IP gateway feature. labels Nov 25, 2024
@julianwiedmann
Copy link
Member Author

/test

@julianwiedmann julianwiedmann marked this pull request as ready for review November 25, 2024 15:06
@julianwiedmann julianwiedmann requested review from a team as code owners November 25, 2024 15:06
Copy link
Member

@ysksuzuki ysksuzuki left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@julianwiedmann julianwiedmann added this pull request to the merge queue Nov 29, 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 Nov 29, 2024
Merged via the queue into cilium:main with commit 7164686 Nov 29, 2024
74 checks passed
@julianwiedmann julianwiedmann deleted the 1.17-bpf-egw-ifindex branch November 29, 2024 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. 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/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants