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

fix iptables for services with external traffic policy set to Local #1773

Merged
merged 2 commits into from
Aug 8, 2022

Conversation

zhangzujian
Copy link
Member

@zhangzujian zhangzujian commented Aug 3, 2022

What type of this PR

  • Bug fixes

Which issue(s) this PR fixes:

Fixes #(issue-number)

Fix recreating iptables rules:

I0803 09:48:12.420759   14109 gateway_linux.go:492] iptables rules -p udp -m set --match-set KUBE-NODE-PORT-LOCAL-UDP dst -j MARK --set-xmark 0x80000/0x80000 not exist, recreate iptables rules
I0803 09:48:12.435808   14109 gateway_linux.go:492] iptables rules -p tcp -m set --match-set KUBE-NODE-PORT-LOCAL-TCP dst -j MARK --set-xmark 0x80000/0x80000 not exist, recreate iptables rules
I0803 09:48:40.900971   14109 gateway_linux.go:492] iptables rules -p udp -m set --match-set KUBE-NODE-PORT-LOCAL-UDP dst -j MARK --set-xmark 0x80000/0x80000 not exist, recreate iptables rules
I0803 09:48:40.909351   14109 gateway_linux.go:492] iptables rules -p tcp -m set --match-set KUBE-NODE-PORT-LOCAL-TCP dst -j MARK --set-xmark 0x80000/0x80000 not exist, recreate iptables rules

@zhangzujian zhangzujian added bug Something isn't working need backport labels Aug 3, 2022
@zhangzujian zhangzujian marked this pull request as ready for review August 3, 2022 12:10
@zhangzujian zhangzujian force-pushed the fix-ipt branch 2 times, most recently from 9e4a1c8 to e31aca5 Compare August 5, 2022 06:49
@zhangzujian zhangzujian merged commit 277f6f6 into kubeovn:master Aug 8, 2022
@zhangzujian zhangzujian deleted the fix-ipt branch August 8, 2022 02:19
zhangzujian added a commit that referenced this pull request Aug 8, 2022
…1773)

* fix iptables for services with external traffic policy set to Local

* do not retry e2e testing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working need backport
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants