Skip to content

bpf: Fix BPF masq when running with non-hybrid DSR#12456

Merged
borkmann merged 1 commit intomasterfrom
pr/brb/fix-bpf-masq-dsr
Jul 8, 2020
Merged

bpf: Fix BPF masq when running with non-hybrid DSR#12456
borkmann merged 1 commit intomasterfrom
pr/brb/fix-bpf-masq-dsr

Conversation

@brb
Copy link
Copy Markdown
Member

@brb brb commented Jul 8, 2020

Previously, BPF masquerading could not be performed when running with non-hybrid DSR due to a call to nodeport_nat_fwd() being compiled out. The latter is responsible for SNATing packets from pods to world.

Fix: 0962c02 ("datapath: Enable BPF MASQ for veth mode in IPv4")

Previously, BPF masquerading could not be performed when running with
non-hybrid DSR due to a call to nodeport_nat_fwd() being compiled out.
The latter is responsible for SNATing packets from pods to world.

Fix: 0962c02 ("datapath: Enable BPF MASQ for veth mode in IPv4")
Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb brb added pending-review 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. labels Jul 8, 2020
@brb brb requested review from a team and borkmann July 8, 2020 11:42
@brb
Copy link
Copy Markdown
Member Author

brb commented Jul 8, 2020

test-me-please

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.005%) to 36.934% when pulling ff93907 on pr/brb/fix-bpf-masq-dsr into a091ba1 on master.

@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 Jul 8, 2020
@borkmann borkmann merged commit d7e2076 into master Jul 8, 2020
@borkmann borkmann deleted the pr/brb/fix-bpf-masq-dsr branch July 8, 2020 19:32
@brb brb mentioned this pull request Jul 15, 2020
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. 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

None yet

Development

Successfully merging this pull request may close these issues.

4 participants