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

iptables needs -m comment for --comment to work #28334

Merged
merged 1 commit into from Oct 28, 2015
Merged

iptables needs -m comment for --comment to work #28334

merged 1 commit into from Oct 28, 2015

Conversation

anlutro
Copy link
Contributor

@anlutro anlutro commented Oct 27, 2015

No description provided.

@jfindlay
Copy link
Contributor

@anlutro, do you know of this fixing any open issues?

@jfindlay jfindlay added Execution-Module Minor Change Platform Relates to OS, containers, platform-based utilities like FS, system based apps labels Oct 27, 2015
@anlutro
Copy link
Contributor Author

anlutro commented Oct 27, 2015

No idea, I just stumbled across the problem when I was testing something unrelated.

@jfindlay
Copy link
Contributor

No problem, then. I know that there have been some issues related to iptables, but I couldn't find anything exactly like this.

cachedout pushed a commit that referenced this pull request Oct 28, 2015
iptables needs -m comment for --comment to work
@cachedout cachedout merged commit 0d8bea6 into saltstack:2015.8 Oct 28, 2015
@cachedout
Copy link
Contributor

Thanks @anlutro

@anlutro anlutro deleted the fix-iptables_comment branch March 23, 2016 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Execution-Module Platform Relates to OS, containers, platform-based utilities like FS, system based apps
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants