Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bpf: nat: fix ICMPv6 ECHO types in snat_v6_rewrite_ingress() #25306

Merged
merged 1 commit into from
May 12, 2023

Conversation

julianwiedmann
Copy link
Member

When revSNATing ICMPv6 packets, check for the ICMPv6 type identifiers.

Fixes: 0759290 ("bpf/nat: review snat_v{4|6}_rewrite_ingress to support more ICMP types")

Fix RevSNAT for ICMPv6 packets.

When revSNATing ICMPv6 packets, check for the ICMPv6 type identifiers.

Fixes: 0759290 ("bpf/nat: review snat_v{4|6}_rewrite_ingress to support more ICMP types")
Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
@julianwiedmann julianwiedmann added kind/bug This is a bug in the Cilium logic. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. release-note/bug This PR fixes an issue in a previous release of Cilium. feature/ipv6 Relates to IPv6 protocol support needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels May 8, 2023
@julianwiedmann julianwiedmann requested a review from a team as a code owner May 8, 2023 07:27
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.13.3 May 8, 2023
@julianwiedmann julianwiedmann requested a review from sahid May 8, 2023 07:27
@julianwiedmann
Copy link
Member Author

julianwiedmann commented May 8, 2023

/test

Job 'Cilium-PR-K8s-1.16-kernel-4.19' failed:

Click to show.

Test Name

K8sAgentPolicyTest Multi-node policy test with L7 policy using connectivity-check to check datapath

Failure Output

FAIL: connectivity-check pods are not ready after timeout

Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.16-kernel-4.19/1310/

If it is a flake and a GitHub issue doesn't already exist to track it, comment /mlh new-flake Cilium-PR-K8s-1.16-kernel-4.19 so I can create one.

Then please upload the Jenkins artifacts to that issue.

@julianwiedmann
Copy link
Member Author

Re-running the failed CI job, it's already gone after 4 days 🤨

@julianwiedmann
Copy link
Member Author

/test-1.16-4.19

@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, 2023
@julianwiedmann julianwiedmann merged commit 405bd39 into cilium:main May 12, 2023
58 checks passed
@julianwiedmann julianwiedmann deleted the 1.14-bpf-icmp6-type branch May 12, 2023 18:39
@jibi jibi mentioned this pull request May 17, 2023
7 tasks
@jibi jibi added backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. and removed needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels May 17, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from main to Backport pending to v1.13 in 1.13.3 May 17, 2023
@jibi jibi added backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. and removed backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. labels May 23, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.13 to Backport done to v1.13 in 1.13.3 May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. feature/ipv6 Relates to IPv6 protocol support kind/bug This is a bug in the Cilium logic. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/bug This PR fixes an issue in a previous release of Cilium. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages.
Projects
No open projects
1.13.3
Backport done to v1.13
Development

Successfully merging this pull request may close these issues.

None yet

4 participants