Commit Graph

908 Commits (ca042dd71920dde33e779195907190b0006c0992)
 

Author SHA1 Message Date
Felix Ingram 3a409a457c Add an "executable" option to the command and shell modules
The option will be passed to the Popen object created and will be used to
execute the command instead of the default shell.
12 years ago
Dag Wieers a5915b3803 Make module output more consistent wrt. changed/failed
- Make sure exit_json() always returns a changed= value
- Modify the yum module to not return failed=False
- Modify install() and latest() similar to remove() in yum module
- Changed exit_json(failed=True, **res) into a fail_json(**res)
- Make sure yum rc= value reflects loop (similar to how we fixed remove())
12 years ago
Stephen Fromm cbebdf744b Wrap getpwnam in try/except in authorized_key module
This also adds module as an argument to keyfile().
12 years ago
Stephen Fromm 88e4be48a3 Rewrite switch_version() to detect branch
Rewrote switch_version() to read .git/HEAD to find branch associated
with HEAD.  If in a detached HEAD state, will read
.git/refs/remotes/<remote>/HEAD.
12 years ago
Dag Wieers f46db974bf Turn 'fail' and 'debug' modules into action_plugins
This makes more sense as there is no purpose in running these remotely.
12 years ago
Michael DeHaan d9a8b1ca95 Merge pull request #1546 from dagwieers/get_url-nonthirsty-fix
Don't require thirsty mode for directory destination
12 years ago
Stephen Fromm 0480443759 Use supplied remote name when cloning git repository 12 years ago
Stephen Fromm 3ec51ce587 Checkout branch before rebase in switch_version 12 years ago
Stephen Fromm d6f7ffd4bc More work in git module
Rename pull() to fetch().  It does a git fetch and then a
git fetch --tags.
Add _run() method to handle all subprocess.Popen calls.  Change
all previous calls to subprocess.Popen to use _run().
12 years ago
Brian Coca 924dc5006c fix case in which file is created and backup is requested but fails, not it just doesn't attempt to backup the missing file
Signed-off-by: Brian Coca <briancoca+ansible@gmail.com>
12 years ago
Dag Wieers 3b7feb8980 Don't require thirsty mode for directory destination
There is no need to require thirsty mode when the destination is a directory. We add the basename of the url to the destination directory and proceed with that. If that file exists in non-thirsty mode continue as expected.

I also cleaned up some of the logic that is no longer necessary if we simply rewrite the destination from the very start the way it is expected.
12 years ago
Fabian Arrotin 1151381ab1 Fixed the way dom0/domU xen role is defined in the setup module 12 years ago
Stephen Fromm f49d2bc557 Fix to user module to use self when calling group_exists 12 years ago
Daniel Hokka Zakrisson 8fba4cc04f Keep fireball from spreading its socket around 12 years ago
Nigel Metheringham fc3b04e281 FreeBSD user mod only fires when there are changes
Also fix missing -G on groups change
12 years ago
Nigel Metheringham c6c7189e23 Take SHADOWFILE from existing user object rather than class 12 years ago
Nigel Metheringham ed5bf78485 Use the existing framework when running ssh_keygen 12 years ago
Nigel Metheringham eecf6cf94f Ensure user owns their ssh key directory on creation
Lost section from commit 8049777
12 years ago
Michael DeHaan bb2929b508 Merge branch 'playbook_ssh_ask_pass' of git://github.com/clatour/ansible into devel
Conflicts:
	examples/ansible.cfg

+ doc tweak
12 years ago
Michael DeHaan 15af4c63e5 Merge branch 'service_bsd' of git://github.com/bcoca/ansible into devel 12 years ago
Michael DeHaan ade071eb29 Fix user example 12 years ago
Michael DeHaan 78669c57d5 Module documentation should have options sorted 12 years ago
Michael DeHaan 61a4674413 Merge branch 'nigelm_freebsd' of git://github.com/nigelm/ansible into merge-service
Make things more reusable, correct some errors along the SSH key path

Conflicts:
	library/user
12 years ago
Nigel Metheringham 871d3afdf4 Added some documentation to the classes 12 years ago
Daniel Hokka Zakrisson 1ec38eb435 Fix service module for varnish initscript in EPEL
It outputs 'dead but pid file exists'.
12 years ago
Brian Coca e18ef3d640 Now can handle rc.conf and init.d/rc.d based systems
Signed-off-by: Brian Coca <briancoca+ansible@gmail.com>
12 years ago
Dag Wieers 1954e60e95 Use return codes > 255 for Ansible-specific errors.
I had made and pushed this change after you already pulled the request.

@dhozac indicated that it would probably be better to use return codes > 255 for anything related to Ansible itself. Which makes sens :)
12 years ago
Michael DeHaan 8d3f73d562 pep8 fixes 12 years ago
Michael DeHaan 958b2909f8 Rename 'domainname' to 'domain' 12 years ago
Michael DeHaan d7f740ffbe Merge pull request #1490 from sfromm/issue1484
Make sure ownership is set correctly for user's new ssh keys
12 years ago
Michael DeHaan 614bbcc5a7 Merge pull request #1488 from dagwieers/lineinfile-create
Add option create= to lineinfile module
12 years ago
Michael DeHaan 1a70cf965c Merge pull request #1487 from skvidal/devel
fix up remove and fix fixme checking to see if pkg was removed
12 years ago
Michael DeHaan 125a30eef7 Merge pull request #1474 from jhoekx/group-by-plugin-template
Add a group_by action plugin.
12 years ago
Serge van Ginderachter 4ebea7acba Add domainname to ansible facts 12 years ago
Serge van Ginderachter 8a823acf20 Fix typo 12 years ago
Dag Wieers 835d7d854e Add option create= to lineinfile module
We use the lineinfile module to modify configuration files of a proprietary application. This application reads configuration options from files, but does not require those files to exist (if the default options are fine). However this application may modify the configuration file at will, so we cannot copy or template those files. And after a silent install the configuration may not exist (depending on the response file).

Whatever the case, during deployment we need to make sure some configuration options are set after the installation.

So the cleanest way to handle this situation is to allow the lineinfile module to create the file if it is missing (and this is the expected behavior). When I proposed this behavior, @sergevanginderachter needed the same functionality and was now working around it as well.
12 years ago
Stephen Fromm f0d2a1260b Make sure ownership is set correctly for user's new ssh keys 12 years ago
Michael DeHaan 86e744d658 Merge pull request #1480 from dagwieers/command-fix2
Use return codes > 255 for Ansible-specific errors.
12 years ago
Michael DeHaan 94c1ba753a Merge pull request #1479 from wzzrd/selinux
More useful error message for if not HAVE_SELINUX
12 years ago
Dag Wieers f26148d4b9 Use return codes > 255 for Ansible-specific errors.
I had made and pushed this change after you already pulled the request.

@dhozac indicated that it would probably be better to use return codes > 255 for anything related to Ansible itself. Which makes sens :)
12 years ago
Seth Vidal 0c65102c42 fix up remove and fix fixme checking to see if pkg was removed 12 years ago
Nigel Metheringham 0fc939d33f Turn off some of the additional debug syslogging 12 years ago
Nigel Metheringham 346f8ffe0e Rework of user module to support portability
Split module into a main calling function, and a generic
(Linux useradd/usermod/userdel) User class.
Added a __new__ function that selects most appropriate superclass
Added a FreeBSD User class
Tested against FreeBSD 9.0
12 years ago
Seth Vidal 9dd9db6296 fix removal case where first pkg in a list is not installed at all
and the next one gets ignored despite being installed.

also clean up unnecessary second check.
12 years ago
Maxim Burgerhout fd9c0ca341 More useful error message for if not HAVE_SELINUX 12 years ago
Jeroen Hoekx e2419cbf9d Add a group_by action plugin. 12 years ago
Dag Wieers 2c4d559523 Make sure we always return 'rc' from the command module
If this is not a certainty, playbooks will fail without an 'rc' and checking both if there is an rc, and whether the 'rc' is (not) 0 is very complicated. (especially because ${something.rc} will not be substituted and all that)
12 years ago
Daniel Hokka Zakrisson 447ea1624a When atomically replacing files, copy attributes 12 years ago
Michael DeHaan d55e7f35cd Merge pull request #1455 from bleader/devel
user module: fix typo in documentation
12 years ago
Michael DeHaan 68bf6b1343 Merge pull request #1453 from sfromm/issue1211
Use shutil.copyfile so as to preserve file ownership in authorized_keys
12 years ago