Commit graph

4756 commits

Author SHA1 Message Date
Michael DeHaan
817b0cdc80 Instantiate filter plugins only once to save some efficiency. 2013-04-20 09:51:20 -04:00
Michael DeHaan
bac3449db7 Restructure hacking readme. 2013-04-20 09:36:09 -04:00
Michael DeHaan
e2d9e1b5c2 Convert 'hacking' instructions to markdown, add more info. 2013-04-20 09:34:38 -04:00
Michael DeHaan
cfe8af493f Since using platform.node, be sure to just return the hostname component. 2013-04-20 09:34:37 -04:00
Michael DeHaan
6e1514e975 Merge pull request #2727 from dsedivec/devel
Don't send unicode instances to the selinux module
2013-04-20 06:21:27 -07:00
Michael DeHaan
5b44c3d52c Merge commit '53ac0bb' into devel 2013-04-20 09:13:10 -04:00
Michael DeHaan
87788b1d17 Ignore inventory config files when using an inventory directory. 2013-04-20 09:10:19 -04:00
Michael DeHaan
58d72b28c5 Fixup authorized key documentation formatting 2013-04-20 09:10:19 -04:00
Michael DeHaan
adac8f5312 Showcase the new-style lookup plugin access in the authorized_key docs rather than the old-style $FILE 2013-04-20 09:10:19 -04:00
Michael DeHaan
53ac0bbec2 Instantiate callback plugins only once so we can set play/task objects on them and they'll stick. 2013-04-20 09:09:35 -04:00
Dale Sedivec
c8f4a56cad Don't send unicode instances to the selinux module
This fixes #2632.  Briefly: specifying things like paths using complex
args in a playbook will make the objects unicode instances.  The selinux
module does not accept unicode instances for its char * arguments; it
wants str instances.

Per mpdehaan's comment on #2632 I just went ahead and converted all
paths to UTF-8.  I don't know if it would be better to do something like
converting to locale.getpreferredencoding(), but I factored all the
conversions out into new method _to_filesystem_str, so there's only one
place that needs to be changed in the future.
2013-04-19 20:12:11 -05:00
Michael DeHaan
df93d7dd97 Ignore inventory config files when using an inventory directory. 2013-04-19 19:01:19 -04:00
Michael DeHaan
66afe13346 Merge pull request #2725 from jtsoi/patch-1
Typo in example, extra space breaks playbook.
2013-04-19 14:54:16 -07:00
jtsoi
8c11624f14 Typo in example, extra space breaks playbook. 2013-04-19 22:43:50 +03:00
Michael DeHaan
8e7dc3c79c Remove an extra space in the module execution line if no environment was set. 2013-04-19 08:11:56 -04:00
Michael DeHaan
5656817776 Fixup authorized key documentation formatting 2013-04-18 22:43:14 -04:00
Michael DeHaan
cd0602889d Showcase the new-style lookup plugin access in the authorized_key docs rather than the old-style $FILE 2013-04-18 22:38:48 -04:00
Michael DeHaan
0f86b26823 Merge pull request #2720 from base10/devel
Small spelling correction to authorized_key module docs
2013-04-18 19:34:10 -07:00
Nathan L. Walls
e8f46822e5 Small spelling correction to authorized_key module docs 2013-04-18 22:16:28 -04:00
Michael DeHaan
c35ef43517 Merge pull request #2716 from skvidal/first_found_skip_fix
clean up first_found to fix a few issues:
2013-04-18 13:51:48 -07:00
Seth Vidal
7b8cec3f59 clean up first_found to fix a few issues:
- add a skip option so it won't raise an exception if you don't match anything
 - make it work as a drop-in replacement for first_available_file
 - document in the module comments all of the above cases
2013-04-18 16:47:10 -04:00
Michael DeHaan
5f1e2afc34 Revert "make atomic_replace use shutil.copy2 instead of os.rename() so it will work across filesystems".
AR function was leaving some tmp files behind, want to revert, will have better implementation soon, this is the old way now.

This reverts commit f74a1fa4f0.
2013-04-18 15:46:42 -04:00
Michael DeHaan
3238d0947d Merge pull request #2712 from lwade/devel
correct reference to module
2013-04-18 05:29:13 -07:00
lwade
a154a5ddd4 correct reference to module 2013-04-18 10:36:07 +01:00
Michael DeHaan
d8bf87b008 (A) include errors in inventory scripts should they occur.
(B) allow registration with ignore_errors: True
2013-04-17 22:27:00 -04:00
Michael DeHaan
6544af9616 Merge pull request #2705 from b6d/utils-fix
Fix if-statement that always evaluates True
2013-04-17 11:13:58 -07:00
Bernhard Weitzhofer
fb0a99e5d8 Fix if-statement that always evaluates True 2013-04-17 19:40:06 +02:00
Michael DeHaan
a02202eff7 Merge pull request #2701 from chamill/devel
Fix 'sequence' plugin error.
2013-04-17 08:48:15 -07:00
C. Morgan Hamill
88eb5516eb Fix 'sequence' plugin error.
If 'terms' is a string, replace it with a single item list.
2013-04-17 11:45:28 -04:00
Michael DeHaan
40b78f96de Merge pull request #2700 from chamill/devel
Fix errors in lookup plugins.
2013-04-17 08:41:14 -07:00
C. Morgan Hamill
663d37b537 Fix errors in lookup plugins.
Lookup plugins 'sequence' and 'template' now import 'ansible.utils'
appropriately in order to use the 'listify_lookup_plugin_terms'
function.

