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
11 years ago
Brian Coca
202ac68995
Correctly examine path for relative links in file module
...
Fixes #7515
Fixes #7537
11 years ago
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.
11 years ago
Brian Coca
c24956032d
unified calls to fs function vs duplicate functions for file and
...
directory
11 years ago
James Cammarata
062e82937f
Fixing absolute path calculation for file module when linking
...
Fixes #7403
11 years ago
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
11 years ago
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.
11 years ago
James Cammarata
a9311a5dcb
Merge branch 'file_new_dirs_perms' of https://github.com/bcoca/ansible into bcoca-file_new_dirs_perms
11 years ago
Mikhail Emelchenkov
a2ca0441ae
Fixed incorrect handling of paths contaiin whitespaces
11 years ago
James Cammarata
a18f4b6230
Merge branch 'devel' of https://github.com/jjshoe/ansible into jjshoe-devel
11 years ago
James Cammarata
d9bb1a480a
Merge pull request #7013 from orgoj/replace-patch-changed
...
module replace - report changed only if contents is changed
11 years ago
Brian Coca
413a50f87b
reworded copy option docs to be clearer
11 years ago
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
11 years ago
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
11 years ago
Michael DeHaan
6b26efdee0
Merge pull request #7119 from jjshoe/patch-4
...
Finish reverting a broken commit in #5679
11 years ago
James Cammarata
ac1948c04c
Minor tweak to rsync_path construction in synchronize module
...
Fixes #7182
11 years ago
Carlos E. Garcia
7f5dd5e85d
multiple spelling error changes
11 years ago
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.
11 years ago
Joel Schuweiler
5bad971873
Finish reverting #b73cddc
11 years ago
jjshoe
a34478d6f3
Ensure the validation string has %s in it
11 years ago
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>
11 years ago
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.
11 years ago
Michael Heca
a1706e6411
module replace - report changed only if contents is changed
11 years ago
James Cammarata
e8b33636ae
Get the bin path for commands in the unarchive module
...
Fixes #6927
11 years ago
James Tanner
b0556ab1fb
Addresses #6702 Remove version_added from description string
11 years ago
James Tanner
42fbbd0502
Addresses #6702 set version_added for rsync_opts
11 years ago
jctanner
bb3ce0c744
Merge pull request #6702 from jjshoe/devel
...
Allow you to pass in arbitrary rsync options
11 years ago
James Tanner
dfdc3b82a0
Fixes #6929 remote file module doc fragments from copy and template
11 years ago
James Tanner
68cd7258b6
Fixes #6894 create docs fragment for files
11 years ago
James Cammarata
567f3819b1
Fixing newline escapes in lineinfile
...
Fixes #5679
11 years ago
James Cammarata
b73cddc485
Revert "Fix #5679 again after recent changes in core"
...
This reverts commit 2bfaacd170
.
Fixes #6821
11 years ago
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.
11 years ago
Peter Gehres
3ac731087c
Fix for #6353 adding a newline between assembled files
11 years ago
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.
11 years ago
James Tanner
9fe2bcd076
Merge #5903 from rhaido/ansible
...
Update files documentation for usage of force=True and hardlinks
11 years ago
cgtx
f6abc17e8a
backreferences should be double escaped
11 years ago
Michael DeHaan
8baaf458c7
Repairs for documentation build.
11 years ago
Michael DeHaan
c1fbf80f70
Slight docs tweak, split lines according to code standards
11 years ago
Andrea.Mandolo
838b914a0d
correct documentation comment and version_added in "synchronize" module
11 years ago
Andrea.Mandolo
c039dbe300
Add STDOUT lines in response of "synchronize" module
11 years ago
Andrea.Mandolo
f9ec53cdef
Add "checksum" option support to 'synchronize' module
11 years ago
Michael DeHaan
37efa1640f
Merge branch 'update_docs_validate_cmd_not_passed_via_shell' of git://github.com/rwehner/ansible into devel
...
Conflicts:
library/files/template
11 years ago
Michael DeHaan
2ddb251538
Merge pull request #6597 from tarka/handle-uncreated-symlink-tmpfile
...
Handle missing symlink tempfile
11 years ago
jjshoe
d57910b93a
Allow you to pass in arbitrary rsync options. I also added help for it, and what version it was added in.
11 years ago
Jeff Geerling
8960cba4a3
Fixed template validation docs and example.
11 years ago
jjshoe
12c84dd6bd
Show argument dependency
...
failed: [192.168.1.2] => {"cmd": "rsync --delay-updates -FF --compress --timeout=10 --delete-after --rsh 'ssh -i /home/jjshoe/.vagrant.d/insecure_private_key -o StrictHostKeyChecking=no' --rsync-path 'sudo rsync' --out-format='<<CHANGED>>%i %n%L' /tmp/app vagrant@192.168.1.2:/home/ubuntu/app", "failed": true, "item": "", "rc": 1}
msg: rsync: --delete does not work without --recursive (-r) or --dirs (-d).
rsync error: syntax or usage error (code 1) at main.c(1453) [client=3.0.9]
11 years ago
James Cammarata
f2c06cadc3
Cleanup from merge of replace module and CHANGELOG update
11 years ago
James Cammarata
9775872f83
Merge branch 'devel' of https://github.com/EvanK/ansible into EvanK-devel
11 years ago
James Cammarata
d545b73ccb
Merge branch 'file_fixes_agg' of https://github.com/bcoca/ansible into bcoca-file_fixes_agg
11 years ago
Robert Wehner
8a19d31391
Update 'validate=' parameter documentation to make clear that the command
...
passed via this paramter is not passed via shell and so all shell features
will not work.
11 years ago
James Tanner
ea5186ca63
Fixes #6590 add set_remote_user parameter to synchronize
...
This allows usage of custom ssh configs for remote hosts where
the inventory user does not match the configured user.
11 years ago
Steve Smith
e157355fd6
Only unlink the symlink tempfile on error if it was created in the
...
previous operation.
11 years ago
Brian Coca
1b38bd8a40
fixed issue with tmp file (path can include filename) and added missing
...
else which made it try to link x2
11 years ago
Brian Coca
6e6ad97239
now correctly checks absolute path for src= existance for links
...
also updated docs to be a bit clearer on symlink behaviour
11 years ago
Brian Coca
f6a75ebaf1
fixed issue with directory permissions on copy #6552
11 years ago
James Cammarata
5d8c9d3f0f
Merge branch 'file_fixes_clean' of https://github.com/bcoca/ansible into bcoca-file_fixes_clean
11 years ago
Michael DeHaan
7286beb15e
Merge pull request #6515 from lotia/fix-acl-doc-and-error
...
Fix acl doc and error
11 years ago
Ali Asad Lotia
4a006a78b3
Whitespace fixes in incompatible param check
...
- Remove extra whitespace in conditional that checks if etype, entity or
permissions are also set when entry is set.
11 years ago
Michael DeHaan
5c05982dea
Merge pull request #6448 from jirutka/fix-5679-2
...
Fix #5679 again after recent changes in core
11 years ago
Cristian Ciupitu
0749112286
Micro-optimization: replace s.find(x)!=-1 with x in s
...
timeit shows a speedup of ~3x on Python 2.7.5 x86_64.
It also makes the code a bit shorter.
11 years ago
Ali Asad Lotia
9180013682
Fix acl module doc and error messages
...
- Fix typos in module doc
- Provide clearer error message when entry and etype are both missing
11 years ago
Brian Coca
2d25577e11
Fixes and cleanup to file functions and module
...
- unified set attribute functions ... not sure why 2 identical functions
exist with diff names, now there are 3 while i repoint all modules to 1
- fixed issue with symlinks being created w/o existing src when force=no
- refactored conditionals, simplified where possible
- added tests for symlink to nonexistant source, with both force options
- made symlink on existing attomic (force)
11 years ago
Michael DeHaan
01b7465a03
Merge pull request #6504 from follower/patch-2
...
Fix typo
11 years ago
follower
af6109c169
Fix typo
...
"resort" -> "result"
11 years ago
Brian Coca
b81a3bbb20
fixed bad detection of change on default and premature break
11 years ago
Jakub Jirutka
2bfaacd170
Fix #5679 again after recent changes in core
11 years ago
Brian Coca
463a1c21ea
now stat skips md5 if file isn't readable
11 years ago
James Tanner
ba0fec4f42
Force command action to not be executed by the shell unless specifically enabled
11 years ago
Dolph Mathews
8808e029f9
spelling correction for "separate"
...
s/seperate/separate/
http://en.wiktionary.org/wiki/separate
11 years ago
Richard C Isaacson
a483e42583
unarchive module: Add creates functionality.
...
Closes GH-5354.
Forklift copy logic into unarchive. Provides integration unit tests.
11 years ago
Michael DeHaan
7710867922
Merge pull request #6308 from muffl0n/file_fix_documentation
...
Module "file": Fix link to "path" in documentation
11 years ago
James Cammarata
9f2999df79
Un-escape newlines in delimiters for assemble module
11 years ago
Sven Schliesing
f62cc1e351
Fix link in documentation
...
c(dest) -> c(path)
11 years ago
Michael DeHaan
6c43e0e010
Simplify file module parameter description.
11 years ago
Richard C Isaacson
f17e549ff9
Typo fix.
11 years ago
Richard C Isaacson
809b714bfc
Replace mode with octal renaming to mode.
...
Closes GH-6176
This way for mode we return in an octal format which is immediately usable compared to transforming it later.
11 years ago
Richard Isaacson
cca7fd3c0c
Merge pull request #1 from bcoca/stat_updaates
...
added octal representation of mode and made md5 checksumming optional
11 years ago
Richard C Isaacson
7b6c7366d1
Minor cleanup.
11 years ago
Richard C Isaacson
890202e4c0
copy module: cleanup and if directory_mode not set default to umask
11 years ago
Richard C Isaacson
cf4a6b39cc
Copy Module: Add the ability to set directory attributes on recursive copy.
...
Closes GH-6194.
These changes pass attribute modifications through to new directories during a recursive copy with an addition allowing for the directory mode to be set independantly.
11 years ago
James Cammarata
32a2636dc7
Add expanduser calls on src/dest in the module too
11 years ago
James Cammarata
e659d55d62
Merge branch 'devel' of https://github.com/muffl0n/ansible into muffl0n-devel
11 years ago
Brian Coca
53d71a3949
added octal representation of mode and made md5 checksumming optional
...
Signed-off-by: Brian Coca <briancoca+dev@gmail.com>
11 years ago
James Cammarata
3b2d409906
Fixing several bugs in assemble and updating tests
...
Bugfixes:
* the remote_src param was not being converted to a boolean correctly,
resulting in it never being used by the module as the default behavior
was remote_src=True (issue #5581 )
* the remote_src param was not listed in the generic file params, leading
to a failure when the above bug regarding remote_src was fixed
* the delimiter should always end with a newline to ensure that the file
fragments do not run together on one line
Fixes #5581
11 years ago
James Cammarata
f03d6a67c8
Adding new tests for assemble role
11 years ago
James Cammarata
2157880029
Adding the backup destination to the resut dictionary for lineinfile
11 years ago
muffl0n
b0f00cc097
already call expanduser in action_plugin
11 years ago
Michael DeHaan
30611eaac5
Merge pull request #6036 from mxxcon/patch-1
...
docs_pull_request: Add parameters highlighting
11 years ago
James Cammarata
9c9a940af6
Merge branch 'fix-5679' of https://github.com/jirutka/ansible into jirutka-fix-5679
11 years ago
mxxcon
4e4be3edc0
docs_pull_request: Add parameters highlighting
...
Added highlighting to parameter references in docs so that they standout a bit better.
11 years ago
Michael DeHaan
56befa9468
Document ansible_managed changing the template.
11 years ago
Michael DeHaan
6f9ca7cfa5
Some minor docs corrections.
11 years ago
Richard C Isaacson
598b9c6b7c
Cleanup per notes.
...
Some small changes to per notes from @mpdehann.
11 years ago
Richard C Isaacson
cb7c2b7524
Updates for the unarchive module and action_plugin.
...
There is a bit going on with the changes here. Most of the changes are cleanup of files so that they line up with the standard files.
PR #5136 was merged into the current devel and brought up to working order. A few bug fixes had to be done to get the code to test correctly. Thanks out to @pib!
Issue #5431 was not able to be confirmed as it behaved as expected with a sudo user.
Tests were added via a playbook with archive files to verify functionality.
All tests fire clean including custom playbooks across multiple linux and solaris systems.
11 years ago
Richard C Isaacson
e6c9705058
Merge remote-tracking branch 'pib/unarchive_remote' into pull_5136
11 years ago
Mike Grozak
f386e137fe
Updated the force parameter description; added one example how to create the symlinks pointing to non-absolute paths
11 years ago
Mike Grozak
d3b544e842
Found a way how to have symlinks towards files without absolute paths in prefixes; fixed incorrect force setting on the hard links, which can not point to non-existing files
11 years ago
Mike Grozak
2b4688a0bd
Started the development of the 'chdir' functionality for file module - creation symlinks without the base directory prefix
11 years ago
jctanner
a1d2b5f257
Merge pull request #5363 from bcoca/acl_improved
...
acl module: acl now supports 'default'
11 years ago
Brian Coca
54a79bfc75
fixed docstring and changed type to etype avoiding python builtin
...
Signed-off-by: Brian Coca <briancoca+dev@gmail.com>
11 years ago