Skip to content

Conversation

julianwiedmann
Copy link
Member

Get rid of the goto logic, and replace it with a straight-forward switch() statement that considers the different CT results.

No change in behavior intended.

Get rid of the goto logic, and replace it with a straight-forward switch()
statement that considers the different CT results.

No change in behavior intended.

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. kind/cleanup This includes no functional changes. release-note/misc This PR makes changes that have no direct user impact. labels Sep 30, 2024
@julianwiedmann
Copy link
Member Author

/test

@julianwiedmann julianwiedmann marked this pull request as ready for review October 1, 2024 05:36
@julianwiedmann julianwiedmann requested a review from a team as a code owner October 1, 2024 05:36
@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 2, 2024
@julianwiedmann julianwiedmann added this pull request to the merge queue Oct 2, 2024
Merged via the queue into cilium:main with commit 0b78380 Oct 2, 2024
68 of 69 checks passed
@julianwiedmann julianwiedmann deleted the 1.17-bpf-lxc-ingress-policy branch October 2, 2024 14:25
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