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

[FEAT] Consider incorporating Kubernete's iptables_wrapper #263

Closed
1 task done
jess-sol opened this issue May 9, 2023 · 3 comments
Closed
1 task done

[FEAT] Consider incorporating Kubernete's iptables_wrapper #263

jess-sol opened this issue May 9, 2023 · 3 comments

Comments

@jess-sol
Copy link

jess-sol commented May 9, 2023

Is this a new feature request?

  • I have searched the existing issues

Wanted change

This container was failing for me on RockyLinux 8.6. Rocky 8 uses nftables, but it seems this image uses legacy iptables, which causes a slew of problems. I've solved these problems in the past by using the excellent iptables-wrapper made by the Kubernetes authors for exactly these kinds of issues.

I build a simple derived container for my own uses which seems to solve the problem well, but I'm no iptables expert, so I'd love to see testing of this done on more setups.

Reason for change

The iptables-wrapper repo has a great explanation in their readme, but this would support both legacy and nft iptables host implementations with minimal effort.

Proposed code change

Example derived container for PoC:

mkdir contrib
wget -P contrib https://raw.githubusercontent.com/kubernetes-sigs/iptables-wrappers/master/iptables-wrapper-installer.sh

cat <<'EOF' > Dockerfile
FROM linuxserver/wireguard:latest
COPY contrib/iptables-wrapper-installer.sh /
RUN bash /iptables-wrapper-installer.sh
RUN apk add nftables
EOF

Happy to submit a PR if there's interest in a change like this

@jess-sol jess-sol added the enhancement New feature or request label May 9, 2023
@github-actions
Copy link

github-actions bot commented May 9, 2023

Thanks for opening your first issue here! Be sure to follow the relevant issue templates, or risk having this issue marked as invalid.

@github-actions
Copy link

github-actions bot commented Jun 8, 2023

This issue has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 8, 2023
@aptalca aptalca moved this from Issues to Done in Issue & PR Tracker Aug 15, 2023
@github-actions
Copy link

github-actions bot commented Sep 8, 2023

This issue is locked due to inactivity

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Archived in project
Development

No branches or pull requests

1 participant