Commit Graph

22 Commits (62575fdf48a2ef8e5b2a5da98c6e7313028ed02d)

Author SHA1 Message Date
Sudheer Satyanarayana fa18bbbdb0 which not who (#2557)
minor text fix
2016-12-08 11:34:31 -05:00
Kevin Hildebrand 755535ed7f Fix the interface handling code to allow permanent and non-permanent operations. Also avoid using add_interface because it breaks in cases where the interface is already bound to a different zone. 2016-12-08 11:34:14 -05:00
codehopper-uk e2e0f51739 Basic ability to set masquerade options from ansible, according to current code design/layout (mostly) (#2017)
* Support for masquerade settings

Ability to enable and disable masquerade settings from ansible via:
- firewalld: mapping=masquerade state=disabled permanent=true zone=dmz

Placeholder added (mapping) to support masquerade and port_forward
choices initially - port_forward not implemented yet.

* Permanent and Immediate zone handling differentiated

* Corrected naming abstraction for masquerading functionality

Removed mapping tag with port_forward choices - not applicable!

* Added version info for new masquerade option

Pull Request #2017 failing due to missing version info
2016-12-08 11:34:06 -05:00
Michael Scherer e986aad9aa Remove dead code (#1303)
The review on https://github.com/ansible/ansible-modules-extras/pull/1303
show the problem was already fixed, so we just need to remove the
code.
2016-12-08 11:34:04 -05:00
mo@oclab.net 20fd04b9e7 firewalld: fixes documentation
- removes warning, aligning to existing documentation
- adds version
2016-12-08 11:34:02 -05:00
mo@oclab.net 8b2d484032 firewalld: add/remove interfaces to/from zones 2016-12-08 11:34:02 -05:00
Michael Scherer 61bdf9ee0f Fix #1809, use the proper method to fail 2016-12-08 11:33:56 -05:00
Chad Nelson 89baffa06f Doc wrongly indicates permanent is required
But it isn't. :)
2016-12-08 11:33:48 -05:00
Björn Lichtblau f9383cb8ef Check to make sure the firewalld client is connected before proceeding.
Fixes #1138

Original patch referenced in https://github.com/ansible/ansible/issues/6911 ( f547733b1f ) was undone by 6f2b61d2d8
2016-12-08 11:33:45 -05:00
ag-wood 67cd464ac7 Fix for ansible-modules-extras issue #1080 2016-12-08 11:33:41 -05:00
Michael Scherer 4e4085ad7e Add a more explicit error message, fix #1282 2016-12-08 11:33:37 -05:00
Daniel Nägele d705647873 fix #894 by actually updating with the modified settings 2016-12-08 11:33:18 -05:00
varnav 1050b0f03f Small improvement in documentation 2016-12-08 11:33:14 -05:00
Timothy Vandenbrande 8a28ac3da7 updated version added for source into the docs 2016-12-08 11:32:55 -05:00
Timothy Vandenbrande 0861037fd9 added a source/network add/remove to/from zone for firewalld - removed useless comment 2016-12-08 11:32:55 -05:00
Greg DeKoenigsberg c65a612d1f Changes to author formatting, remove emails 2016-12-08 11:32:51 -05:00
Rene Moser db3e4c8c65 firewalld: remove BabyJSON
See https://github.com/ansible/ansible-modules-extras/issues/430
2016-12-08 11:32:45 -05:00
Brian Coca 2753cd186a moar quotes 2016-12-08 11:32:39 -05:00
Sebastian Gumprich 906cca041b Added documentation and example for port ranges.
Also added punctuation marks.
2016-12-08 11:32:29 -05:00
Cosmo Petrich 9bcf232880 Remove default from 'permanent' and 'state' docs
These fields are both required, and thus do not actually have defaults.
2016-12-08 11:32:22 -05:00
Michael Scherer 3dab019dca Add a new option immediate= to immediately apply a permanent change
Currently, either you apply the change in the configuration
of firewalld ( without permanent=True ), or you apply it live.
I most of the time want to do the 2 at the same time, ie open the
port ( so I can use the service ) and make sure it stay open on reboot.
2016-12-08 11:32:22 -05:00
Michael DeHaan 5879c503bb File extensions! 2016-12-08 11:32:18 -05:00