Commit graph

987 commits

Author SHA1 Message Date
René Moser
d159e6af34 Merge pull request #1425 from jonasvermeulen/devel
Added resizing code based on %-values
2016-03-19 00:55:51 +01:00
Michael Scherer
6b2bc9df04 Fix #1809, use the proper method to fail 2016-03-17 18:07:47 +01:00
Emilien Macchi
c289aa4cb9 system/puppet: allow to run puppet -e
-e or --execute [1] allows to execute a specific piece of Puppet code
such a class.

For example, in puppet you would run:
puppet apply -e 'include ::mymodule'

Will be in ansible:
puppet: execute='include ::mymodule'

[1] http://docs.puppetlabs.com/puppet/latest/reference/man/apply.html#OPTIONS
2016-03-17 13:04:24 -04:00
René Moser
aeb82091a6 Merge pull request #1634 from EmilienM/puppet/certname
system/puppet: add --certname parameter
2016-03-16 22:38:04 +01:00
René Moser
074c179793 Merge pull request #1787 from dougluce/cronvar-root-cron
cronvar.py: support absolute destinations
2016-03-16 19:20:49 +01:00
Rene Moser
cd3daf5763 iptables: add defaults to docs 2016-03-16 19:06:56 +01:00
René Moser
f479db69a4 Merge pull request #1822 from Comcast/feature/iptables_dscp_update
add support for setting dscp marks with iptables module
2016-03-16 18:56:33 +01:00
René Moser
186a68a382 Merge pull request #1597 from yta/fix_check_mode_issue_in_osx_defaults
osx_defaults: Do not make any changes in check mode
2016-03-15 22:27:39 +01:00
Brian Coca
df217fe892 added new puppet path to bin search
fixes #1835
2016-03-11 10:14:15 -05:00
David Hocky
404267b064 add version added to new params 2016-03-09 11:20:34 -05:00
David Hocky
b89f0f44d0 fix doc issue 2016-03-09 11:16:27 -05:00
David Hocky
c9e3c57ee9 add support for setting dscp marks with iptables module 2016-03-09 11:04:58 -05:00
Jonas Vermeulen
b5d75234a3 Added resizing based on %values. Included support for VG|PVS|FREE 2016-03-08 21:54:03 +01:00
Jonathan Mainguy
deb72106d2 fixes bug where puppet fails if logdest is not specified 2016-03-05 14:08:28 -05:00
Doug Luce
0298dac401 Use os module for checking absolute/joining paths 2016-03-03 15:10:19 -08:00
Doug Luce
84def4398f cronvar.py: support absolute destinations
Mainly so /etc/crontab can be written to.
2016-03-03 14:57:12 -08:00
Brian Coca
479f99678b Merge pull request #1719 from bborysenko/lvol-doc-fix
The size option is required for lvol module
2016-02-25 14:45:51 -05:00
Borys Borysenko
a3641cac4e The size option is required for lvol module with state=present 2016-02-25 16:56:05 +02:00
Michael Scherer
f63ef1fee8 Add documentation to explain that pvs will be created if needed
Fix #1720
2016-02-24 12:43:37 +01:00
Emilien Macchi
6193ed4b0d system/puppet: add --certname parameter
certname [1] can be a useful parameter when we need to specify a
certificate name different from the default value [2] in Puppet.

Ex: the hosts have different names, in advanced network isolation
setups.

Also, it can be used when we want to run Puppet with a specific node
definition and not using hostname or fqdn to match the nodes where we want to
run Puppet [3] (not recommended by Puppetlabs though).

[1] https://docs.puppetlabs.com/puppet/latest/reference/configuration.html#certname
[2] Defaults to the node’s fully qualified domain name
[3] http://docs.puppetlabs.com/puppet/latest/reference/lang_node_definitions.html#naming
2016-02-22 20:21:17 -05:00
Travis J Parker
1862a813db fixes documented command for svc sending SIGUSR1 2016-02-22 12:14:03 -08:00
Matt Martz
f881eb10d9 Add actual version to version_added for svc module 2016-02-22 11:14:40 -06:00
Ton Kersten
362760413f Fix facter path 2016-02-19 10:43:55 +01:00
James Cammarata
816d3bff6f Adding version_added field to logdest field for puppet module 2016-02-09 14:37:15 -05:00
Monty Taylor
e704525d46 Add option to send puppet apply logs to syslog
While returning puppet logs as ansible stdout is useful in some cases,
there are also cases where it's more destructive than helpful. For
those, local logging to syslog so that the ansible logging makes sense
is very useful.

