Skip to content

Conversation

julianwiedmann
Copy link
Member

Clean up the global maps.h header a bit, and co-locate the maps with the rest of the relevant code.

@julianwiedmann julianwiedmann added area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. kind/cleanup This includes no functional changes. release-note/misc This PR makes changes that have no direct user impact. labels Jun 12, 2024
@julianwiedmann julianwiedmann requested a review from ysksuzuki June 12, 2024 10:38
@julianwiedmann julianwiedmann requested review from a team as code owners June 12, 2024 10:38
@julianwiedmann
Copy link
Member Author

/test

@julianwiedmann julianwiedmann enabled auto-merge June 12, 2024 10:38
@julianwiedmann julianwiedmann changed the title bpf: move a feature-specific maps into their header files bpf: move feature-specific maps into their header files Jun 12, 2024
Declutter the maps.h header, and reduce the usage of ENABLE_EGRESS_GATEWAY.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
Declutter the maps.h header, and reduce the usage of
ENABLE_BANDWIDTH_MANAGER

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
Declutter the maps.h header, and reduce the usage of HAVE_ENCAP.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
@julianwiedmann
Copy link
Member Author

/test

@julianwiedmann julianwiedmann added this pull request to the merge queue Jun 14, 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 Jun 14, 2024
Merged via the queue into cilium:main with commit 3e30619 Jun 14, 2024
@julianwiedmann julianwiedmann deleted the 1.16-bpf-maps branch June 14, 2024 04:32
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. kind/cleanup This includes no functional changes. 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