Skip to content

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. area/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 labels May 8, 2023
@julianwiedmann julianwiedmann requested a review from a team as a code owner May 8, 2023 07:27
@julianwiedmann julianwiedmann requested review from ldelossa and 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
@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 backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. and removed needs-backport/1.13 labels May 17, 2023
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. 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.
Projects
No open projects
Status: Released
Development

Successfully merging this pull request may close these issues.

4 participants