Also, 'dnstxt' and 'env' now check to see if 'terms' is a string;
without this calls like '{{ lookup('env', 'HOME') }}' fail.
2013-04-17 11:31:38 -04:00
Michael DeHaan
cce2cad8dc Fix with_sequence doc error 2013-04-16 20:42:06 -04:00
Michael DeHaan
500b9432a2 Merge pull request #2596 from blair/ec2_facts-provides-ec2-region
ec2_facts: add ansible_ec2_placement_region key/value pair for EC2 regio...
2013-04-16 17:28:13 -07:00
Dale Sedivec
515fd9e915 copy action plug-in check mode respects force=no
The copy action accepts force=no, which tells it not to replace an
existing file even if it differs from the source.  The copy action
plug-in wasn't respecting this option when operated in check mode, so it
would report that changes are necessary in check mode even though copy
would make no changes when run normally.

Runner._remote_md5 was changed to make the logic for setting rc perhaps
a little more clear, and to make sure that rc=0 when the file does not
exist.
2013-04-16 20:22:51 -04:00
Michael DeHaan
6cb6223deb Merge pull request #2608 from lwade/ec2zone
Add availability zone parameter
2013-04-16 17:20:55 -07:00
Michael DeHaan
22e3c389b3 Use platform.node() for the hostname, leave fqdn calculated the same way 2013-04-16 20:10:21 -04:00
Michael DeHaan
6a00513731 Add version added to module docs 2013-04-16 20:07:18 -04:00
Michael DeHaan
7c6341718e Merge branch 'combine_vars' of git://github.com/laggyluke/ansible into exp
Conflicts:
	lib/ansible/inventory/vars_plugins/group_vars.py
	lib/ansible/runner/__init__.py
	lib/ansible/utils/__init__.py
	test/TestPlayBook.py
2013-04-16 20:06:06 -04:00
C. Morgan Hamill
863cb50530 Allow '.yml'/'.yaml' extension on group_vars files.
Look for a file with the base name of the group/host, first without
a file extension, then with a '.yml' extension, and, finally, with
a '.yaml' extension, loading vars from only the first one found.
2013-04-16 19:59:23 -04:00
Michael DeHaan
454076590c Merge pull request #2580 from blair/ec2_vol-improvements
Ec2 vol improvements
2013-04-16 16:58:54 -07:00
Michael DeHaan
6d66b0e965 Merge pull request #2692 from j2sol/rax
Minor fixes to the rax module
2013-04-16 16:52:43 -07:00
Michael DeHaan
b54e955595 Remove a debug statement. 2013-04-16 19:09:12 -04:00
Michael DeHaan
86d47bce5f Make more lookup plugins tolerant of new variable system, with a little better 'do what I mean' logic to resolving
what happens if you get a string back as a template result.
2013-04-16 19:07:19 -04:00
Michael DeHaan
c0f8af5202 Make more lookup plugins happy with newstyle variables. Not quite done, but this fixes up with_items/with_nested/file/fileglob. 2013-04-16 18:50:30 -04:00
Michael DeHaan
f72649d0f9 Merge pull request #2698 from skvidal/atomic_replace_and_cron
Atomic replace and cron
2013-04-16 12:53:41 -07:00
Seth Vidal
f74a1fa4f0 make atomic_replace use shutil.copy2 instead of os.rename() so it will
work across filesystems
2013-04-16 15:15:58 -04:00
Seth Vidal
5df2dadcdb clean up how it puts the files in place - in f18 /tmp is tmpfs which
means ln and os.rename() won't work across fs.
2013-04-16 15:08:27 -04:00
Jesse Keating
59d0fb6c95 Only accept supported services for rax module
Even though others are possible, fail early on unsupported ones.
2013-04-15 17:22:42 -07:00
Jesse Keating
55ef1d6e47 Update documentation for rax module
Only list the services supported
Don't put in unncessary required: false
Use better formatting for the example
2013-04-15 17:20:45 -07:00
Michael DeHaan
84fb92dac6 Working on the FAQ. 2013-04-14 19:32:06 -04:00