Commit Graph

5338 Commits (1de2028677bc1eda2444f54f25cfab122c221147)
 

Author SHA1 Message Date
Stoned Elipot c9f268ac48 Fix typo in git module documentation preventing it to be processed by ansible-doc 12 years ago
Stoned Elipot 81e0a7adcc Fix typo in npm module documentation preventing it to be processed by ansible-doc 12 years ago
Nicolas Grilly 75ceb80572 Escape args injected in new style modules 12 years ago
Scott Sturdivant 922a1efbc0 Fixes #2924: User creation always reports 'changed' 12 years ago
Michael Vogt 22b0cf9275 use the same text for -M in ansible.1 and ansible-playbook.1 12 years ago
Michael Vogt 8338e75463 document in the man-page that {ansible,ansible-playbook} -M can also be overriden with the ANSIBLE_LIBRARY environment 12 years ago
Scott Sturdivant 5516ece859 Ensure the tmp_path is sufficiently open. 12 years ago
Nicolas Grilly 7aee588918 Idiomatic Python: use in operator instead of method find 12 years ago
Nicolas Grilly 7babd30cf7 Idiomatic Python: use isinstance instead of type 12 years ago
James Martin 3c131dbd2b Initial Commit. 12 years ago
Scott Sturdivant b2542bff72 Chdir to tempdir to prevent permissions problems. 12 years ago
Scott Sturdivant d91bc1692e Make pkgng and with_items result in a single module call. 12 years ago
Scott Sturdivant b274fb9774 Fixes #2991: Service module pattern argument ignored. 12 years ago
Lorin Hochstein dbd9d92861 OpenStack Identity (keystone) user module
Module for managing users, tenants, and roles within OpenStack.
12 years ago
Kristof Keppens e2e10c1977 fix for facter being installed in a different location 12 years ago
Kent Frazier 4c6583bd00 Fix bug with include-level vars and sudo_user.
If a variable was provided for an include, in either of these ways:

    ---
    - hosts: all
      tasks:
      - include: included.yml param=www-data
      - include: included.yml
        vars:
          param: www-data

and then that param was used as the value of sudo_user in the included
tasks:

    ---
    - name: do something as a parameterized sudo_user
      command: whoami
      sudo: yes
      sudo_user: $param

you would receive a "failed to parse: usage: sudo" error back and the
command would not execute.

This seemed to be due to a missing call to template.template somewhere,
because the final value being passed through ssh was still `$param`.
After some digging, the issue seems to instead have been a problem with
providing the wrong context to the template for expansion. Inside the
`Task` logic, it was passing `play.vars` as the context, where
`module_vars` seemed more appropriate. After replacing it, my test case
above ran without issue. There was a comment above suggesting that the
template call might be unnecessary, but removing it made the original
error return, since it is not getting escaped later down the line. I
removed the comment since it was inaccurate.

I tried to actually incorporate my test case above into the test suite
as a regression test, but was unable to figure out how to structure it.
The existing test infrastructure seemed to only be testing for correct
number of counts in things (ok vs. changed, etc.), without regard for
whether the content generated by the command is correct. If there is an
example of a test similar to this one (where I would want to check the
JSON generated to make sure sudo_user had been converted), please let me
know and I will be happy to submit an additional patch.
12 years ago
William Bowling 222f004bfa Fixes apt_repository always reporting file uri repos as changed 12 years ago
bennojoy b16e2a1d48 OpenStack Modules 12 years ago
Mark Mandel f2f3fb1a8e Vagrant external inventory file, to dynamically determine your vagrant vm ip. 12 years ago
Mark Mandel 0ab58774bd Ignore Intellij IML files. 12 years ago
Bryan Larsen f629baef11 add basename and dirname to the core jinja2 filters 12 years ago
Stoned Elipot d4d0753e03 Allow virtualenv_command argument of the pip module to be a pathname 12 years ago
Stoned Elipot 83cce7d2e0 Fix pip module documentation (YAML quoting) 12 years ago
Michael DeHaan 8f3b2b281f Merge pull request #2967 from jonEbird/devel
Inventory script for Spacewalk servers based on the spacewalk-reports pa...
12 years ago
Jon Miller dc8feb799c Do not need to chmod cache file each time
This is particularly important if you've manually ensured you have write
access via group permissions and not by owner.
12 years ago
Michael DeHaan aecc954aa4 Merge pull request #2965 from bpennypacker/devel
newrelic_deployment bug fix to support both python 2.4 and 2.6
12 years ago
Jon Miller 5111e0c2f7 Inventory script for Spacewalk servers based on the spacewalk-reports package 12 years ago
Bruce Pennypacker d4429fcb91 Fixed handling of urlopen result to work properly under both python 2.4 and python 2.6 12 years ago
Michael DeHaan 753ebea186 Prefer categorized modules to uncategorized, such that if someone does an install on top of old content
the categorized modules will win over the 1.1 uncategorized modules, as the packages may not automatically
clean up the old content.
12 years ago
Michael DeHaan cd57d59913 If ansible is configured to use a log file but cannot write to it, exit, but don't traceback. 12 years ago
Michael DeHaan 496f06c3c9 Pass variables to lookup function for those that want to use them. 12 years ago
Michael DeHaan 16709ecd76 Replace tab with spaces. 12 years ago
Dimos Alevizos e96bc981ce Set paramiko's logging level
Excplicity set paramiko's logging level to WARNING.
By default it inherits ansible's DEBUG logging level (set in
callbacks.py) and fills the log file with useless debug messages.
Obviously it only applies if log_path is set in ansible.cfg
12 years ago
Les Aker 1179d1bbd8 corrected indentation 12 years ago
Les Aker 2adc335481 ignore unicode errors in stdout 12 years ago
Michael DeHaan 14aa54172c Merge pull request #2955 from jarv/devel
pip module blowing up when name is not included in arg list
12 years ago
John Jarvis daf4c358f7 fixes case where name is omitted from pip arg list
This code:
```
    if name.endswith('.tar.gz') or name.endswith('.tar.bz2') or
name.endswith('.zip'):
        is_tar = True
```
was not checking whether name is defined since it is an
optional param.
12 years ago
Michael DeHaan 7dfb29f8f9 Merge pull request #2952 from mmoya/bug-fixes
Don't hardcode chroot path
12 years ago
Michael DeHaan e53a9e5743 Merge pull request #2954 from jarv/devel
fixing syntax error
12 years ago
John Jarvis 423fe82d94 fixing syntax error 12 years ago
Elan Ruusamäe d3a2b1dbbb no need to init None to None 12 years ago
Maykel Moya 60f24bb077 Remove unused modules 12 years ago
Maykel Moya f52e3dee70 Don't hardcode chroot executable path 12 years ago
Michael DeHaan 9db256adce Merge pull request #2951 from DazWorrall/vim_swp
Add vim swap files to gitignore
12 years ago
Darren Worrall 0a28d26d6e Add vim swap files to gitignore 12 years ago
Michael DeHaan 5f98c6c246 Merge pull request #2948 from jcbagneris/devel
Fixes in apt module doc: typos, examples
12 years ago
Michael DeHaan 206b34783f Merge pull request #2942 from stoned/doc-requirements
Fix empty requirements specifications in modules documentation
12 years ago
Michael DeHaan 328065506e Merge pull request #2941 from bcoca/setup_bin
no uses get_bin_path for 2nd use of ip tool
12 years ago
Michael DeHaan 39673fe8c0 Merge pull request #2940 from madema/patch-3
-o and -r option don't exist on AIX
12 years ago
Ben Ritcey 0938d4d89a Allow the remote MD5 checks to be run via sudo 12 years ago