Skip to content

Make sure iptables chains are recreated on firewall reload#947

Merged
mavenugo merged 1 commit intomoby:masterfrom
aboch:iptfd
Feb 12, 2016
Merged

Make sure iptables chains are recreated on firewall reload#947
mavenugo merged 1 commit intomoby:masterfrom
aboch:iptfd

Conversation

@aboch
Copy link
Copy Markdown
Contributor

@aboch aboch commented Feb 11, 2016

Signed-off-by: Alessandro Boch aboch@docker.com

Signed-off-by: Alessandro Boch <aboch@docker.com>
@mrjana
Copy link
Copy Markdown
Contributor

mrjana commented Feb 11, 2016

LGTM

@mavenugo
Copy link
Copy Markdown
Contributor

LGTM.

Is there a consistent way to reproduce the issue & is it possible to possible to write an IT for this ?
(either in libnetwork or docker/docker ? )

@aboch
Copy link
Copy Markdown
Contributor Author

aboch commented Feb 12, 2016

@mavenugo
Have firewalld running, start docker daemon, reload firewalld service, verify the docker chains are in the iptables o/p.

I verified the fix in an ubuntu VM with firewalld and in a fedora one, after a firewalld restart the chains are reinstalled (the daemon iptables logs are also very useful in showing what is being probed and then installed).

Please note the other issue, where the rules are sometimes wiped out is still there.
It can happen on system events firewalld reacts on, unrelated to docker.
For this second issue, I am sometime able to reproduce just running some commands on firewalld, like firewalld --debug 10. Unfortuantely I can confirm firewalld signalHandler in iptables does not receive any notification when the rules are wiped out.

@mavenugo
Copy link
Copy Markdown
Contributor

@aboch thanks for the clarification.

mavenugo added a commit that referenced this pull request Feb 12, 2016
Make sure iptables chains are recreated on firewall reload
@mavenugo mavenugo merged commit f6e9c17 into moby:master Feb 12, 2016
@aboch aboch deleted the iptfd branch March 7, 2016 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants