Commit Graph

902 Commits (a19a13ffa7f2bb44f123dcecad3c880ac729b85a)

Author SHA1 Message Date
Seth Vidal 7e0cb4f052 add ec2 module 12 years ago
Dag Wieers 2fdb217be3 Documentation of svr4pkg module breaks 'make rpm'
Without this fix, generating documentation results in:

```
Traceback (most recent call last):
  File "hacking/module_formatter.py", line 376, in <module>
    main()
  File "hacking/module_formatter.py", line 365, in main
    text = template.render(doc)
  File "/usr/lib64/python2.6/site-packages/jinja2/environment.py", line 669, in render
    return self.environment.handle_exception(exc_info, True)
  File "hacking/templates/man.j2", line 20, in top-level template code
    {% for desc in v.description %}@{ desc | jpfunc }@{% endfor %}
  File "hacking/module_formatter.py", line 94, in man_ify
    t = _ITALIC.sub(r'\\fI' + r"\1" + r"\\fR", text)
TypeError: expected string or buffer
```
12 years ago
Michael DeHaan 03591e0fc8 Merge pull request #1551 from dagwieers/local_action-fail
Turn 'fail' and 'debug' modules into action_plugins
12 years ago
Michael DeHaan f3cbcecf53 Merge pull request #1563 from lllama/devel
Add an "executable" option to the command and shell modules
12 years ago
Michael DeHaan 03f4b0f923 Merge pull request #1560 from dagwieers/module-output-consistency
Make module output more consistent wrt. changed/failed
12 years ago
Michael DeHaan cbff02139f Merge pull request #1557 from sfromm/issue1412
Updates to git module: use git-fetch and other changes
12 years ago
Michael DeHaan 42375d14b3 Merge pull request #1556 from sfromm/issue1099
Wrap getpwnam in try/except in authorized_key module
12 years ago
Boyd Adamson 528a457848 Add svr4pkg module for traditional Solaris packages 12 years ago
Felix Ingram e0f0e8c023 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 fe0c70fe9d 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 3ec64d19a8 Wrap getpwnam in try/except in authorized_key module
This also adds module as an argument to keyfile().
12 years ago
Stephen Fromm 6aa51a7cf3 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 d8459425a0 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 612aed55bb Merge pull request #1546 from dagwieers/get_url-nonthirsty-fix
Don't require thirsty mode for directory destination
12 years ago
Stephen Fromm 49d41da152 Use supplied remote name when cloning git repository 12 years ago
Stephen Fromm 328951ef53 Checkout branch before rebase in switch_version 12 years ago
Stephen Fromm ccc04cf891 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 5ea260f44b 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 82ec405b05 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 29882f97e6 Fixed the way dom0/domU xen role is defined in the setup module 12 years ago
Stephen Fromm 2fab846f17 Fix to user module to use self when calling group_exists 12 years ago
Daniel Hokka Zakrisson cf290a0064 Keep fireball from spreading its socket around 12 years ago
Nigel Metheringham f905e751ac FreeBSD user mod only fires when there are changes
Also fix missing -G on groups change
12 years ago
Nigel Metheringham 1ab1c8d374 Take SHADOWFILE from existing user object rather than class 12 years ago
Nigel Metheringham 75a2b2ba2d Use the existing framework when running ssh_keygen 12 years ago
Nigel Metheringham 19402a47cd Ensure user owns their ssh key directory on creation
Lost section from commit 8049777
12 years ago
Michael DeHaan 6f938c9f4c 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 d639844923 Merge branch 'service_bsd' of git://github.com/bcoca/ansible into devel 12 years ago
Michael DeHaan 970d06e03b Fix user example 12 years ago
Michael DeHaan 626203a7c9 Module documentation should have options sorted 12 years ago
Michael DeHaan c96f2c968e 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 cdfa81d905 Added some documentation to the classes 12 years ago
Daniel Hokka Zakrisson 20775c0b73 Fix service module for varnish initscript in EPEL
It outputs 'dead but pid file exists'.
12 years ago
Brian Coca 13f75e5a8e 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 9268aa0289 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 c0747b7baa pep8 fixes 12 years ago
Michael DeHaan 27e518a0b1 Rename 'domainname' to 'domain' 12 years ago
Michael DeHaan 9251709941 Merge pull request #1490 from sfromm/issue1484
Make sure ownership is set correctly for user's new ssh keys
12 years ago
Michael DeHaan 768d293949 Merge pull request #1488 from dagwieers/lineinfile-create
Add option create= to lineinfile module
12 years ago
Michael DeHaan 91cf257e9c 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 0853ece810 Merge pull request #1474 from jhoekx/group-by-plugin-template
Add a group_by action plugin.
12 years ago
Serge van Ginderachter 1db563e064 Add domainname to ansible facts 12 years ago
Serge van Ginderachter 47d882f374 Fix typo 12 years ago
Dag Wieers 1e3dcfce4b 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 4619e99443 Make sure ownership is set correctly for user's new ssh keys 12 years ago
Michael DeHaan f0000a89b0 Merge pull request #1480 from dagwieers/command-fix2
Use return codes > 255 for Ansible-specific errors.
12 years ago
Michael DeHaan 730031ec1f Merge pull request #1479 from wzzrd/selinux
More useful error message for if not HAVE_SELINUX
12 years ago
Dag Wieers 8d4f9febd8 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 49dce05cdd fix up remove and fix fixme checking to see if pkg was removed 12 years ago
Nigel Metheringham 9a2545957f Turn off some of the additional debug syslogging 12 years ago
Nigel Metheringham 3e0e572cce 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 7c1e0de141 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 82cc382c6a More useful error message for if not HAVE_SELINUX 12 years ago
Jeroen Hoekx b336d40101 Add a group_by action plugin. 12 years ago
Dag Wieers 63fa33b0b7 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 7a8009f979 When atomically replacing files, copy attributes 12 years ago
Michael DeHaan 523f2bf327 Merge pull request #1455 from bleader/devel
user module: fix typo in documentation
12 years ago
Michael DeHaan cbcc133cdd Merge pull request #1453 from sfromm/issue1211
Use shutil.copyfile so as to preserve file ownership in authorized_keys
12 years ago
Michael DeHaan 5b36e374d7 Merge pull request #1451 from dhozac/command-strip-newlines
Only strip trailing newlines from output, preserving other space
12 years ago
Michael DeHaan 06fa3e820d Merge pull request #1448 from jpmens/user_user1
User is alias for name in 'user' module
12 years ago
bleader 5a5751d1d0 user module: fix typo in documentation
tyep -> type

