Commit graph

882 commits

Author SHA1 Message Date
colynn.liu
9b981cecc6 fixed python 2.4 compatibility
nf=tempfile.NamedTemporaryFile(delete=False)
TypeError: NamedTemporaryFile() got an unexpected keyword argument 'delete'
2016-01-22 18:05:48 +08:00
Brian Coca
174bef08ae Merge pull request #1133 from lightblu/devel
Check to make sure the firewalld client is connected before proceeding
2016-01-18 20:51:58 -05:00
Tom X. Tobin
4c4a58e77b osx_defaults: Fix boolean value parsing
Values for boolean types were being unconditionally treated as strings
(by calling `.lower()`), thus breaking value parsing for actual boolean
and integer objects.

It looks like the bug was introduced in:

- 130bd670d8

Fixes #709.
2016-01-14 12:54:48 -05:00
Brian Coca
aae5847064 Merge pull request #1403 from LinusU/linusu-iptables-cleanup
iptables: cleanup
2016-01-14 11:04:37 -05:00
Jeroen Geusebroek
e088c9f819 Fix documentation default value for gluster_volume start_on_create 2016-01-13 21:55:56 +01:00
Brian Coca
a42203bdb3 Merge pull request #1476 from jarrpa/jarrpa-patch-1
gluster_volume: allow probing ourselves
2016-01-12 10:55:17 -05:00
Brian Coca
5f8b288c31 Merge pull request #1389 from brwilson/devel
Properly handle adding multiple bricks to volume
2016-01-12 10:52:50 -05:00
Brian Coca
3aa2d3744d Merge pull request #1482 from ansible/json-imports-fallback
Update for modules which import json.
2016-01-12 10:00:37 -05:00
Brian Coca
4d2a20b2f4 Merge pull request #1382 from Jmainguy/lvol_snapshot
add snapshot feature to lvol
2016-01-11 19:31:04 -05:00
Toshio Kuratomi
d6af6f8477 Update for modules which import json.
Some do not use the json module directly so don't need import json.
Some needed to fallback to simplejson with no traceback if neither was installed