This defaults to stdout so that behavior does not change for people.
2016-02-09 10:38:56 -06:00
yta
2e9797ca8c osx_defaults: Do not make any changes in check mode 2016-02-04 01:32:47 +09:00
Toshio Kuratomi
ec5c03381b Merge pull request #1378 from johanwiren/fix_properties_handling
Fix zfs properties handling
2016-02-02 13:02:43 -08:00
Brian Coca
14f0ad07ab Merge pull request #1548 from vvanholl/devel
Do not fail when action is delete and volume is not present
2016-02-01 23:54:44 -05:00
Brian Coca
284032efff Merge pull request #1538 from bibliotechy/patch-1
Doc wrongly indicates firewalld permanent flag is required
2016-02-01 23:53:45 -05:00
Brian Coca
cd64f20034 added missing version_added 2016-02-01 23:55:41 -05:00
Brian Coca
ff554a76d7 Merge pull request #1402 from doalitic/iptables-add-limit-burst
Add limit-burst support to iptables module
2016-02-01 23:50:15 -05:00
Dale Smith
27cd172900 Fix regression of #821 in puppet of show_diff
Issue #821 fix: 1382576100
Regressed since: 6912ca0aca

Issue: #821
2016-01-28 15:48:18 +00:00
Brian Coca
ffefe5dde8 Merge pull request #1505 from Yannig/devel_fix_pvcreate
Use -f when pvcreate to avoid interactive input.
2016-01-26 00:00:00 -05:00
Brian Coca
98eeb689ce Merge pull request #1514 from doalitic/iptables-fix-append_csv
iptables: fix param check in append_csv function
2016-01-25 23:58:54 -05:00
Brian Coca
a7ad0bc7be Merge pull request #1499 from specnazzz/fix-seport-ports
seport module fails when the `ports` argument is a integer
2016-01-25 23:57:54 -05:00
Brian Coca
ec8bf56448 Merge pull request #1493 from abulimov/filesystem_recreate_with_force
Allow recreation of same FS with force=yes in filesystem module
2016-01-25 23:57:04 -05:00
Brian Coca
1f2b6bf3fd Merge pull request #1492 from alxgu/filesystem_idemponent
Make fileystem module idemponent. Compare devicesize and filesystemsize.
2016-01-25 23:55:34 -05:00
Vincent VAN HOLLEBEKE
ce3e972ce8 Do not fail when action is delete and volume is not present
This prevents failing when a playbook describes a volume deletion and
is launched more that once.

Without this fix, if you run the playbook a second time, it will fail.
2016-01-24 20:28:01 +01:00
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
Chad Nelson
0c2f602788 Doc wrongly indicates permanent is required
But it isn't. :)
2016-01-21 08:23:36 -05: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
Daniel Vigueras
16d17d287f iptables: fix param check in append_csv function 2016-01-18 16:11:13 +01:00
Daniel Vigueras
ece6872b81 iptables: add --limit-burst option 2016-01-18 16:00:09 +01:00
Mo
23aa66a48e firewalld: fixes documentation
- removes warning, aligning to existing documentation
- adds version
2016-01-16 23:02:58 +01:00
Alexander Gubin
8d7b1b0bd6 Make fileystem module idemponent. Compare devicesize and filesystemsize. 2016-01-15 18:35:45 +01:00
Yannig Perré
c276d9f621 Use -f when pvcreate to avoid interactive input.
Fix https://github.com/ansible/ansible-modules-extras/issues/1504
2016-01-15 16:33:42 +01: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
Michal Mach
ee2c730b7e Fix seport module issue when ports argument is interpolated from a variable and is a int 2016-01-14 11:47:19 +01:00
Jeroen Geusebroek
e088c9f819 Fix documentation default value for gluster_volume start_on_create 2016-01-13 21:55:56 +01:00
Alexander Bulimov
ab9b0558d3 Allow recreation of same FS with force=yes 2016-01-13 17:39:42 +03: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
Mo
c8e35023a4 firewalld: add/remove interfaces to/from zones 2016-01-10 23:18:09 +01: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
Jonathan Mainguy
83f4f16235 Enable stdout and stderr on sucessful runs, making show_diff useable
omit color symbols as ansible makes them illegible
2015-12-15 11:36:55 -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
Greg Hurrell
cc7c929725 osx_defaults: add "host" attribute
This allows us to configure defaults using the `-currentHost` or `-host`
arguments to the `defaults` executable.
2015-12-12 09:23:04 -08:00
Johan Wirén
98c84c560d Keep, but ignore createparent option 2015-12-11 00:24:29 +01:00
Johan Wirén
cd691fad0d Removes the createparent property.
The least surprising thing is to always create parents
2015-12-10 23:32:13 +01:00
Johan Wirén
5437b4a15b Only considers local attributes when comparing state
This should fix #1092
2015-12-10 22:53:06 +01:00
Johan Wirén
fed4b28020 Documentation fixes 2015-12-09 00:38:45 +01:00
Johan Wirén
58d23cba68 Update example 2015-12-09 00:03:29 +01:00
Johan Wirén
e549d96723 Changes how zfs properties are handled
This moves the validation of properties to the zfs command itself. The
properties and their choices were not really correct anyway due to
differences between OpenZFS and Solaris/ZFS.
2015-12-08 23:59:31 +01: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