Skip to content

Conversation

giorio94
Copy link
Member

The tunnel map is only created and populated when running in tunnel mode. When running in native routing mode, let's additionally ensure that it gets properly unpinned in case it already existed, such as after changing routing mode during testing, to prevent possible leftover entries.

The tunnel map is only created and populated when running in tunnel
mode. When running in native routing mode, let's additionally ensure
that it gets properly unpinned in case it already existed, such as
after changing routing mode during testing, to prevent possible
leftover entries.

Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
@giorio94 giorio94 added area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. area/daemon Impacts operation of the Cilium daemon. release-note/misc This PR makes changes that have no direct user impact. labels Oct 25, 2024
@giorio94
Copy link
Member Author

/test

@giorio94 giorio94 marked this pull request as ready for review October 25, 2024 12:53
@giorio94 giorio94 requested a review from a team as a code owner October 25, 2024 12:53
@giorio94 giorio94 requested a review from jibi October 25, 2024 12:53
@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 Oct 28, 2024
@julianwiedmann julianwiedmann added this pull request to the merge queue Oct 28, 2024
Merged via the queue into cilium:main with commit ff9b9fe Oct 28, 2024
70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/daemon Impacts operation of the Cilium daemon. 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.

3 participants