Fixes #1298
2016-01-11 13:06:22 -08:00
Jose A. Rivera
12bf504173 gluster_volume: allow probing ourselves
We should allow "gluster peer probe" to determine if a given "host" maps to the
localhost, and detect that case accordingly.
2016-01-10 12:50:45 -06:00
Jonathan Mainguy
a58b847d5c Remove advertising show_diff feature
--diff will be added to more modules soon, and we want puppet module to utilize this instead of show_diff
2016-01-06 10:04:55 -05:00
Casey Lang
3de4f10bb5 Fix puppet module formatting issue
The `->` in the `show_diff` option doc seemed to be causing the docs
page to break. Not sure why, since it was still valid YAML.
2016-01-05 18:22:38 -06:00
Brian Coca
82a4cf84be Merge pull request #1081 from ag-wood/firewalld-rr-fix
Fix for - Firewalld traces with ALREADY_ENABLED when using port name or re-ordered rich rule
2016-01-04 09:21:46 -05:00
Jeroen Geusebroek
3647b29bee Improved fix for #1074. Both None and '' transform to fqdn. 2016-01-03 21:09:37 +01:00
Jeroen Geusebroek
30c5b303ce Fix for issue #1074. Now able to create volume without replica's. 2016-01-03 08:49:20 +01:00
Matt Martz
9342bd7c38 Import module snippets in the correct location 2015-12-22 15:38:29 -06:00
Matt Martz
0826256e0f Remove unneeded spaces at the end of ohai 2015-12-22 15:31:41 -06:00
Michael Scherer
2f3dc1352f Replace choices=BOOLEANS by type='bool', fix #1326 2015-12-21 23:39:20 +01:00
Linus Unnebäck
cdf7fa2250 iptables: cleanup 2015-12-18 19:40:43 +01:00
Benjamin Wilson
1fe0330abe Properly handle adding multiple bricks to volume 2015-12-16 16:39:42 -05:00
Jonathan Mainguy
7b857b073e add snapshot feature to lvol 2015-12-15 21:49:14 -05:00
Monty Taylor
6912ca0aca Sync minor fixes from OpenStack Infra
Infra has been keeping a local copy of this waiting for ansible 2 to
release. In getting ready for ansible 2 (and our ability to delete our
local copy of the file, I noticed we had a couple of minor cleanups.

Also, the timeout command is there to improve life and workaround puppet
deficiencies. However, it's not working around deficiencies on systems
that do not have the timeout command if we blindly use it.

The puppet specific timeout options are more complex and out of scope of
this.

Issue: #1273
2015-12-14 23:40:01 -05:00
Brian Coca
ab1693a075 Merge pull request #1079 from wkeeling/bugfix_634
Fixes multiple param handling by modprobe.py
2015-12-14 21:47:15 -05:00
Gordon Fierce
b0196edb34 Fix documentation to prevent formatting error where each letter has its own line. 2015-12-07 18:47:26 -05:00
Toshio Kuratomi
a10bdd6be9 Merge pull request #1302 from mscherer/fix_1282
Add a more explicit error message on missing requirement for firewalld
2015-12-04 07:32:35 -08:00
Michael Scherer
037ff89063 Add a more explicit error message, fix #1282 2015-12-04 11:46:20 +01:00
Guillaume Grossetie
32658b9d3b Resolves #1312, Improve pam_limits documentation
Adds comment parameter and improve examples.
2015-12-01 14:27:50 +01:00
GUILLAUME GROSSETIE
c28a0031bb Resolves #1290, Adds limit_type choice "-" 2015-11-26 11:37:59 +01:00
Brian Coca
767ec29640 Merge pull request #1260 from xavpaice/devel
Added stdout and stderr to puppet output for rc=2
2015-11-24 12:51:14 -08:00
Brian Coca
367b88a2ab removed json import 2015-11-21 16:07:12 -08:00
Xav Paice
c1cf8e671a Added stdout and stderr to puppet output for rc=2 2015-11-18 14:34:46 +13:00
Romain Brucker
e0bdd2e7f6 Editing iptable module to use -m state --state instead of -m conntrack --ctstate 2015-11-10 09:21:32 -06:00
Romain Brucker
2b04f0c5cf Fixing limit type from list to string 2015-11-03 11:47:28 -06:00
Romain Brucker
dbee2266e1 Adding limit feature to iptables module 2015-11-03 11:41:30 -06:00
Brian Coca
6a87eed586 made ctstate default to [] and evaluation conditional on the list being popoulated 2015-11-03 12:12:39 -05:00
Brian Coca
ea2fd78e6a fixed default from None to [] for ctstate 2015-11-03 12:03:00 -05:00
Brian Coca
9fb6054500 avoid index error on empty list, key being true means its not None nor []
fixes #13009
2015-11-02 21:50:31 -05:00
Brian Coca
f5ed8d0c6e made ctstate accept lists 2015-11-02 12:11:38 -05:00
Daniel Vigueras
b0278c1f6a Add conntrack module ctstate support to iptables 2015-11-02 10:55:54 +01:00
Monty Taylor
ea5af4c27a Revert "Change show_diff to default to yes"
This was originally to match what puppet agent --test is, since the
rest of the options defaulted to on are grabbed from --test. However,
some security concerns have since been raised - namely that since this
is not the same invocation as --test but instead a remote orchestration
of puppet, the fact that passwords leak into the diff is a dangerous
default.

This reverts commit b86762c180.
2015-11-01 10:50:58 +09:00
Brian Coca
360734ec09 whitespace fixes 2015-10-31 14:31:08 -04:00
Romain Brucker
c648edfbae Adding comment support for iptables module 2015-10-30 11:29:05 -05:00
Brian Coca
3099469b7f fixed typos in cryptab, thanks @timw
fixes #1176
2015-10-29 21:05:33 -04:00
Brian Coca
c685b3d387 corrected version added
fixes #1171
2015-10-27 19:36:07 -04:00
Alexander
64d5502fbe fix typo error
fix spell error for whether
2015-10-26 23:03:51 +08:00
Matt Carroll
5160c4567c Fix ZFS create
This was failing due to the createparent variable being referenced but
never actually loaded from properties
2015-10-25 01:06:58 +01:00
Toshio Kuratomi
5d4474a2b2 Merge pull request #1052 from machacekondra/pam_limits_fix
system: pam_limits: add support for unlimited/infinity/-1
2015-10-23 14:32:46 -07:00
Jonathan Mainguy
b86762c180 Change show_diff to default to yes, to match what puppet agent --test is, since the rest of the options defaulted to on are grabbed from --test 2015-10-23 13:24:39 -04:00
Ondra Machacek
cfec045697 system: pam_limits: add support for unlimited/infinity/-1
Early pam_limits module didn't support special values for items.
This patch is adding support to special values unlimited, infinity and -1.

Issue: https://github.com/ansible/ansible-modules-extras/issues/1033
Signed-off-by: Ondra Machacek <machacek.ondra@gmail.com>
2015-10-22 20:57:38 +02:00