Signed-off-by: bleader <bleader@ratonland.org>
12 years ago
Stephen Fromm a033ad4d74 Use shutil.copyfile so as to preserve file ownership in authorized_keys 12 years ago
Daniel Hokka Zakrisson 73cd3ef617 Only strip trailing newlines from output, preserving other space 12 years ago
Jan-Piet Mens 89f28a98dd User is alias for name in 'user' module 12 years ago
Michael DeHaan 895715b65b Merge pull request #1445 from sfromm/issue1439
Skip check of primary group when checking other group membership
12 years ago
Stephen Fromm 544dc76704 Create temp file in directory where ssh key lives 12 years ago
Stephen Fromm b2e94e53fc Write keys file to temp location to avoid potential corruption 12 years ago
Stephen Fromm e97916cfc1 Skip check of primary group when checking other group membership 12 years ago
Michael DeHaan f2a7ca9a36 Merge pull request #1435 from stijnopheide/mysql-grant
Mysql grant, take 2
12 years ago
Michael DeHaan 3e491513a9 Merge pull request #1432 from skvidal/devel
yum enablerepo/disablerepo option complete
12 years ago
Dag Wieers f4c4537327 Resolve traceback when python-selinux is not installed 12 years ago
Seth Vidal 0a153c6798 add comments that enablerepo/disablerepo are new in 0.9 12 years ago
Stijn Opheide 733693ffdc - removed space escapes from MySQL SHOW GRANTS regex
- proper checking for with grant option (if this is not the only option the user has)
- added revoking of grant option
12 years ago
Stijn Opheide 6b92b69fe6 added the GRANT privilege for a mysql user (WITH GRANT OPTION) 12 years ago
Jan-Piet Mens 82955606c4 fix DOCUMENTATION for user 12 years ago
Seth Vidal d8337dab06 tested and verified yum enable/disablerepo option for yum module 12 years ago
Seth Vidal d89d07550f first cut at yum module with enable/disable repo 12 years ago
Michael DeHaan 9626111d5b Merge pull request #1427 from bleader/devel
mount: update example with mandatory option
12 years ago
Stephen Fromm 5d8c130a7f Document ssh_key options are in version 0.9 12 years ago
bleader e16e3fb4ca mount: update example with mandatory option
- 'state' is required, but wasn't in example

