Skip to content

Conversation

julianwiedmann
Copy link
Member

Cover a few left-overs from #38308. The egressgw change is a fix, while the complexity-test parts are just cleanups.

This occurrence had snuck in after I coded up
105cd80 ("datapath: require HAVE_FIB_IFINDEX"). This check is now
always true (as the loader no longer inserts HAVE_FIB_IFINDEX). Remove it.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
These feature macros no longer exist in-tree, remove them.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
@julianwiedmann julianwiedmann requested a review from ysksuzuki May 12, 2025 04:53
@julianwiedmann julianwiedmann requested review from a team as code owners May 12, 2025 04:53
@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. labels May 12, 2025
@julianwiedmann
Copy link
Member Author

/test

@julianwiedmann julianwiedmann enabled auto-merge May 12, 2025 04:53
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

@julianwiedmann julianwiedmann added this pull request to the merge queue May 12, 2025
@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 12, 2025
Merged via the queue into main with commit f8e32a8 May 12, 2025
297 checks passed
@julianwiedmann julianwiedmann deleted the pr/jwi/main/bpf-fib-fixups branch May 12, 2025 07:07
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. 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