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

docs(troubleshooting.md): add firewalld related docs #403

Merged
merged 4 commits into from
Jan 2, 2024

Conversation

mzz2017
Copy link
Contributor

@mzz2017 mzz2017 commented Jan 2, 2024

Background

由于新版本(v0.5.0+)不再使用两次 NAT 方案,所以放行 port 12345 的方案无法使得防火墙放行 dae 流量。

firewalld 会拦截 reroute packets,firewalld 无法支持匹配 mark,因此只能使用 nft 来加入放行规则。

The new version (v0.5.0+) no longer uses the double NAT scheme, so the solution to allow port 12345 cannot make the firewall allow dae traffic.

Firewalld will block reroute packets. Firewalld does not support matching marks, so only NFT (nftables) can be used to add allow rules.

Checklist

Full Changelogs

  • [Implement ...]

Issue Reference

Closes #[issue number]

Test Result

@mzz2017 mzz2017 requested a review from a team as a code owner January 2, 2024 15:24
@dae-prow dae-prow bot added the documentation Improvements or additions to documentation label Jan 2, 2024
Copy link
Contributor

@sumire88 sumire88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wonderful work. Thanks for your effort in troubleshooting wan binding issue related to firewalld. TBH, it is hard to detect without knowing the underlying implicit deny action made by firewalld. The proposed action is indeed a very nice workaround.

@sumire88
Copy link
Contributor

sumire88 commented Jan 2, 2024

@mzz2017 Please proceed to merge this. Thanks for the hard work.

@mzz2017
Copy link
Contributor Author

mzz2017 commented Jan 2, 2024

Thanks @jschwinger233

@mzz2017 mzz2017 merged commit 99606d6 into main Jan 2, 2024
1 check passed
@mzz2017 mzz2017 deleted the mzz/docs_firewalld_conflict branch January 2, 2024 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants