Commit Graph

266 Commits (5a8ce71a4a2504214916ec07a8eafe71ad844fb3)

Author SHA1 Message Date
James Cammarata 58a0bcbe51 Fix issue with symlink path detection
Fixes #7627
Fixes #7664
11 years ago
James Cammarata 10074d3622 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 4e4ffa669a Correctly examine path for relative links in file module
Fixes #7515
Fixes #7537
11 years ago
jjshoe b01da1b780 Ensure the validation string has %s in it 11 years ago
James Cammarata 1248d2ce8a Fixing absolute path calculation for file module when linking
Fixes #7403
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
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
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
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 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