Commit graph

9828 commits

Author SHA1 Message Date
techraf
7d09792758 Added 'backup' parameter to get_url module 2016-12-08 11:24:00 -05:00
Yoichi Imai
b9b08666f5 Fix iam_policy pdoc variable 2016-12-08 11:24:00 -05:00
Thomas Quinot
ef3308159f Allow cron_file to be an absolute path
Support specifying an absolute path (typically /etc/crontab) rather than
a path relative to /etc/cron.d, to allow modifying the main system crontab.
Particularly useful for target systems that have /etc/crontab but no
/etc/cron.d.
2016-12-08 11:24:00 -05:00
Michael Scherer
519e699b98 Split a shell snippet in 2 to avoid using use_unsafe_shell=True
Since use_unsafe_shell is suspicious from a security point
of view (or it wouldn't be unsafe), the less we have, the less
code we have to toroughly inspect for a security audit.

In this case, the '&&' can be replaced by doing 2 calls to run_command.
2016-12-08 11:24:00 -05:00
Jason Paige
a2f4d46c9b updating choices for rds parameter engines 2016-12-08 11:24:00 -05:00
Aaron Kurtz
e3782689bc Remove 'cache_table' from required params
Starting in Django 1.7, the createcachetable command looks for cache
table names in the CACHES settings dictionary, so cache_table is no
longer required, but is still allowed.
2016-12-08 11:23:59 -05:00
Bermudez, Jaime
efbdea909e Simplify logic around "present" state and skip dupes. If the policy doesn't exist or the policy document doesn't match or if we're not skipping dupes, then we go ahead and update the respective user/group/role policy. 2016-12-08 11:23:59 -05:00
daniel-sc
f654bd45f8 Allowing specifying ip by address 2016-12-08 11:23:59 -05:00
daniel-sc
40565b7293 Allowing to specify specific external ip for gce instances 2016-12-08 11:23:59 -05:00
Albert Mikaelyan
a61095b0b1 Update yum.py 2016-12-08 11:23:59 -05:00
Vladimir Mihailenco
2bdfea4a43 rackspace: set headers when uploading files to CF.
Otherwise CDN (Akamai) downloads file without the headers. The sequence
is following:
1. Ansible uploads file to CF.
2. Akamai downloads the file and caches it in CDN.
3. Ansible sets headers.

As a result Akamai serves file without headers.

This is backwards incompatible change, because headers keys are not
prefixed with `x-object-meta-`. Which allows user to set headers like
`Access-Control-Allow-Origin`.
2016-12-08 11:23:59 -05:00
bobgroves
18d4a33891 Makes rax_dns_record override default to true 2016-12-08 11:23:59 -05:00
bobgroves
65697dea14 Adds overwrite option to rax_dns_record 2016-12-08 11:23:59 -05:00
Jordan Bach
b6213ffa80 git: set remote branch before switching versions when using depth argument 2016-12-08 11:23:59 -05:00
Luca Berruti
0673a4919f cron module: add enviroment variables management 2016-12-08 11:23:59 -05:00
Henrique Rodrigues
c2e62ed9a9 Fix misspelling of the word 'certificate' 2016-12-08 11:23:59 -05:00
Brian Coca
b25c2a4261 added note about awslogs version added 2016-12-08 11:23:59 -05:00
Brian Coca
6e73c61298 added version_added 2016-12-08 11:23:59 -05:00
Constantin07
f36c567022 Added delete_on_termination option for volume attachment 2016-12-08 11:23:59 -05:00
Matt Ferrante
2dd53a8d91 updated s3 module documentation 2016-12-08 11:23:59 -05:00
jefferson fermo
9d9daeb73c add awslog option in docker log driver. this option is available starting with ansible 2.1 2016-12-08 11:23:59 -05:00
Brian Coca
4c6515e299 reverse order of cron commands
does not affect linux but fixes issue with busybox caring about order
fixes #2807
2016-12-08 11:23:58 -05:00
nitzmahone
821fb6c0ab fix win_get_url var renames ala xpicio fix PR #2378
reimplements #2378 sans formatting merge conflicts
2016-12-08 11:23:58 -05:00
Jeroen Seegers
7be08b00b0 Update cron.py
Minor update to documentation for the cron module to reflect the required "name" parameter when the value of "state" is "absent".
2016-12-08 11:23:58 -05:00
Brian Coca
f50fbe9dd2 updated s3 docs to reflect mode choices
fixes #2318
2016-12-08 11:23:58 -05:00
autotune
90631aef1b 1113 fix ec2 subnet tags 2016-12-08 11:23:58 -05:00
nitzmahone
fef6e5102e fix version_added on win_msi wait arg 2016-12-08 11:23:58 -05:00
Brian Coca
a5e4969dc4 updated version_added 2016-12-08 11:23:58 -05:00
Joost Cassee
267dcc8eb6 Add wait_timeout option to route53 module
This option is used to make sure the module does not block forever.
2016-12-08 11:23:58 -05:00
Joost Cassee
e91bcba078 Add wait option to route53 module
This option makes the module wait for the changes to replicate to all
Amazon DNS servers.
2016-12-08 11:23:58 -05:00
Mark van Driel
a24cdbe15e Fixed "invalid privileges string: set expected at most 1 arguments, got 2" 2016-12-08 11:23:58 -05:00
Jonathan Mainguy
e9a752f769 add ansi support 2016-12-08 11:23:58 -05:00
Ben Visser
cfed7e11d8 two choices for cache engine. adding them.
revert spacing

undoing quotes around default. will do another pull requests

Making documentation more explicit for elasticache engine choices.
2016-12-08 11:23:58 -05:00
Matt Clay
d861329d5e Added filename option to apt_repository module. 2016-12-08 11:23:58 -05:00
Hillel Antonov
8d5cebd597 Adding gathering environment variables to gather facts on Windows making similar in functionality to gather facts on Linux 2016-12-08 11:23:58 -05:00
Hans-Joachim Kliemeck
1354ef0285 fix problem with file content and high cpu/out of memory errors 2016-12-08 11:23:58 -05:00
Timothy R. Chavez
2ef913a288 Pass network argument to add_ips_to_server
The `network` argument needs to be passed through `add_ips_to_server`
so that the default value can be properly overridden from ansible.
2016-12-08 11:23:58 -05:00
Marios Andreopoulos
1ae765d61f fix: do not set scopes account to service account used for instance creation 2016-12-08 11:23:58 -05:00
Hans-Joachim Kliemeck
41ab7f8aed do not move item, instead copy item because acl will be kept 2016-12-08 11:23:57 -05:00
Alexey Wasilyev
4d30ae313d Add cpu_shares option for docker module 2016-12-08 11:23:57 -05:00
krdlab
c680611fe4 Fix self.changed value in ElbManager (ec2_elb_lb) 2016-12-08 11:23:57 -05:00
Jordi Gutiérrez Hermoso
4682ea1c3c hg: discard changes without changing the current revision
The command `hg up -C` by default moves to the latest revision on the
current branch. The `discard` function was trying to update to a
different branch, in case it was provided, by passing a `-r REVISION`
argument. Not only is this not the intended effect of the `discard`
function, but this also could update to a different branch that hasn't
been pulled yet, which is how we were experiencing trouble.

Instead, we unconditionally do `hg up -C -r .` to "update" to the
current revision (i.e. to "."), while `-C/--clean`ing the current
directory. This is similar to `hg revert --all`, except that it also
undoes the merge state of the working directory, in case there was
any.
2016-12-08 11:23:57 -05:00
Will Rouesnel
b565da78b3 docker: Fix state=reloaded to detect default LoggingDriver for docker
Previously the logging module hard coded the default logging driver. This means
if the docker daemon is started with a different logging driver, the ansible
module would continually restart it when run.

This fix adds a call to docker.Client.info(), which is inspected if a logging
driver is not supplied in the playbook, and the container only restarted if
the logging driver applied differs from the configured default.

In usage, this has solved issues with using alternative logging drivers.
2016-12-08 11:23:57 -05:00
Corwin Brown
c075c000c8 Adding Facter 2016-12-08 11:23:57 -05:00
Brian Coca
351de21ee7 made port default depending on db engine 2016-12-08 11:23:57 -05:00
Mark van Driel
d67f2eb104 Fixes require ssl in combination with other privileges
Fixes require ssl in combination with grant option

Refactoring: code cleanup to make it easier to understand

Code rewritten inspired by @willthames

Added WITH GRANT OPTION as exception; when only REQUIRESSL and/or GRANT are specified we have to add USAGE
2016-12-08 11:23:57 -05:00
Phil
3781c5ccb0 uses get-attr and fixes file path issues by escaping quotes around path 2016-12-08 11:23:57 -05:00
schwartzmx
5ff4d219e0 updates win_msi to allow install to wait - adds additional wait param - updates to use Start-Process to allow -Wait - this will wait for install/uninstall to complete before continue 2016-12-08 11:23:57 -05:00
Jan Duzinkiewicz
9c6db69827 forcing git fetch if sha1 are equal between repos but branch exists only at remote
Add an explanatory comment to the code
2016-12-08 11:23:57 -05:00
Brian Coca
3afaa2fef4 really fixed the doc formatting 2016-12-08 11:23:57 -05:00
Brian Coca
1fe3387971 crrected version added 2016-12-08 11:23:57 -05:00
Brian Coca
e2062c44d9 made note about choice availability in ansible ver 2016-12-08 11:23:57 -05:00
Brian Coca
0ca9356675 added versions note in description for new choice 2016-12-08 11:23:56 -05:00
Brian Coca
f5ddbd5c49 fixed win_feature docs
added version_added for source and fixed some yaml parsing
2016-12-08 11:23:56 -05:00
Pavel Derendyaev
3129154a2a Add "timeout" option to Docker module 2016-12-08 11:23:56 -05:00
Trond Hindenes
abcae97422 update doc 2016-12-08 11:23:56 -05:00
Trond Hindenes
37a24cc1b1 bugfix win_feature.ps1 2016-12-08 11:23:56 -05:00
Trond Hindenes
21b79123a4 Add source as optional parameter 2016-12-08 11:23:56 -05:00
KIKUCHI Koichiro
2974358b2b Add check_mode support to mysql_user. 2016-12-08 11:23:56 -05:00
Arata Notsu
a9f23ac54f apt: export env vars before run dpkg
Without this change, some trouble may occur when "deb" parameter
is used as env vars controlling dpkg are not set. For example,
installing a package that requires user input will never end since
DEBIAN_FRONTEND=noninteractive is not set.

So export env vars in APT_ENV_VARS before run dpkg, like in cases
using apt-get/aptitude.
2016-12-08 11:23:56 -05:00
Jonathan Mainguy
47d0719cfe Fix bad merge 2016-12-08 11:23:56 -05:00
Brian Coca
288a5c9a77 adds diff info for file info 2016-12-08 11:23:56 -05:00
nitzmahone
60ca6e2f85 fix win_file to stop on errors 2016-12-08 11:23:56 -05:00
Jon Hadfield
d0e37fa542 return allocation id for vpc address. 2016-12-08 11:23:56 -05:00
Dmitry Labutin
b20066ff8e Fix issue #2721: os_object parameters and docs don't align (file/filename) 2016-12-08 11:23:56 -05:00
Dmitry Labutin
60b1273d55 Github uses both HTTP and HTTPS protocols 2016-12-08 11:23:56 -05:00
Ben Visser
a01fe88263 staging that num_nodes is required when state=present 2016-12-08 11:23:56 -05:00
Brian Coca
1367e73c94 be smarter when dealing with policy_json input
Now module will assume that if the argument is a string it is already formated as json
and will only try to convert non strings into json string.
Also removed unused 'msg' var declarations and the ifs that set them
fixes #2009
2016-12-08 11:23:56 -05:00
Vic Iglesias
353db36515 Update GCE PEM deprecation text and examples 2016-12-08 11:23:56 -05:00
Barnaby Court
73d47ec7a4 add no_log to the password parameter 2016-12-08 11:23:55 -05:00
Charles Paul
980691338a adding no_log to password 2016-12-08 11:23:55 -05:00
Michael Scherer
d321670b38 Add more precise documentation on the requirements parameter
Fix #427
2016-12-08 11:23:55 -05:00
Michael Scherer
83b45ae047 Fix doc inconsistency (#2630) 2016-12-08 11:23:55 -05:00
Vic Iglesias
612d9c3e92 Use credentials_file for GCE module auth config 2016-12-08 11:23:55 -05:00
tksarah
7ad225eedd Edit azure.py 2016-12-08 11:23:55 -05:00
Etherdaemon
eba78d08f9 Update to wait and wait_timeout with a maximum of 10 minutes timeout 2016-12-08 11:23:55 -05:00
luto
610de975e6 vsphere_guest: when creating a guest, gather facts about it *before* disconnect 2016-12-08 11:23:55 -05:00
Etherdaemon
cd3f306518 Proposing a wait_for function to ensure elb has been successfully removed 2016-12-08 11:23:55 -05:00
Josh Rendek
bddcf737c9 Fix quote on grants of more than one word
Fixed a quote location on the grants composed of more than one word. Current docs will raise an error with a invalid privilege command.
2016-12-08 11:23:55 -05:00
Mike Buzzetti
f20b09e2ec Add evaluate taget health for alias based records 2016-12-08 11:23:55 -05:00
sebi
76af56eaca added unless stopped option 2016-12-08 11:23:55 -05:00
Martin Matuska
4cce825ae9 rhn_register: add sslcacert and systemorgid options 2016-12-08 11:23:55 -05:00
Brian Coca
7d6c704404 fix bad linebreak 2016-12-08 11:23:55 -05:00
Michael Scherer
69036826a8 Fix #2629, replace choices=BOOLEANS by type='bool' 2016-12-08 11:23:55 -05:00
Alexander Gubin
b0d54092ee command: Add warnings for dnf and zypper 2016-12-08 11:23:55 -05:00
luto
d20c4368d2 vsphere_guest: add power status to facts 2016-12-08 11:23:55 -05:00
stephane
40e7ed9518 Add support for pip force-reinstall
The pip command allows a user to force reinstallation,
but the module doesn't currently support it. Add
"force-reinstall" as a possible state.
2016-12-08 11:23:54 -05:00
Manuel Tiago Pereira
057cbacf8a Add support for legacy registries to pull_image().
Since we now have several exceptions to the assumption that the
result of the pull would be on the last status line returned by
docker-py's pull(), I've changed the function so that it looks
through the status lines and returns what if finds on it.
Despite the repeated `break`s, the code seems simpler and a little
more coherent like this. From what I've checked using
`https://github.com/jlafon/ansible-profile`, the execution time is
mostly the same.
2016-12-08 11:23:54 -05:00
Sam Thursfield
eba767902c os_server: Add some error checking for the 'nics' parameter
If this parameter was not of the right type, the module would fail with a
traceback, with a "AttributeError: 'str' object has no attribute 'get'"
exception.

It now gives a proper error message on type errors.
2016-12-08 11:23:54 -05:00
nitzmahone
317ea13759 win_get_url doc/strict-mode fixes
plus cleaning up from bad merge
2016-12-08 11:23:54 -05:00
nitzmahone
feb077261e minor win_get_url doc update 2016-12-08 11:23:54 -05:00
T.Kuramochi
dfda2e9f57 Add HTTP Proxy options
Update a document file for win_get_url.ps1.

Update add a prefix proxy_ for this variables

Update a document file for win_get_url.ps1.

Update win_get_url.ps1 20150907
2016-12-08 11:23:54 -05:00
Toshio Kuratomi
7a492764d0 Fix up documentation 2016-12-08 11:23:54 -05:00
Toshio Kuratomi
4fa9de5af3 entrypoint feature added in 2.1 2016-12-08 11:23:54 -05:00
Toshio Kuratomi
eb65f819f5 labels is actually not aded in 1.9.4 but in 2.1 2016-12-08 11:23:54 -05:00
Andrew Pashkin
c811a3f4bc Add 'entrypoint' parameter to Docker module 2016-12-08 11:23:54 -05:00
Andrew Pashkin
8342397427 Harden matching running containers by "command" in the Docker module
Before this patch:

  - Command was matched if 'Command' field of docker-py
    representation of Docker container ends with 'command' passed
    to Ansible docker module by user.
  - That can give false positives and false negatives.
  - For example:
      a) If 'command' was set up with more than one spaces,
         like 'command=sleep  123', it would be never matched again
         with a container(s) launched by this task.
         Because after launching, command would be normalized and
         appear, in docker-py API call, just as 'sleep 123' - with one
         space. This is false negative case.
      b) If 'entrypoint + command = command', for example
         'sleep + 123 = sleep 123', module would give false positive
         match.

