Michael DeHaan
8da853ec7a
Slight typo fix.
2014-07-15 18:08:11 -04:00
Henry Finucane
49889bd808
Make 'compress' optional
2014-07-12 10:13:04 -07:00
Andreas Fleig
30ad859fe0
ini_file: replace lambda with named function
...
Fixes #6876
2014-07-08 10:17:03 -05:00
James Cammarata
9564818e1e
Merge branch 'fix_lineinfile_newlines' into devel
2014-06-23 12:38:12 -05:00
James Cammarata
48dbb79fbe
Merge branch 'issue_5679_lineinfile' into fix_lineinfile_newlines
2014-06-20 00:19:35 -05:00
Norman J. Harman Jr
3770813cc0
lineinfile supports symlinks.
2014-06-19 17:38:34 -05:00
trbs
d5fa690aaf
In check mode if a symlink does not exist there is no need to check the ownership properties (which causes an exception)
2014-06-19 15:47:46 +02:00
James Cammarata
aac194e639
Double check whether the parent directory really exists using stat()
...
Fixes #7760
2014-06-18 14:54:44 -05:00
James Cammarata
5c27273ed4
Merge branch 'unarchive-fix-7777-simple' of https://github.com/toddmowen/ansible into toddmowen-unarchive-fix-7777-simple
2014-06-18 13:48:57 -05:00
James Cammarata
cb13b30362
Use file documentation fragement for the copy module
2014-06-16 15:36:15 -05:00
Todd Owen
43c9edad8a
unarchive: do not assume tar supports -C
...
Fixes #7777
(But don't rewrite the tar invocation in is_unarchived(), since a
tar that supports "--diff" certainly supports "-C" as well).
2014-06-15 22:38:41 +08:00
Todd Owen
a4fe91050b
unarchive: improve failure detection and reporting
...
Related to #7777
2014-06-15 13:13:18 +08:00
James Cammarata
d90f0bdfad
Merge branch 'fix-lineinfile-eof-newline' of https://github.com/ahtik/ansible into ahtik-fix-lineinfile-eof-newline
2014-06-12 09:13:49 -05:00
Aleksey Khudyakov
6865ffe93c
Escape backslash in assemble module docs
2014-06-11 04:38:57 +11:00
James Cammarata
8e3b512293
Merge pull request #7377 from mikhailguru/devel
...
Fixed handling of paths containing whitespaces in the synchronize module
2014-06-10 06:50:07 -05:00
Ahti Kitsik
1caf9be109
Bugfix to lineinfile to properly handle files with no newline at EOF when using insertafter=EOF.
...
Looks like #6881 is also affected by this bug.
Currently when using lineinfile insertafter=EOF for a file that has no newline then the added line is not prefixed with a newline.
As a concrete example, ubuntu distro default /etc/ssh/sshd_config has no newline so lineinfile module cannot be used to add a new line to the end of the file.
2014-06-09 19:58:45 +03:00
James Cammarata
43f96a636b
Fix issue with symlink path detection
...
Fixes #7627
Fixes #7664
2014-06-04 23:31:59 -05:00
James Cammarata
7483acde5d
Fix linking issue to directories in file module
...
The os.path.isdir() returns True for symlinks that point to directories,
so we also need to check to make sure it's not a symlink too.
Fixes #7657
2014-06-04 22:48:08 -05:00
Brian Coca
202ac68995
Correctly examine path for relative links in file module
...
Fixes #7515
Fixes #7537
2014-05-28 11:47:44 -05:00
Kit Westneat
fabe8521cb
Fixed documentation for etype parameter on acl module.
...
The etype parameter just had a copy of the default parameter documentation.
Changed it to reflect what it really is for.
2014-05-27 13:34:01 -04:00
Brian Coca
c24956032d
unified calls to fs function vs duplicate functions for file and
...
directory
2014-05-24 09:28:45 -04:00
James Cammarata
062e82937f
Fixing absolute path calculation for file module when linking
...
Fixes #7403
2014-05-21 12:49:42 -05:00
telsacolton
06963c7826
Make file hardlink error message reasonble
...
When we hard-link a file, and the link path already exists, give the link path in the error message instead of the source
2014-05-14 16:54:26 -05:00
telsacolton
0d8104b8b7
Give reasonable error when symlink fails
...
When a symlink fails because the path already exists and force=no, we should output path in the error message instead of source.
2014-05-14 16:31:20 -05:00
James Cammarata
a9311a5dcb
Merge branch 'file_new_dirs_perms' of https://github.com/bcoca/ansible into bcoca-file_new_dirs_perms
2014-05-14 13:53:08 -05:00
Mikhail Emelchenkov
a2ca0441ae
Fixed incorrect handling of paths contaiin whitespaces
2014-05-13 17:05:44 +04:00
James Cammarata
a18f4b6230
Merge branch 'devel' of https://github.com/jjshoe/ansible into jjshoe-devel
2014-05-09 00:31:39 -05:00
James Cammarata
d9bb1a480a
Merge pull request #7013 from orgoj/replace-patch-changed
...
module replace - report changed only if contents is changed
2014-05-08 15:14:19 -05:00
James Cammarata
b753625dbf
Refuse to convert a non-empty directory into a link with the file module
...
Also adds an integration test for the above.
Fixes #7254
2014-05-02 14:46:38 -05:00
someda
50080b3774
fix #7244 recursive copy fails when setting directory attributes
...
- reverse the order of new_directory_list returned from split_pre_existing_dir in copy module
2014-05-02 01:38:16 +09:00
Michael DeHaan
6b26efdee0
Merge pull request #7119 from jjshoe/patch-4
...
Finish reverting a broken commit in #5679
2014-04-29 16:47:46 -04:00
James Cammarata
ac1948c04c
Minor tweak to rsync_path construction in synchronize module
...
Fixes #7182
2014-04-29 15:47:16 -05:00
Skyler Leigh Amador
a8d9470938
Remove --timeout option to rsync from synchronize defaults
...
The current 10-second default timeout for rsync seems to be behind issue #6809 .
As a workaround for the underlying issue in rsync when different versions are
used on the source and destination sides, don't include the timeout option
unless the user specifies a timeout > 0.
2014-04-27 20:21:54 -07:00
Joel Schuweiler
5bad971873
Finish reverting #b73cddc
2014-04-27 09:36:13 -05:00
jjshoe
a34478d6f3
Ensure the validation string has %s in it
2014-04-27 09:30:36 -05:00
Brian Coca
f079d022a3
now when making multiple dirs in path to make dir, same permissions are assigned (but only for NEW dirs)
...
Signed-off-by: Brian Coca <briancoca+ansible@gmail.com>
2014-04-23 10:38:38 -04:00
Erika Heidi
036c7a7759
Possible docs error: 'path' instead of 'src'
...
I came to this example today (line 50), and when I tried it, I got an error:
`msg: src and dest are required for creating links`
So, I changed **path** to **src** and it worked fine. Looks like the example is wrong, since the docs say "dest" is already an alias for "path", in which case we would have 2 "path" definitions.
2014-04-23 16:00:22 +02:00
Michael Heca
a1706e6411
module replace - report changed only if contents is changed
2014-04-16 09:25:56 +02:00
James Cammarata
e8b33636ae
Get the bin path for commands in the unarchive module
...
Fixes #6927
2014-04-15 20:39:22 -05:00
James Tanner
b0556ab1fb
Addresses #6702 Remove version_added from description string
2014-04-11 11:18:05 -04:00
James Tanner
42fbbd0502
Addresses #6702 set version_added for rsync_opts
2014-04-11 11:04:05 -04:00
jctanner
bb3ce0c744
Merge pull request #6702 from jjshoe/devel
...
Allow you to pass in arbitrary rsync options
2014-04-11 10:43:14 -04:00
James Tanner
dfdc3b82a0
Fixes #6929 remote file module doc fragments from copy and template
2014-04-10 16:06:35 -04:00
James Tanner
68cd7258b6
Fixes #6894 create docs fragment for files
2014-04-08 13:20:15 -04:00
James Cammarata
567f3819b1
Fixing newline escapes in lineinfile
...
Fixes #5679
2014-04-08 11:42:29 -05:00
James Cammarata
b73cddc485
Revert "Fix #5679 again after recent changes in core"
...
This reverts commit 2bfaacd170
.
Fixes #6821
2014-04-07 09:55:59 -05:00
Richard C Isaacson
dc93b31d22
assemble module: fix insertion of newlines when not needed.
...
This builds on GH-6359 and changes the logic so that a newline is only inserted between fragments if the previous fragment does not end with a newline.
2014-04-03 16:32:35 -05:00
Peter Gehres
3ac731087c
Fix for #6353 adding a newline between assembled files
2014-04-03 16:32:35 -05:00
Michael DeHaan
1fa19e29e8
Use common file argument system previously implemented by @sivel for rax modules on the file modules as well (copy/file/template).
...
Application to other cloud providers would make very good sense.
2014-04-03 16:52:39 -04:00
James Tanner
9fe2bcd076
Merge #5903 from rhaido/ansible
...
Update files documentation for usage of force=True and hardlinks
2014-03-31 12:53:03 -04:00