Commit graph

27381 commits

Author SHA1 Message Date
Berislav Lopac
7f351a46e5 replace iteritems with items to ensure python3 compatibility 2016-12-08 11:35:10 -05:00
Andy Dirnberger
3a26a1bfcc Make Homebrew-related modules run on Python 3
Both the `homebrew` and `homebrew_cask` modules iterate over
dictionaries using `iteritems`. This is a Python 2-specific method whose
behavior is similar to `items` in Python 3+. The `iteritems` function in
the six library was designed to make it possible to use the correct
method.
2016-12-08 11:35:09 -05:00
Michael Scherer
64c994c641 Make the 'make' module run on python 3
Traceback:

    Traceback (most recent call last):
      File \"/tmp/ansible_d28_6uwl/ansible_module_make.py\", line 153, in <module>
        main()
      File \"/tmp/ansible_d28_6uwl/ansible_module_make.py\", line 119, in main
        rc, out, err = run_command(base_command + ['--question'], module, check_rc=False)
      File \"/tmp/ansible_d28_6uwl/ansible_module_make.py\", line 79, in run_command
        return rc, sanitize_output(out), sanitize_output(err)
      File \"/tmp/ansible_d28_6uwl/ansible_module_make.py\", line 95, in sanitize_output
        return output.rstrip(b(\"\\r\\n\"))
    TypeError: rstrip arg must be None or str

There is also a six.iteritems issue, fixed using six.
2016-12-08 11:35:09 -05:00
Rene Moser
583944207d redshift: fix version added 2016-12-08 11:35:09 -05:00
Koki Nomura
8ccf166a56 blockinfile: Fixes #1926 by comparing a marker to a whole line instead of a line prefix (#3339) 2016-12-08 11:35:09 -05:00
Marius Gedminas
cffd5601b6 letsencrypt: Locale-independent date parsing (#3314)
Should fix #3155.
2016-12-08 11:35:09 -05:00
Jiri Tyr
9453f6b97a yum_repository: Add diff support (#3460) 2016-12-08 11:35:09 -05:00
Jens Carl
90e0dc2a89 redshift: Fix error with boolean parameters (#3467) 2016-12-08 11:35:09 -05:00
Matthew Krupcale
cb765ce590 gluster_volume: Fixes gluster peer probe / volume creation issue. (#3486)
* gluster_volume: Fixes issue when creating a new volume failing due to peers not being present. The peers which are not 'localhost' should invoke wait_for_peer, but the find method returns -1 (not 0) on non-localhost peers.
2016-12-08 11:35:09 -05:00
Dag Wieers
c8a700834c Performance improvement using in-operator for hash lookups
Just a small cleanup for the existing occurrences.

Using the in-operator for hash lookups is faster than using .has_key()
http://stackoverflow.com/questions/1323410/has-key-or-in
2016-12-08 11:35:09 -05:00
Dag Wieers
a417a4f4b3 Performance improvement using in-operator on dicts
Just a small cleanup for the existing occurrences.

Using the in-operator for hash lookups is faster than using .keys()
http://stackoverflow.com/questions/29314269/why-do-key-in-dict-and-key-in-dict-keys-have-the-same-output
2016-12-08 11:35:09 -05:00
Nijin Ashok
b06003e5d2 Fix issue in adding RAW disk in block storage domain (#3432)
By default, sparse option is true in ovirt. So the raw disk
    creation in a block storage domain will fail with error "Disk
    configuration (RAW Sparse) is incompatible with the storage domain
    type".

    The commit adds sparse option where it is send as False when
    format is raw and True when format is qcow2
2016-12-08 11:35:09 -05:00
Maarten Bezemer
dd51ec94e7 mongodb_user: Allow pymongo version 3.2 in combination with mongodb 3.2 (#3474) 2016-12-08 11:35:09 -05:00
René Moser
ae9bc0cded cloudtack: cs_instance: doc fix for missing tag alias (#3484) 2016-12-08 11:35:09 -05:00
René Moser
a0c8b598ff cloudstack: fix state=absent, ip_address not None but falsy (#3483) 2016-12-08 11:35:09 -05:00
Abdoul Bah
318a06f6d6 New module proxmox_kvm (#3292)
* New module proxmox_kvm

* fixed qxl value vor vga param

> | Name | Type | Format | Description |
> |------|------|--------|-------------|
> | vga | enum | std \| cirrus \| vmware \| qxl \| serial0 \| serial1 \| serial2 \| serial3 \| qxl2 \| qxl3 \| qxl4 | Select the VGA type. If you want to use high resolution modes (>= 1280x1024x16) then you should use the options 'std' or 'vmware'. Default is 'std' for win8/win7/w2k8, and 'cirrus' for other OS types. The 'qxl' option enables the SPICE display sever. For win* OS you can select how many independent displays you want, Linux guests can add displays them self. You can also run without any graphic card, using a serial device as terminal. |

* Fix create_vm() fail on PV 4.3

* Set default for force as null in doc

* proxmox_kvm: revision fixes

* proxmox_kvm: more revision fixes

* Fix indentation

* revision fixes

* Ensure PEP-3110: Catching Exceptions

* KeyError, to KeyError as -- PEP-3110: Catching Exceptions

* Fix Yaml document syntax; Notes: => Notes -

* Refix documentation issue

* Fix Documentation

* Remove Notes: in description

* Add current state and it return value

* Update documentation

* fixed local variable 'results' referenced before assignment

* Fix fixed local variable 'results' referenced before assignment

* minor fixes in error messages

* merge upstream/devel int devel

* minor fixes in error messages

* Fix indentation and documentation

* Update validate_certs description
2016-12-08 11:35:09 -05:00
Brian Haggard
84e79770b3 Required and default are mutually exclusive (#3475) 2016-12-08 11:35:09 -05:00
René Moser
72eb8c7be9 ipify_facts: implement timeout (#3369) 2016-12-08 11:35:08 -05:00
jctanner
d2bb9c1298 vmware_guest: Fix the esxi_hostname docstring to match the arg dict's required= (#3479)
Fixes #3476
2016-12-08 11:35:08 -05:00
Davis Phillips
c2dcf62aee removed tabs and fixed formatting 2016-12-08 11:35:08 -05:00
Davis Phillips
24e8428a7b mend 2016-12-08 11:35:08 -05:00
Davis Phillips
e51f6c18eb added vDS support for config spec and note/annotation support 2016-12-08 11:35:08 -05:00
Toshio Kuratomi
4d3fec908b Remove need for translate/maketrans due to py3 differences
* translate() has a different api for text vs byte strings
* maketrans must be imported from a different location on py2 vs py3

Since this is such a small string outside of a loop we don't have to
worry too much about speed so it's better to have a single piece of code
that works on both py2 and py3
2016-12-08 11:35:08 -05:00
Abdul-Anshad-A
c2f4e7c852 Initial effort for pyVmomi snapshot ops
Make description optional during snapshot create
2016-12-08 11:35:08 -05:00
Davis Phillips
ce7c7b6804 Removed list value from description in dns_servers 2016-12-08 11:35:08 -05:00
Davis Phillips
bb017c4af8 updated version_added to 2.3 2016-12-08 11:35:08 -05:00
Davis Phillips
ae9aa2390f Added version_added to all the new params 2016-12-08 11:35:08 -05:00
Davis Phillips
ebdd9cf1a5 remove commented code and fixed formatting 2016-12-08 11:35:08 -05:00
Davis Phillips
995cc2cd2d closes 3305 adds customize support 2016-12-08 11:35:08 -05:00
Rene Moser
e1d79f341e cloudstack: new module cs_vpc 2016-12-08 11:35:08 -05:00
René Moser
8873263894 cloudstack: add new module cs_nic (#3392) 2016-12-08 11:35:08 -05:00
Fabio Alessandro Locati
9a1b5b5e18 Use native YAML (#3435) 2016-12-08 11:35:08 -05:00
Tristan Bessoussa
f68a45ca9a Fixed wrong variable name in the let's encrypt doc (#3398)
It could lead to unwanted error when dummy-paste to try this module.
2016-12-08 11:35:07 -05:00
James Hart
276c3410a5 consul: Pass through service_id if specified on a check (#3295)
Fixes #3249
The python-consul library already supports this, so it is just a simple
case of enablement.
This does not break the current logic in `add` of parsing as a check,
then parsing as a service if that fails… because service_name is
mandatory on a service registration and is invalid on a check
registration.
2016-12-08 11:35:07 -05:00
Fabio Alessandro Locati
39f4e17838 Improve yaml code and add an if to make the import more standard (#3424) 2016-12-08 11:35:07 -05:00
Fabio Alessandro Locati
7af000bdb1 Use native YAML (#3436) 2016-12-08 11:35:07 -05:00
Fabio Alessandro Locati
1e00adefd5 Use native YAML in the examples (#3441) 2016-12-08 11:35:07 -05:00
Matthieu Dolci
8822021132 ec2_customer_gateway: state present expect bgp_arn instead of bgp_asn (#3366) 2016-12-08 11:35:07 -05:00
Fabio Alessandro Locati
69fa44b6d1 Use native YAML (#3437) 2016-12-08 11:35:07 -05:00
Fabio Alessandro Locati
616c86529f Use native YAML (#3438) 2016-12-08 11:35:07 -05:00
Fabio Alessandro Locati
be834293ed Use native YAML (#3439) 2016-12-08 11:35:07 -05:00
Fabio Alessandro Locati
0a338a11c2 Normalize yaml example (#3426) 2016-12-08 11:35:07 -05:00
Fabio Alessandro Locati
df6200accc Normalize yaml example (#3425) 2016-12-08 11:35:07 -05:00
Fabio Alessandro Locati
07d96b62b5 Use native YAML (#3434) 2016-12-08 11:35:07 -05:00
Fabio Alessandro Locati
47297bb19c Use native YAML (#3433) 2016-12-08 11:35:07 -05:00
mattwwarren
727e533bfc ec2_elb_facts: fix errors with no names input (#3381)
* None being passed around results in a Bad Time (tm)

* need to return the full set of elbs for an empty list

* logic is hard
2016-12-08 11:35:07 -05:00
Jose Armesto
14fee8d5f7 ec2_lc_find: Set default value for AssociatePublicIpAddress, since is missing when is false (#3417) 2016-12-08 11:35:07 -05:00
Thomas Krahn
9814136d3e Ipa bugfixes (#3421)
* ipa_group: Fix: 'list' object has no attribute 'get'

* ipa_hbacrule: Fix: 'list' object has no attribute 'get'

* ipa_host: Fix: 'list' object has no attribute 'get'

* ipa_hostgroup: Fix: 'list' object has no attribute 'get'

* ipa_role: Fix: 'list' object has no attribute 'get'

* ipa_sudocmd: Fix: 'list' object has no attribute 'get'

* ipa_sudocmdgroup: Fix: 'list' object has no attribute 'get'

* ipa_sudorule: Fix: 'list' object has no attribute 'get'

* ipa_user: Fix: 'list' object has no attribute 'get'

* ipa_sudorule: Fix: invalid 'cn': Only one value is allowed

* ipa_hostgroup: module returns changed if assigned hosts or hostgroups are not in lowercase
2016-12-08 11:35:06 -05:00
Jason Ritchie
0f6aa07d88 Detect and correct PowerShell mishandling nssm Unicode output as UTF8 (#2867)
* extra detail on which step triggered 'change', detect and handle powershell mishandling nssm's unicode as utf8

* Simpler handling of nssm output encoding

Thanks @nitzmahone for a cleaner way to control PowerShell's behavior
2016-12-08 11:35:06 -05:00
Henrique Rodrigues
54c5ebf697 New ec2_group_facts module to be able to get facts from EC2 security groups (#2591)
Add `ec2_group_facts` module to gather facts from EC2 security groups
2016-12-08 11:35:06 -05:00