This patch fixes it, by making matching more explicit - against
'Config'->Cmd' field of 'docker inspect' output, provided by docker-py
API and with proper normalization of user input by splitting it to
tokens with 'shlex.split()'.
2016-12-08 11:23:54 -05:00
Omar Khan
369fe78c60 Add stop_timeout option to docker module 2016-12-08 11:23:54 -05:00
Toshio Kuratomi
a708ad65c4 Add version_added to documentation 2016-12-08 11:23:54 -05:00
Joe Quadrino
b6e2b2dce1 add devices parameter for docker module 2016-12-08 11:23:54 -05:00
Yann Hamon
1faf8dd5c2 Docker module: restarted should update the container when necessary 2016-12-08 11:23:54 -05:00
Andy Nelson
444895f889 Update to ec2_vpc.py to: 1 allow interface ids and vpc peering connections as route targets 2 set state to "terminated" when VPC is removed 3 fix some comment typos
updates per PR comments
2016-12-08 11:23:54 -05:00
HAMSIK Adam
2af105bcbc Convert enabled value to boolean to actually work, make sure we can set expiration period to 0(None) to disable it 2016-12-08 11:23:53 -05:00
Pedro Romano
bcae9a2b5e GCE libcloud 0.15 support and code cleanup * Code formatting (indentation and white space) fixes for improved PEP8 conformity. * Remove redundant backslashes inside parentheses. * Test for object identity should be 'is not None'. * Test for membership should be 'not in'. * Fit docstring to the PEP8 79 character limit. * Use forward compatible Python 2.6+ 'except .. as' syntax for exception handling. * Support libcloud > 0.15 'metadata' argument format. 2016-12-08 11:23:53 -05:00
Toshio Kuratomi
65b9243056 mariadb isn't always the last elemen of the version string 2016-12-08 11:23:53 -05:00
Toshio Kuratomi
0a64899eba Fix mysqldump usage of config_file 2016-12-08 11:23:53 -05:00
Brian Coca
f8a36c4047 service goes back to failing when absent
if no tools and no init script, this should always fail
2016-12-08 11:23:53 -05:00
Alberto Gireud
58c39f47c1 Update root_volume variable 2016-12-08 11:23:53 -05:00
Toshio Kuratomi
dafb39da2e Account for mariadb versioning 2016-12-08 11:23:53 -05:00
luto
d48dffc298 vsphere_guest: support putting a guest into a nested folder 2016-12-08 11:23:53 -05:00
Jonathan Mainguy
0cdb2719e5 Unify all 3 mysql modules.
Use same connection method, use config_file, and add ssl support
2016-12-08 11:23:53 -05:00
Toshio Kuratomi
8d293651df Fix os_server docs build 2016-12-08 11:23:53 -05:00
Lee H
7feb64d31c - add version_added as requested to host_all 2016-12-08 11:23:53 -05:00
Lee H
5606197c28 - remove user_anonymous as the same thing can be accomplished by user='', but leave in the examples for removing anonymous users 2016-12-08 11:23:53 -05:00
Donovan Jones
8b9cd7f410 fix typo in os_server security_groups description 2016-12-08 11:23:53 -05:00
Brian Coca
9259b51c97 updated module docs, added choices to state 2016-12-08 11:23:53 -05:00
Tobias Smolka
cf5403afa3 Making cluster parameter optional 2016-12-08 11:23:53 -05:00
Brian Coca
63fd67f156 added mime option to stat module
it uses file magic to now return mime_type and charset of a file as
per output of `file -i /path`
2016-12-08 11:23:53 -05:00
Lee H
95895177d0 - add example showing removal of anonymous user accounts 2016-12-08 11:23:52 -05:00
Monty Taylor
a715c91d0b Add support for network, boot_from_volume and volumes
nics is a great flexible parameter, but it's wordy. Shade now supports
a simple parameter too, which is just "network" and takes a name or id.
Add passthrough support.

