cc173b445e
Fixed module doc typos
2016-12-08 11:24:06 -05:00
bbfb957bc1
recursive copy is not available(yet) on remote_src
2016-12-08 11:24:06 -05:00
1b642fcf75
Fix typo in the copy module error message
...
Fix the typos in the error message shown on trying to use remote_src=yes for copying directories
2016-12-08 11:24:06 -05:00
23c07ccf2c
cleaner error on recursive remote copy
...
TODO: add recursive copy support when remote_src=yes
probably need to make shared the logic in the action plugin.
2016-12-08 11:24:06 -05:00
8d0d251003
removed 'overquoting' of example
...
fixes #3029
2016-12-08 11:24:05 -05:00
1ecc8544f6
Add diff for both file contents and file attributes to lineinfile
...
Using the difflist feature added in ansible/ansible@c337293 we can add
two diffs to the `diff` dict returned as JSON: A `before` and `after` pair of
changed file contents and the diff of the file attributes.
n.b.: the difflist handling from the above commit is logically broken.
PR will follow.
Example output:
TASK [change line and mode] ************************************************************
changed: [localhost]
--- before: /tmp/sshd_config (content)
+++ after: /tmp/sshd_config (content)
@@ -65,21 +65,21 @@
X11DisplayOffset 10
PrintMotd no
PrintLastLog yes
TCPKeepAlive yes
#UseLogin no
#MaxStartups 10:30:60
#Banner /etc/issue.net
# Allow client to pass locale environment variables
-AcceptEnv LANG LC_*
+AcceptEnv LANG LC_* GF_ENV_*
Subsystem sftp /usr/lib/openssh/sftp-server
# Set this to 'yes' to enable PAM authentication, account processing,
# and session processing. If this is enabled, PAM authentication will
# be allowed through the ChallengeResponseAuthentication and
# PasswordAuthentication. Depending on your PAM configuration,
# PAM authentication via ChallengeResponseAuthentication may bypass
# the setting of "PermitRootLogin without-password".
# If you just want the PAM account and session checks to run without
--- before: /tmp/sshd_config (file attributes)
+++ after: /tmp/sshd_config (file attributes)
@@ -1,3 +1,3 @@
{
- "mode": "0700"
+ "mode": "0644"
}
2016-12-08 11:24:02 -05:00
f6aa1ff9a4
Remove duplicate documentation fields
2016-12-08 11:24:01 -05:00
ceba43e701
Better error message when rsync nad ssh aren't present where synchronize is run
...
Fixes https://github.com/ansible/ansible/issues/9305
2016-12-08 11:24:01 -05:00
25ccb98d49
Add custom or 'prepared' diff field to synchronize module
...
This PR depends on pull requeest ansible/ansible#14105
rsync has a custom diff output that cannot easily be expressed as
`/usr/bin/diff before after`
2016-12-08 11:24:01 -05:00
457dfb3b4b
Add an internal param instructing synchronize to replace localhost with the host that's being ssh'd from
2016-12-08 11:24:01 -05:00
9e686c7294
Add note about precautions of using become with fetch. Addresses https://github.com/ansible/ansible/issues/14064
2016-12-08 11:24:01 -05:00
5e4fe92676
Fix documentation build
2016-12-08 11:24:01 -05:00
5f35d0d597
Document that synchronize in 2.0.0.x has broken sudo behaviour.
...
Part of the changes for #13825
2016-12-08 11:24:01 -05:00
288a5c9a77
adds diff info for file info
2016-12-08 11:23:56 -05:00
83b45ae047
Fix doc inconsistency ( #2630 )
2016-12-08 11:23:55 -05:00
63fd67f156
added mime option to stat module
...
it uses file magic to now return mime_type and charset of a file as
per output of `file -i /path`
2016-12-08 11:23:53 -05:00
9c131ea619
added a reference to the template module for clarity
2016-12-08 11:23:52 -05:00
3ad67d93ea
Fix: Default ACL parameters are not correctly handled
2016-12-08 11:23:51 -05:00
f0a3914f7b
simplified lowercasing
2016-12-08 11:23:50 -05:00
3a91853e7e
Add "default" entry option back (removed in e95bcae), update will translate entry to standard parameters so compatibility with BDS is kept
2016-12-08 11:23:50 -05:00
df4109d946
Fix #2475 ini_file module: bracklets in key break idempotence
...
Escape the regex special characters in the option name.
2016-12-08 11:23:50 -05:00
fe0aa01ce8
Fix 'stat' module document
2016-12-08 11:23:50 -05:00
579e444243
improved error message when no handler found
2016-12-08 11:23:50 -05:00
3025451b10
fixed typo EEXISTS is actually EEXIST
...
fixes #2585
2016-12-08 11:23:50 -05:00
4c93b5dab6
Add documentation to 'file' AnsibleModule definition for internals.
...
The parameters 'diff_peek' and 'validate' are not expected to be used
by users. They are internal. To make it clear, this change adds the
comments 'Internal use only' to each of those definitions to make
it clear that they are actually used, just not by end-users.
2016-12-08 11:23:50 -05:00
e86ba78685
Update documentation for 'file' module to include 'diff_peek'.
...
The 'diff_peek' option isn't documented at all, and provides a
rudimentary check that the content isn't binary. Documentation is
added to explain the option.
The 'validate' option has a declaration, but isn't implemented.
Therefore it may as well be removed from the module.
2016-12-08 11:23:49 -05:00
52d499d268
create non-existent ini file
...
fixing fail_json
more verbose fail msg
2016-12-08 11:23:48 -05:00
e0b8b0d789
This fixes copy with the remote_src option
...
It was broken in 6e37f1dcef0 when the remote_src was added. Need to
pass the absolute path to copy2 instead of a tuple.
2016-12-08 11:23:48 -05:00
139201ad83
fixed remote_src support, now actually copies and does not move
2016-12-08 11:23:48 -05:00
f2e83fa452
Added the checksum_algo alias to the stats module.
2016-12-08 11:23:47 -05:00
4e787f17cf
Make the syntax work with Python 2.4
2016-12-08 11:23:47 -05:00
bd03dd67ff
ini_file should only change what was specified and nothing more #5860
...
See also:
http://alesnosek.com/blog/2015/08/03/improving-ansibles-ini-file-module/
2016-12-08 11:23:47 -05:00
f7f8df1773
since find doesn't make changes, support check mode and gather data for other tasks in check mode
2016-12-08 11:23:44 -05:00
a179d06a91
Simplify logic to handle options set to empty string
...
Fixes #2125
2016-12-08 11:23:44 -05:00
14455f4acc
final form, use_regex now controls if patterns is glob or regex - fixed cases in which stat fails (dangling symlink) - now properly reports name of skipped paths
2016-12-08 11:23:43 -05:00
3e5dc1fd74
changed so regexes and shell globs work transparently
2016-12-08 11:23:43 -05:00
8a1c035fac
Add capability for stat module to use more hash algorithms
...
Specifically, the stat module now has a checksum_algorithm parameter.
This lets the module utilize one of the hash algorithms available on the host
to return the checksum of the file.
This change is backwards compatible. The checksum_algorithm defaults to
sha1 and still returns its result to the stat.checksum property.
2016-12-08 11:23:43 -05:00
7c0dc323a2
corrected docs for stat's lnk_source fixes #12850
2016-12-08 11:23:42 -05:00
baeb6d8dc0
fixed bug in spec
2016-12-08 11:23:42 -05:00
a964754484
added regex support to find, also added 'singular' aliasess to patterns and paths
2016-12-08 11:23:42 -05:00
d6f474761d
documented new remote_src option
2016-12-08 11:23:41 -05:00
a05df49998
Fixed confusion in the docs
...
As 'path' is actually alias for 'dest', this could be confusing.
2016-12-08 11:23:37 -05:00
58573187f8
Change the example of rsync_opts to use a list instead of a string
...
Fixes #9889
2016-12-08 11:23:37 -05:00
4a4674ca24
Fix syntax error in synchronize docstring
2016-12-08 11:23:36 -05:00
dabf94ab1f
Explanation of privilege escalation in notes
...
An attempt to make clear how privilege escalation works with respect to the src/source host and dest/destination host. One existing note was incorporated into three new ones, iterating each.
2016-12-08 11:23:36 -05:00
6695cc5c21
Fix physical walk on acl module for Linux
...
`-h` is for BSD [1] while `-P`/`--physical` is for Linux [2].
This commit fixes that option now that acl module is (temporarily)
only supported for Linux.
I will re-add `-h` when fixing BSD support.
[1]: http://www.freebsd.org/cgi/man.cgi?format=html&query=setfacl(1)
[2]: http://linuxcommand.org/man_pages/setfacl1.html
2016-12-08 11:23:33 -05:00
186cc7cb17
Make sure entry is not sent when acl state=query
2016-12-08 11:23:33 -05:00
3568f1cd78
Make sure permission-less entries are accepted when state=absent
...
Also, remove that try condition as, at that stage, no permissions with
other than 2 or 3 fields are sent to the function.
2016-12-08 11:23:33 -05:00
a93445c798
Remove support for `d[efault]:` in entry permissions
...
It is not documented in [the Ansible doc page][1] nor
[the BSD setfacl man entry][2] (which means it might not be compatible
with BSD) so removing it does not break the API.
On the other hand, it does not conform with POSIX 1003.1e DRAFT
STANDARD 17 according to the [Linux setfacl man entry][3] so safer to
remove.
Finally, the most important reason: in non POSIX 1003.e mode, only ACL
entries without the permissions field are accepted, so having an
optional field here is very much error-prone.
[1]: http://docs.ansible.com/ansible/acl_module.html
[2]: http://www.freebsd.org/cgi/man.cgi?format=html&query=setfacl(1)
[3]: http://linuxcommand.org/man_pages/setfacl1.html
2016-12-08 11:23:33 -05:00
1b34f1a7ac
Fix wrong expectation regarding entry format in acl module
2016-12-08 11:23:32 -05:00
6c298e20c0
Fix wrong processing of lines returned by setfacl in test mode
2016-12-08 11:23:32 -05:00
08e8f732c5
Fix minor whitespace issues
2016-12-08 11:23:32 -05:00
2ed50d5e5b
assemble: add file validation support.
2016-12-08 11:23:31 -05:00
a1992ad890
updated encoding comment and expanded authors field
2016-12-08 11:23:31 -05:00
288a9d8500
Set version number in recursive option description
2016-12-08 11:23:30 -05:00
17170992c3
Ensure Python 2.4 compatibility and Linux-restricted support
...
- Make build_entry compatible with Python 2.4
- Re-add missing warning/comment that was forgotten while refactoring
- Replace `all()` with a good ol' for-loop Python 2.4 compatibility
- Make a condition check more explicit (when `state` is `query`)
- Make sure this module can only be run with on a Linux distribution
- Add a note about Linux-only support in the documentation
- Set the version in which recursive support was added, 2.0
2016-12-08 11:23:30 -05:00
9ab2e9c740
Fix obvious copy-paste error in module docs
2016-12-08 11:23:30 -05:00
a10b0e6fff
Modify the default for synchronize's dest_port again
2016-12-08 11:23:30 -05:00
052320b452
Update docs to reflect precedence for synchronize's dest_port param
2016-12-08 11:23:29 -05:00
9ba5a438a8
Allow mode as a synchronize parameter (handled in action plugin)
2016-12-08 11:23:29 -05:00
47065c2a43
Removed check whether destination directory is writable from files/unarchive.py. This check will prevent extraction of an archive if the archive does not actually write to the destination directory but only writes to any writable sub-directories of it. The underlying tar command will report errors should it try to write to read-only directories.
2016-12-08 11:23:29 -05:00
edf531efd7
Add recursive support for ACL module ( fix #5053 , fix #5550 , fix #7276 )
2016-12-08 11:23:28 -05:00
a01a800b65
Fi the local_rsync_path parameter used by the action plugin
2016-12-08 11:23:26 -05:00
ba9ba7d0f5
In copy, set the mode before running th validation
...
Because some programs that do validation (like visudo) may require
the permissions to be more restricted.
Fixes ansible/ansible#11385
2016-12-08 11:23:26 -05:00
56354731ce
fixed fragment docs
2016-12-08 11:23:26 -05:00
9623cd570c
Add missing GPLv3 License header
...
Fixes #1643
2016-12-08 11:23:25 -05:00
70b3290a3b
Move validate command into doc fragment and fix a few typos
2016-12-08 11:23:24 -05:00
07ccd3c86b
Handle ini file properties that contain interpolation errors in the initial values
2016-12-08 11:23:23 -05:00
88881415e9
synchronize: add flag for verifying target host.
...
Add the possibility to verify the target host using a "verify_host"
flag. It is disabled by default to not change the module behaviour.
2016-12-08 11:23:22 -05:00
e4dd0d98d4
now captures any exceptions when trying to create directories
2016-12-08 11:23:21 -05:00
60d160a2e9
Also document in example that unarchive download was added in 2.0
2016-12-08 11:23:20 -05:00
b664603a4d
Improved stat documentation
2016-12-08 11:23:20 -05:00
f8f4a7dcdf
added ignore hidden to assemble
2016-12-08 11:23:18 -05:00
17a40aa259
Handle race condition in directory creation.
...
If we try to make a directory, but someone else creates the directory
at the same time as us, we don't need to raise that error to the user.
They asked for the directory to exist, and now it does. This fixes
the race condition which was causing that error to be raised, and
closes #1648 .
2016-12-08 11:23:16 -05:00
6acfa5fcee
unarchive: fix @bcoca's remarks, issue #1575
2016-12-08 11:23:16 -05:00
b898cb656b
unarchive: fix work with 0 bytes archives
...
This change is in response to issue #1575
2016-12-08 11:23:15 -05:00
db646757c1
Add "force" description
...
Add "force" option description
2016-12-08 11:23:15 -05:00
fd69c5687b
Do not insert extra newline if line already contains it
...
When using YAML multi-line strings, e.g.:
- lineinfile:
dest: /tmp/foo
line: >
foo
bar
the line already ends with a newline. If an extra newline is appended unconditionally it will lead to inserting an extra newline on each run.
2016-12-08 11:23:13 -05:00
870446dd6b
Update version_added to 2.0 for the partial option
2016-12-08 11:23:12 -05:00
81ea358b09
synchronize module: add partial option
2016-12-08 11:23:12 -05:00
5f27a073ba
Bump amount of file to download in a chunk to 64k.
2016-12-08 11:23:11 -05:00
dd0659c83d
add download ability to unarchive module
2016-12-08 11:23:11 -05:00
9c3f9d23c8
Add the ability to grep for content
2016-12-08 11:23:09 -05:00
eb881d7d5d
Proper author info for all remaining modules
2016-12-08 11:23:07 -05:00
75ef963922
removes bool type from validate.
2016-12-08 11:23:03 -05:00
0180be25c4
standardizes bools in argument_spec
2016-12-08 11:23:03 -05:00
fbc8d4ca4a
use the right way to unescape line string Reference https://github.com/ansible/ansible/issues/10864
2016-12-08 11:23:03 -05:00
322518d06c
stat doc fix fixes #1371
2016-12-08 11:23:02 -05:00
79dd0466b5
Adding directory creation example to file module.
2016-12-08 11:23:00 -05:00
b4775234c9
minor docfix
2016-12-08 11:23:00 -05:00
7e5a4afb6b
Update module "imports" to new style
...
Fixes #1351
2016-12-08 11:23:00 -05:00
20d7f929be
Fix use of codecs.escape_decode()
2016-12-08 11:22:59 -05:00
5204e94bda
Slightly more future-proof version of the lineinfile fix
2016-12-08 11:22:59 -05:00
1002fafedf
Correct lineinfile documentation about double quoting... Hasn't been the case for years.
2016-12-08 11:22:59 -05:00
329266975b
update flat description
...
default behavior is to append the `hostname/path/to/file`, not prepend as currently stated by the flat arg docs
2016-12-08 11:22:59 -05:00
0567404c03
generic fix for Exceptions that heppen when trying to set permissions on extracted files fixes ansible/ansible#10934 which is really a corner case
2016-12-08 11:22:58 -05:00
118382ca70
synchronize: don't add ssh_args to ssh_opts when false or null
2016-12-08 11:22:56 -05:00
1ceaa78dc0
Add version_added for the use_ssh_args option
2016-12-08 11:22:55 -05:00
4faf0d59da
Add ability to specify using ssh_args in synchronize
2016-12-08 11:22:55 -05:00
1ddae11bd1
Implement feature to return list of files in the archive
2016-12-08 11:22:54 -05:00
Jose Moreira
Brian Coca
Guruprasad
Tobias Wolf
Toshio Kuratomi
Matt Martz
Michael Scherer
Adam Fields
quoing
Ales Nosek
krdlab
Dylan Martin
Charles Ferguson
Charles Paul
Maarten Claes
Timothy Appnel
Ryan Sydnor
varnav
James Cammarata
skylerbunny
Jérémie Astori
Matthias Frey
Marius Gedminas
Markus Juenemann
Mike Putnam
Jeff Widman
Isaac Simmons
Patrik Lundin
Mitchell Ludwig
Matthew Gilliard
verm666
Vladimir Martsul
gimoh
Juan Picca
Jonathan Mainguy
Ritesh Khadgaray
Greg DeKoenigsberg
Tristan Fisher
jaypei
Greg Taylor
Jason Paige
Ryan Hartkopf