Signed-off-by: bleader <bleader@ratonland.org>
12 years ago
Jan-Piet Mens feab57e270 Reformat code: examples consistently
- added terse syntax to modules.rst
 - added description of special variables to template module
12 years ago
Stephen Fromm 769bd9125a Help git module work when working in 'no branch' scenario
Detect when on a 'no branch' branch.  If so, checkout the HEAD branch
as reported by 'git remote show <remote>'.  That should put the repo
back on a branch such that git can then merge changes as necessary.
In addition, removed hard-coded references to origin and replaced
with remote var.
12 years ago
Michael DeHaan cbc12f0dba Various performance streamlining and making the file features usable in all modules without daisy chaining. 12 years ago
Michael DeHaan 4104478abd Added 'script', a module that can push and insantly run a remote script. Replaces the need to copy+command.
This is a bit of a rough draft, some error handling still needs to be added.
12 years ago
Michael DeHaan 6b8448051f Merge pull request #1389 from sfromm/issue1214
Add ability to create SSH key for user in user module
12 years ago
Michael DeHaan 3c739b1a88 Merge pull request #1339 from inertialbit/add-basic-pg-role-attribute-support
add role_attr_flags parameter to postgresql_user
12 years ago
Stephen Fromm 87954f2110 Use os.path.join where appropriate 12 years ago
Stephen Fromm 63ba441163 Add ability to create SSH key for user in user module
This allows one to create a SSH key for user.  You may define:
ssh_key_type, ssh_key_bits, ssh_key_file, ssh_key_comment,
and ssh_key_passphrase.  If no passphrase is provided, the
key will be passphrase-less.  This will not overwrite an existing key.
In the JSON returned, it will provide the ssh_fingerprint and
ssh_key_file.
12 years ago
Michael DeHaan 68cf9a96fc Merge pull request #1388 from goozbach/selinux_bugs
make policy only be required if state is not disabled
12 years ago
Derek Carter 99180e7da3 make policy only be required if state is not disabled 12 years ago
Seth Vidal 326b160241 when a pkg is installed, but not from any repo, don't error out
that it is not installed.

Also when a pkg is both installed and in a repo do not look it up
more than once
12 years ago
Michael DeHaan d58bc4da4e Merge pull request #1361 from lantins/devel
Copy module can replace symlink with a real file.
12 years ago
Michael DeHaan 05866bba1e Merge pull request #1372 from jpmens/rst_template1
module_formatter: .rst now correctly handles code examples
12 years ago
Jonathan Palley 702469f7bf symlink fix 12 years ago
Jan-Piet Mens 575459465e module_formatter: .rst now correctly handles code examples
- fixed template (it was the template), adding indentation with Jinja2
 - added description of code examples to man-page template (was missing)
 - fixed fireball, cron, and debug module examples to confrom
12 years ago
Luke Antins 0dbd3aa821 Add examples to user module documentation. 12 years ago
Luke Antins 4f17aa9fa0 Fix indent of fail module example to prevent unwanted `<blockquote></pre></p> <br/></blockquote>` from showing up in docsite output. 12 years ago
Luke Antins 7d41045b2f Erroneous space in facter documentation. 12 years ago
Luke Antins 6354898b09 Copy module can replace symlink with a real file. 12 years ago
Michael DeHaan 54257a6a4c Remove "the the" 12 years ago
Michael DeHaan edb7a054ae Merge pull request #1349 from bcoca/fix_bsd_dmesg_jails
fixed but in pulling data from dmesg command, wrong function to use as f...
12 years ago
Brian Coca 620aeeb572 fixed but in pulling data from dmesg command, wrong function to use as file for for loop
Signed-off-by: Brian Coca <briancoca+ansible@gmail.com>
12 years ago
Michael DeHaan 5bb8dcea08 Fix help text 12 years ago
Michael DeHaan 6f14e35c36 Merge pull request #1335 from phgrau/devel
Use && for executing next command if, and only if, command1 returns an e...
12 years ago
Michael DeHaan 7583704144 "force=" code under the file module is problematic, remove it 12 years ago
Jan-Piet Mens e35d5188f9 typo 12 years ago
Jeremiah Heller 37bdefae74 add role_attr_flags parameter to postgresql_user
Pass role_attr_flags a list of comma separated role
attributes when creating or updating a user.
12 years ago
Philipp Grau 4290a904ff Add note: Debian Squeeze has no add-apt-repository 12 years ago
Maxim Burgerhout 2f6b9e69a7 Test for python-pip as well as pip
On Red Hat, CentOS and Fedora systems, the pip binary will be called python-pip
instead of pip. This commit makes the pip module also check for python-pip.

The reason we check for python-pip *first*, is to have ansible fail on not
finding 'pip' and reporting *that*. This is consistent with current behaviour
and will not confuse users of Debian et al., where the 'python-pip' binary
never exists.

Tested on Fedora 18 and Ubuntu 12.04.
12 years ago
Tim Bielawa 8831e59700 The pause 'prompt' parameter now requires quotes around the message. 12 years ago
Michael DeHaan 8dadf8a905 Use 'path' in file documentation, though src/dest work, it's less intuitive. 12 years ago
Michael DeHaan 2c90298841 Merge pull request #1314 from apenney/bugfix/git
Check rc instead of parsing for errors.
12 years ago
Michael DeHaan c5d2f6b0d3 implement lookup plugins for arbitrary enumeration over arbitrary things. See the mailing list for some cool examples. 12 years ago
Michael DeHaan 29d49d415f fix mentioning of redundant aliases in module docs, remove was removed anyway 12 years ago
Michael DeHaan 621708e976 user module should be using module.BOOLEANS for choices 12 years ago
Michael DeHaan aa98c84a62 HP and vsphere modules are crazy awesome but I am thinking these belong in contrib not core. 12 years ago
Brian Coca 6886683e16 small patch to fact gathering for when no dmesg.boot exists on freebsd, mainly happens in jails
Signed-off-by: Brian Coca <briancoca+ansible@gmail.com>
12 years ago
Michael DeHaan 35163bce96 Merge pull request #1310 from dagwieers/hpilo_boot-cleanup
hpilo_boot: Various clean ups in documentation and code
12 years ago
Michael DeHaan b1ccda00f8 Merge pull request #1311 from dagwieers/hpilo_facts-cleanup
hpilo_facts: Various clean ups in documentation and code
12 years ago
Michael DeHaan 1487b6a1bb Merge pull request #1313 from dagwieers/vsphere_facts-cleanup
vsphere_facts: Various cleanups in documentation
12 years ago
Michael DeHaan a82522a5c4 Merge pull request #1317 from abondis/fix_mysql_db
fix issue with mysql authentication and states not dump or import
12 years ago
Michael DeHaan 20a7227056 Merge pull request #1318 from dsummersl/cron24
fixed cron library so backups are python2.4 compatible
12 years ago
Jeroen Hoekx 25e415c031 Add virt_boot module to define libvirt boot parameters. 12 years ago
Dane Summers 7399224428 fixed cron library so backups are python2.4 compatible 12 years ago
Aurélien Bondis 1f91a70310 fix issue with mysql authentication and states not dump or import 12 years ago
Ashley Penney f4484f800b Further cleanup to add another rc check and remove a now redundent one. 12 years ago
Ashley Penney 6042419624 Check rc instead of parsing for errors. This now catches the case where the remote branch has been deleted and you're still trying to pull against it. 12 years ago
Dag Wieers 5927373c26 Various clean ups in documentation and code
This commit improves the following items:
 - Remove the 'match' functionality, this can now be achieve by using the `fail` module together with `only_if` after running the `hpilo_facts` module. Since this gives more functionality, e.g. comparing server names, but also serial numbers or uuids with other inventory information, this is prefered. An example is added to show how this is achieved.
 - Clean up all C() calls in documentation
 - Added state=poweroff in order to power off a server. The use-case is here that in general we do not want to provision systems that are already running (this enforcement can be disabled using force=yes) but for test systems we should be able to power them off so we can start the normal provisioning process. (We could also force boot them, but that's less elegant)
 - The module now correctly indicates when something has changed. So if a server is powered off that was not off already, this is indicated, or when media boot-settings have been changed, this is also correctly indicated. Previously every call to hpilo_boot was (incorrectly) considered a change.
12 years ago
Dag Wieers 5fe8205d36 vsphere_facts: Various cleanups in documentation
This commit improves the following items:
 - Clean up all C() calls in documentation
 - Reindent the documentation
12 years ago
Dag Wieers 5612cce4a4 hpilo_facts: Various clean ups in documentation and code
This commit improves the following items:
 - Remove the 'match' functionality, this can now be achieve by using the `fail` module together with `only_if` after running the `hpilo_facts` module. Since this gives more functionality, e.g. comparing server names, but also serial numbers or uuids with other inventory information **and** a proper message, this is prefered. An example is added to show how this is achieved.
 - Clean up all C() calls in documentation
 - Remove trailing spaces in HP iLO's Serial Number output so that they can be compared to CMDB or other inventory information
12 years ago
Michael DeHaan dfaef8061a Merge pull request #1289 from abondis/mysql_migration
mysql_db: connection test, dump/import state
12 years ago
Dag Wieers dfdf4ab8eb Damn you, colons... 12 years ago
Aurélien Bondis 1a51af0e38 mysql_db: use the database to test connexion, added dump and import state - requires target=path/to/a/file 12 years ago
Michael DeHaan e74b9e1d34 Add .gitdir 12 years ago
Michael DeHaan e593c828b4 Merge remote branch 'dagwieers/module-mail' into devel 12 years ago
Michael DeHaan 7788dea238 Merge pull request #1268 from dagwieers/hpilo-reboot-fix
Workaround for when HP iLO is in a state it won't accept requests
12 years ago
Michael DeHaan f1dfddc5e5 Fix typo in autogenerated docs 12 years ago
Dag Wieers 2f74cd84bd Module mail to send out emails as an action
Sending mails could be part of the workflow to have teams/responsibles perform specific task. Or simply to notify that a process has finished successfully (e.g. provisioning).
12 years ago
Dag Wieers 790ed81b14 Workaround for when HP iLO is in a state it won't accept requests
This workaround is recommended from HP iLO's documentation, but may not be sufficient in all cases. Time will tell.

I also made a few cosmetic changes with no impact.
12 years ago
Michael DeHaan 06688efb24 Merge pull request #1263 from dagwieers/hpilo_facts-fix
Sigh, another retypo
12 years ago
Dag Wieers 98624cb394 Fixes to documentation 12 years ago
Dag Wieers 3f149e09bc Sigh, another retypo problem 12 years ago
Michael DeHaan 612561ad0e Merge pull request #1250 from dagwieers/hpilo_boot-fixes
Various small fixes to boolean usage and defaults
12 years ago
Michael DeHaan 62a56a219b Merge pull request #1243 from dsummersl/squash2-cron
cron feature (w/o tests)
12 years ago
Michael DeHaan 1b36446ab3 Merge pull request #1255 from dagwieers/module-namespace
Add this module's namespace to the 'module_' namespace.
12 years ago
Tim Bielawa 1e35ac7603 Fix typo in service module docs 12 years ago
Dane Summers c3a6e8dfd6 added new documentation string to cron library 12 years ago
Dag Wieers b39aed8736 Add this module's namespace to the 'module_' namespace.
Much like we currently have *setup* register the variable `module_setup`, we would like other facts-modules register their own namespace. This means that:

 - *network_facts* registers `module_network`
 - *hpilo_facts* registers `module_hw`
 - *vsphere_facts* registers `module_hw`

In retrospect, it would have made more sense to have `setup` register `module_ansible` instead as the setup module uses the `ansible_` namesepace.

Having the `module_` namespace allows us to check whether a certain namespace has already been loaded so we can avoid running the facts module a second time using only_if.

```yaml
 - action: network_facts host=${ansible_hostname_short}
   only_if: is_unset('$module_network')
```
12 years ago
Dag Wieers 5503cfaa44 Various small fixes to boolean usage, using module.params.get and default values 12 years ago
Michael DeHaan c8854a23ea Merge pull request #1251 from dagwieers/hpilo_facts-fixes
Fix typo/cosmetic change
12 years ago