In addition to supporting booting from a pre-existing volume, nova and
shade both support the concept of booting from volume based on an image.
Pass the parameters through.

Shade supports boot-time attachment of additional volumes for OpenStack
instances. Pass through the parameter so that ansible users can also
take advantage of this.
2016-12-08 11:23:52 -05:00
Toshio Kuratomi
687257bd62 Use rpm instead of repoquery for is_installed()
* This keeps us from hitting bugs in repoquery/yum plugins in certain
  instances (#2559).

* The previous is also a small performance boost

* Also in is_installed(), when using the yum API, return if we detect
  a package name has been installed.  We don't need to also check
  virtual provides in that case.  This is another small performance
  boost.

* Sort the list of packages returned by the list parameter.
2016-12-08 11:23:52 -05:00
Brian Coca
3b934016c0 note that create globs only work on 2.0
fixes #2666
2016-12-08 11:23:52 -05:00
Brian Coca
b4f87ac7fe made note that Z/z are only 2.1 options 2016-12-08 11:23:52 -05:00
Jonathan Mainguy
b86224a7ec Add Jmainguy as author, fix hash check 2016-12-08 11:23:52 -05:00
Adam Fields
9c131ea619 added a reference to the template module for clarity 2016-12-08 11:23:52 -05:00
Timothy R. Chavez
37ac954123 Get new server object after adding floating IP
We need a new server object once we add the floating ip, otherwise we
will be operating with the older server object pre-floating-ip
assignment.
2016-12-08 11:23:52 -05:00
Michel Alexandre Salim
325a979876 Set the argument type for ec2_vol's encrypted parameter
If this is not set, Ansible parses the parameter as a string.
This is fine if the parameter is not provided by the caller, but
if it is set to False or True explicitly, ec2_vol receives this as
the string 'False' or the string 'True', both of which are truthy.

Thus, without this fix, setting the parameter results in encryption
always enabled.
2016-12-08 11:23:52 -05:00
Veaceslav Mindru
fa98f55c44 fix typo s/defaults/default 2016-12-08 11:23:52 -05:00
Steve Spencer
d4aca8eacc Sync up with allowable docker volume mounting modes 2016-12-08 11:23:52 -05:00
Steve Spencer
769d7db1d0 Add support for mounting host volumes with Z and z options 2016-12-08 11:23:52 -05:00
Lee Hardy
f2f0f83ee6 - fix user_exists statement with host_all to use only username parameter 2016-12-08 11:23:52 -05:00
Leonty
22648e88cd Corrected misspelling in the 'labels' docker parameter documentation. 2016-12-08 11:23:52 -05:00
Lee Hardy
1113975741 - mysql: add user_anonymous parameter, which interacts with anonymous users - mysql; add host_all parameter, which forces iteration over all 'user'@... matches 2016-12-08 11:23:51 -05:00
Brian Coca
f435f887fa better handling of checkmode for solaris fixes #2296 2016-12-08 11:23:51 -05:00
Aaron Boushley
876bdb62e1 Fix issue with comparing versions improperly.
This allows old versions of docker api to function.
2016-12-08 11:23:51 -05:00
Leonty
8638d9d26c Support 'labels' parameter for docker. 2016-12-08 11:23:51 -05:00
Jumpei Ogawa
ebb0d6301d Add explanation in case that ipv4_range is required 2016-12-08 11:23:51 -05:00
Jumpei Ogawa
256c840942 Add better error message when specified network doesn't exist and ipv4_range is not specified 2016-12-08 11:23:51 -05:00
tobbe
65f12bccb6 add text to the description, more user friendly 2016-12-08 11:23:51 -05:00
tobbe
455da45e62 Add suport for selinux user when adding a new user on selinux enabled systems 2016-12-08 11:23:51 -05:00
Daniele Varrazzo
a057431a78 Detect unchanged pip runs when using a vcs url in name
Should fix bug #1645
2016-12-08 11:23:51 -05:00
Daniele Varrazzo
53e1310a7f Use 'pip freeze' output to detect changes with requirement specified
If the requirements contains a repos url it will always report 'Successfully
installed'; there is no difference in the output to tell apart if
anything new was pulled. Use freeze to detect if the environment changed
in any way.

Should fix ansible/ansible#1705
2016-12-08 11:23:51 -05:00
Derek Smith
b9baed507b updated examples
added mysql 5.7 user password modification support with backwards compatibility

resolved mysql server version check and differences in user authentication management

explicitly state support for mysql_native_password type and no others. fixed some failing logic and updated samples

updated comment to actually match logic.

simplified conditionals and a little refactor
2016-12-08 11:23:51 -05:00
trevoro
b5a3dd4818 adding password_hash support to mysql_user module
fixing user_add arguments error

fixing user_mod arguments error
2016-12-08 11:23:51 -05:00
Michael Scherer
55f2345829 Remove a unneeded use of use_unsafe_shell
Since use_unsafe_shell is suspicious from a security point
of view (or it wouldn't be unsafe), the less we have, the less
code we have to toroughly inspect for a security audit.
2016-12-08 11:23:51 -05:00
Daniel Kimsey
a8f7fda210 Fix yum module failing to initalize yum plugins 2016-12-08 11:23:51 -05:00
nitzmahone
0d0b78dd1e fixed disappearing groups on OSX user module
Ensure that we don't try to modify the groups collection if groups are not specified
2016-12-08 11:23:51 -05:00
Dominique Barton
5c05016d1e bugfix for issue #2537 2016-12-08 11:23:51 -05:00
quoing
3ad67d93ea Fix: Default ACL parameters are not correctly handled 2016-12-08 11:23:51 -05:00
Brian Coca
f0a3914f7b simplified lowercasing 2016-12-08 11:23:50 -05:00