Patrik Lundin
5f37624eb4
Tweak error checking for "enabled" code.
...
Based on input from @jarmani:
* A return value of 2 now means a service does not exist. Instead of
trying to handle the different meanings of rc after running "status",
just look at stderr to know if something failed.
* Skip looking at stdout to make the code cleaner. Any errors should
turn up on stderr.
2014-10-06 21:20:20 +02:00
Patrik Lundin
c6dd88c1d1
Fail if "rcctl status" writes to stderr.
2014-10-06 21:20:20 +02:00
Patrik Lundin
1a8cdb5e3e
Check rc and print error message if any.
...
It is probably good to use stdout before printing a generic error
message as well.
2014-10-06 21:20:20 +02:00
Patrik Lundin
c46e030100
Make "enabled" code aware of --check mode.
2014-10-06 21:20:20 +02:00
Jonathan Armani
1b97390765
Add enable / disable of services for OpenBSD if rcctl is present
2014-10-06 21:20:20 +02:00
James Cammarata
cb69744bce
Adding file doc fragment to those modules in files/ missing it
2014-10-01 20:55:50 -05:00
Michael DeHaan
1c2995d71f
Merge pull request #77 from major/suggest-selinux-enforcing
...
Updating SELinux example doc in lininefile module
2014-10-01 11:53:24 -04:00
Michael DeHaan
797dc9d09a
Merge pull request #87 from offby1/devel
...
Tweak docs to mention `replace`
2014-10-01 11:45:18 -04:00
Eric Hanchrow
cc0a38d268
Tweak docs to mention replace
2014-09-30 10:52:03 -07:00
Major Hayden
6d697780f9
Updating SELinux example doc in lininefile module
...
Encouraging users to use this Ansible module to enable SELinux seems
like a better idea. It also warms Dan Walsh's heart.
Signed-off-by: Major Hayden <major@mhtx.net>
2014-09-30 07:12:23 -05:00
Michael DeHaan
db5668b84c
Update README.md
2014-09-26 18:55:32 -04:00
Michael DeHaan
617a52b20d
Fix filename error.
2014-09-26 17:06:31 -04:00
Michael DeHaan
385a037cd6
package files
2014-09-26 11:05:47 -04:00
Michael DeHaan
c8e1a2077e
file extensions!
2014-09-26 10:37:56 -04:00
Michael DeHaan
417309a626
Restructuring.
2014-09-26 10:13:26 -04:00
Michael DeHaan
8cdea3edf0
Core/extras split WIP.
2014-09-26 09:45:31 -04:00
Michael DeHaan
7e94a05dd6
Core/Extras splitting WIP.
2014-09-26 09:40:58 -04:00
Michael DeHaan
dc8540c755
Not core.
2014-09-26 09:32:13 -04:00
Michael DeHaan
1e47a969b8
Fix typo in README filename
2014-09-26 09:26:39 -04:00
Michael DeHaan
73123b69fa
Move modules into subdirectory.
2014-09-26 09:23:50 -04:00
Michael DeHaan
b2b5cc032c
Add basic top level files.
2014-09-26 09:21:20 -04:00
James Cammarata
a099a3c7f5
Merge branch 'admiller/async_resume' of https://github.com/maxamillion/ansible into maxamillion-admiller/async_resume
2014-09-25 15:28:53 -05:00
James Cammarata
1ef592ff00
Push the expanduser call on the source down to slurp rather than fetch
...
Also moves the calculation of the destination file name until after
the slurp of the file contents, since the source as returned by slurp
may now be different, so we want to use that expanded path locally.
Fixes #8942
2014-09-23 15:32:07 -05:00
James Cammarata
61f6a70401
Skip null id route tables when adding/deleting them in ec2_vpc
...
Fixes #8552
2014-09-23 14:51:56 -05:00
Steven Ringo
bcb78335cf
Expand description for ec2_facts module
2014-09-23 08:53:57 +10:00
James Cammarata
dea00c767b
Merge branch 'dmage-devel' into devel
2014-09-22 15:42:38 -05:00
James Cammarata
bb8bac490a
Merge branch 'devel' of https://github.com/dmage/ansible into dmage-devel
2014-09-22 15:40:06 -05:00
Toshio Kuratomi
8b81062be0
The fix for gh-7284 causes problems for initscripts which have the
...
standards compliant return codes but return a verbose error message via
stdout. Limit the times when we invoke the heuristic to attempt to work
around this.
2014-09-22 14:53:07 -04:00
Toshio Kuratomi
724b484577
Merge pull request #9084 from mscherer/fix9704/fix_service
...
Fix #9704 , correctly set changed as false when nothing is done
2014-09-22 11:47:56 -04:00
Michael DeHaan
062cc9bd04
Merge pull request #9082 from mscherer/fix9080/file_not_working_py24
...
Make file module work on python 2.4, fix #9080
2014-09-21 19:18:44 -04:00
Michael Scherer
7420a450a5
Fix indentation and bug #8895
2014-09-21 17:12:29 +02:00
Michael Scherer
c9a0a7ae4b
Fix #9704 , correctly set changed as false when nothing is done
2014-09-21 14:55:31 +02:00
Michael Scherer
d15931f15a
Make file module work on python 2.4, fix #9080
...
Python 2.4 do not support "except ... as ..." construct, so
revert back to the older syntax.
2014-09-21 01:50:58 +02:00
James Cammarata
0d0576a97f
Add note on ec2_lc regarding immutability of configs once created
...
Fixes #8927
2014-09-19 15:17:46 -05:00
Toshio Kuratomi
74e466557d
Merge pull request #9060 from jamespharaoh/apt-lang
...
add LANG=C to apt module so the string matches on the output always matc...
2014-09-19 12:30:15 -07:00
Toshio Kuratomi
7b100dd389
Be more specific in checking if a job restarted successfully.
...
Fixes : #9056
2014-09-19 12:03:27 -07:00
Toshio Kuratomi
36083c3b90
Failure to enable a service now fails the task
...
Fixes: 8855
2014-09-18 12:39:54 -07:00
James Pharaoh
27ccd01db4
add LANG=C to apt module so the string matches on the output always match
2014-09-18 20:56:21 +02:00
James Cammarata
d5546d7a0a
Merge pull request #9052 from EvanDotPro/docs/fix-rax-public-key-example
...
Update rax root pubkey example
2014-09-18 13:25:36 -05:00
Evan Coury
54a979ddd3
Update rax root pubkey example
...
The example was showing how to use the `files` option to pass in a local file as an authorized public key for root. While this works, it's a bit sloppy, given that there's a specific option, `key_name` which will use one of your public keys on your rackspace account and add it as an authorized key for root. In our case, one of our admins didn't notice the `key_name` option because they scrolled straight to the example and saw the `files` strategy.
I propose that the example still shows `files`, but not using a root public key as an example, and instead also demonstrate the `key_name` option so that it's clear from the example how to get the initial root public key deployed.
2014-09-17 18:06:59 -07:00
Sam Pierson
3f8e45f864
[ #9048 ] convert Route53 TTL to integer before comparison
...
boto/Route53 are returning TTL as a string.
The route53 module is comparing it with an integer and therfore thinks the DNS entry has always changed.
2014-09-17 14:47:27 -07:00
James Cammarata
633f86d30a
Allow filtering by availability zone for exact_count with ec2
...
Fixes #8863
2014-09-17 15:49:26 -05:00
Toshio Kuratomi
a50258d26a
remove debugging that was causing failure
2014-09-17 10:25:54 -07:00
Toshio Kuratomi
2e16bd3a44
Refactor the Linux service_enable() method
...
* Fix check_mode for initctl systems
Fixes #9009
2014-09-17 09:34:33 -07:00
Toshio Kuratomi
104401805b
Revert "Refactor the Linux service_enable() method" needs some debugging
...
This reverts commit e294e31fd3
.
2014-09-17 09:17:33 -07:00
Toshio Kuratomi
c362902f50
Refactor the Linux service_enable() method
...
* Fix check_mode for initctl systems
Fixes #9009
2014-09-17 08:44:32 -07:00
James Cammarata
0257cb1201
Allow git repo queries without a clone when update=no
...
This commit also makes the dest parameter optional, unless update=yes
(the default), since it is not required for queries without an update.
Fixes #8630
2014-09-17 02:05:29 -05:00
James Cammarata
2fac1fd865
Ensure port in docker expose list is a string
...
Fixes #8731
2014-09-16 21:38:49 -05:00
Michael DeHaan
40d2932b5a
Merge pull request #9028 from dhoss/devel
...
updated git module documentation
2014-09-16 15:19:44 -04:00
James Cammarata
7b5c76e122
Merge pull request #9026 from tedder/patch-1
...
rearrange description/short_description
2014-09-16 14:04:45 -05:00