Commit Graph

8983 Commits (330e3237053874040718d9dfe1591b2a167c302e)
 

Author SHA1 Message Date
Peter Sprygada 330e323705 removes state argument from ops_config module
this removes the state argument from ops_config.  The state argument should
not have been there
8 years ago
Peter Sprygada 1d48b47cad Merge pull request #4627 from privateip/ops_template
update ops_template module using refactored network shared modules
8 years ago
Peter Sprygada dd6751d60a Merge pull request #4612 from privateip/ops_command
added new functionality to ops_command
8 years ago
Peter Sprygada 3746ed6a63 Merge pull request #4611 from privateip/ops_facts
update ops_facts with enhancements
8 years ago
Peter Sprygada 244c626822 Merge pull request #4610 from privateip/ops_config
update ops_config module with new enhancements
8 years ago
Matt Clay 95353ded58 Add python3 testing for module PRs. (#4629) 8 years ago
Tobias Wolf 34201d6bc4 Fix database table quoting in privileges_unpack() (#3858)
In Ansible 2.x this module gives `changed = True` for all privileges
that are specified including a table with

    priv: "database.table:GRANT"

Mysql returns escaped names in the format

    `database`.`tables`:GRANT

However in PR #1358, which was intended to support dotted database names
(a crazy idea to begin with), the quotes for the table name were left
out, leading to `curr_priv != new_priv`.

This means that the idempotency comparison between new_priv and
curr_priv is always 'changed'.

This PR re-introduces quoting to the table part of the priv.
8 years ago
Jean Prat 86366eca68 if user is empty, it is not converted to tuple when using host_all (#3038) 8 years ago
James Cammarata e7cf786851 Adding 'end_play' docs to meta module 8 years ago
Peter Sprygada 9c64d1947c update ops_template module using refactored network shared modules
This updates the ops_template module to work with the network shared
modules introduced in Ansible 2.2

Tested with OpenSwitch 0.4.0
8 years ago
Peter Sprygada b99cad4128 added new functionality to ops_command
* commands argument now accepts a dict arguments
* waitfor has been renamed to wait_for with an alias to waitfor
* only show commands are allowed when check mode is specified
* config mode is no longer allowed in the command stack
* add argument match with valid values any, all

Tested on OpenSwitch 0.4.0
8 years ago
Peter Sprygada 6f9ab60f81 Merge pull request #4615 from ansible/gundalow-patch-2
Consistent naming of Arista EOS device
8 years ago
Toshio Kuratomi 958d894c61 We've decided that pythn 3.5 is the minimum python3 version (#4572) 8 years ago
Michael Scherer 6dd2bc2bff Make async work on python 3 (#4583)
Since dict no longer have a iteritems method, we have to
use six to support python 2 and 3.
8 years ago
Michael Scherer cee7473df6 Port mount.py to python3, need to use six.iteritems (#4581) 8 years ago
Michael Scherer 1acb23f8d6 Fix uri to run on python3 (#4580)
Since dict no longer have a method iteritems, we have to use
the six wrapper.
8 years ago
Michael Scherer 2e7cd6e02a Port postgresql module to python3 (#4579)
Iteritems is no longer a dict method in Python3, replace it with
the six wrapper.
8 years ago
Michael Scherer c091a3e9ac Fix mysql_user for python3 (#4576)
dict no longer have a iteritems method, it was replaced
by items. So we need to use six.

    Traceback (most recent call last):
      File \"/tmp/ansible_hjd7d65c/ansible_module_mysql_user.py\", line 587, in <module>
        main()
      File \"/tmp/ansible_hjd7d65c/ansible_module_mysql_user.py\", line 571, in main
        changed = user_add(cursor, user, host, host_all, password, encrypted, priv, module.check_mode)
      File \"/tmp/ansible_hjd7d65c/ansible_module_mysql_user.py\", line 239, in user_add
        for db_table, priv in new_priv.iteritems():
    AttributeError: 'dict' object has no attribute 'iteritems'
8 years ago
Michael Scherer 38992bbd57 Fix user module under python3 (#4560)
Using something like:

    - name: Create ssh keys
      user:
        name: root
        generate_ssh_key: yes
      register: key

result into this traceback on F24

    Traceback (most recent call last):
      File \"/tmp/ansible_jm5d4vlh/ansible_module_user.py\", line 2170, in <module>
        main()
      File \"/tmp/ansible_jm5d4vlh/ansible_module_user.py\", line 2108, in main
        (rc, out, err) = user.modify_user()
      File \"/tmp/ansible_jm5d4vlh/ansible_module_user.py\", line 660, in modify_user
        return self.modify_user_usermod()
      File \"/tmp/ansible_jm5d4vlh/ansible_module_user.py\", line 417, in modify_user_usermod
        has_append = self._check_usermod_append()
      File \"/tmp/ansible_jm5d4vlh/ansible_module_user.py\", line 405, in _check_usermod_append
        lines = helpout.split('\\n')
    TypeError: a bytes-like object is required, not 'str'
8 years ago
Michael Scherer 6063071a46 Fix traceback on python3 (#4556)
Traceback (most recent call last):
  File "/tmp/ansible_csqv781s/ansible_module_systemd.py", line 374, in <module>
    main()
  File "/tmp/ansible_csqv781s/ansible_module_systemd.py", line 263, in main
    for line in out.split('\\n'): # systemd can have multiline values delimited with {}
8 years ago
Peter Sprygada a73d32c33a Merge pull request #4588 from privateip/eos_eapi
update eos_eapi module with new enhancements
8 years ago
Peter Sprygada c215398e2f update eos_eapi module with new enhancements
* add support for vrf configurations
* add support for configing the qos value for eapi
* add config argument to specify the device running-config

Tested on EOS 4.15.4F
8 years ago
John R Barker ec1c490888 Consistent naming of Arista EOS device 8 years ago
Peter Sprygada cbbb4af99a update ops_facts with enhancements
* adds support for default facts subset
* adds support for config facts subset
* maintain legacy facts from ops_facts pre-2.2

Tested on Openswitch 0.4.0
8 years ago
Peter Sprygada 2133b92980 update ops_config module with new enhancements
* add src argument to provide path to config file
* add new choice to match used to ignore current running config
* add update argument with choices merge or check
* add backup argument to backup current running config to control host
* add save argument to save current running config to startup config
* add state argument to control state of config file
* deprecated force argument, use match=none instead

Note: this module only supports transport=cli

Tested on OpenSwitch 0.4.0
8 years ago
Peter Sprygada 38fd67bead Merge pull request #4608 from privateip/ios_command
removes output keyword from command in ios_command
8 years ago
Peter Sprygada a6ffe2e7be removes output keyword from command in ios_command
IOS devices only support a single command output which is structured
text.  This removes the ability to specify the command output format
when providing complex arguments to the commands
8 years ago
Peter Sprygada 6543bb4bdd update nxos_nxapi module with minor enhancements (#4573)
* added new config argument
* added states present and absent
* update to use network shared modules

Tested on NXOS 7.3(0)D1(1)
8 years ago
Timothy Appnel 2538c70d46 Fixes get_url examples in docs and applies native YAML syntax. (#4474) 8 years ago
John R Barker 0c25e968ee digital_ocean_tag name is a required field
Spotted during final review of https://github.com/ansible/ansible-modules-core/pull/4218/files
Simple to change, so fixing post merge
8 years ago
Victor Volle c170107eef Digitalocean tags (replaces #4209) (#4218)
* Fixes #4117: Add DigitalOcean Tag support

* Add GPLv3 license header and RETURN documentation

* ansible.module_utils.urls instead of "requests"
8 years ago
Arun prasath 0d98760b49 Fixes bug #4546 - Made password optional (#4574) 8 years ago
David J Peacock 8f963a7028 Fix #4412: os_security_group_rule support numbered protocols (#4444) 8 years ago
Ryan Brown 0c37949941 Remove spurious `changed` state on iam_policy module (#4381)
Due to a mixup of the group/role/user and policy names, policies with
the same name as the group/role/user they are attached to would never be
updated after creation. To fix that, we needed two changes to the logic
of policy comparison:

- Compare the new policy name to *all* matching policies, not just the
  first in lexicographical order
- Compare the new policy name to the matching ones, not to the IAM
  object the policy is attached to
8 years ago
James S. Martin 978716cf4c Shows messages for uncaught exceptions from called modules in async_wrapper output. (#4591) 8 years ago
Rick Mendes 02c47f5b0c Fixes #3144 (#4305) 8 years ago
Vytis Banaitis c6c281b125 Fix change status on django_manage collectstatic. (#4239)
Ignore blank lines in collectstatic output.
8 years ago
Guillaume Delpierre 2a25e27979 gem: add support for --env-shebang (#4377)
* gem: add support for --env-shebang

* fix version added
8 years ago
Levi Morales 688fa6fc54 Fixes #2322 fix for missing append check on Darwin systems (#4256) 8 years ago
Evan Kaufman 5728ef89f0 Implemented file content diff for replace module (#4479) 8 years ago
Matías Pizarro 521c9a2e20 Use named parameters in new_droplet method call (#4483)
This mirrors the original method signature and guards against any
change in order parameter
8 years ago
Anton Onufriev 2ca16dfaad Remove colors from git-branch output for correct local search (#4545)
We got an error while switching on existent local branch
because git module can not find branch in function get_branches
if we have color.branch=always in git config.
8 years ago
Michael Scherer 5310bab12f Port the module to run on python 3 (#4496)
One of the usual issue is that run_command return bytes,
so we have to adapt the string to either be bytes too,
or convert to string.

This result into that kind of traceback:

    Traceback (most recent call last):
      File \"/tmp/ansible_ej32yu2w/ansible_module_git.py\", line 1009, in <module>
        main()
      File \"/tmp/ansible_ej32yu2w/ansible_module_git.py\", line 873, in main
        git_version_used = git_version(git_path, module)
      File \"/tmp/ansible_ej32yu2w/ansible_module_git.py\", line 788, in git_version
        rematch = re.search('git version (.*)$', out)
      File \"/usr/lib64/python3.5/re.py\", line 173, in search
        return _compile(pattern, flags).search(string)
    TypeError: cannot use a string pattern on a bytes-like object

Another issue is filter being a object instead of a list.
8 years ago
Peter Sprygada 12122952db Merge pull request #3950 from grybak-arista/eos_exit_fix
Move call to filter_exit out of command execution block
8 years ago
James Pharaoh 5bce0049a1 fix error in documentation for stat checksum (#4582) 8 years ago
Peter Sprygada 9e97431308 Merge pull request #4578 from privateip/nxos_facts
convert nxos_facts map from dict to frozenset
8 years ago
Ilja Bauer acb7d873f8 Replaced use of bare variables with full variable syntax (#4149) 8 years ago
Gennady Trafimenkov 6ac58a5832 Clarify what checksum algorithm is used by copy modules (#4198) 8 years ago
Ali Ok ab55355b00 Fix minor syntax error in authorized_key module doc (#4491) 8 years ago
Robin Roth 9da4c42679 Clarify that refspec is needed to checkout a git hash (#4564)
fixes #1531
8 years ago