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

Install iptables as a dependency on redhat systems #61

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lukas-bednar
Copy link
Contributor

The docker daemon will fail to start without it.

Signed-off-by: Lukas Bednar [email protected]

The docker daemon will fail to start without it.

Signed-off-by: Lukas Bednar <[email protected]>
@mongrelion
Copy link
Owner

It's weird that the docker package doesn't define iptables as a hard requirement itself.
Any thoughts on this?

@lukas-bednar
Copy link
Contributor Author

That is good point, this role used to work just fine. Now with latest centos8 images it fails.
The most probable reason is that today you can decide if you use firewalld or iptables?
Maybe in the past they used to have iptables part of image by default ?

@mongrelion
Copy link
Owner

Ok that kind of makes sense. But then with this change we would be making the decision for the user to use iptables over firewalld.
Can we set iptables as a default dependency but make it so that the user can choose firewalld if they want? Is that an idea? Otherwise we're kind of forcing everyone to use iptables.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants