Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

validate files: fix usage error in example tasks #3350

Merged
merged 1 commit into from
Jun 26, 2013
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
8 changes: 4 additions & 4 deletions library/files/copy
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ description:
options:
src:
description:
- Local path to a file to copy to the remote server; can be absolute or relative.
- Local path to a file to copy to the remote server; can be absolute or relative.
required: false
default: null
aliases: []
Expand Down Expand Up @@ -75,7 +75,7 @@ options:
required: false
author: Michael DeHaan
notes:
- The "copy" module can't be used to recursively copy directory structures to the target machine. Please see the
- The "copy" module can't be used to recursively copy directory structures to the target machine. Please see the
"Delegation" section of the Advanced Playbooks documentation for a better approach to recursive copies.
'''

Expand All @@ -87,7 +87,7 @@ EXAMPLES = '''
- copy: src=/mine/ntp.conf dest=/etc/ntp.conf owner=root group=root mode=644 backup=yes

# Copy a new "sudoers" file into place, after passing validation with visudo
- copy: src=/mine/sudoers dest=/etc/sudoers validate='visudo -c %s'
- copy: src=/mine/sudoers dest=/etc/sudoers validate='visudo -cf %s'
'''

def main():
Expand All @@ -96,7 +96,7 @@ def main():
# not checking because of daisy chain to file module
argument_spec = dict(
src = dict(required=False),
original_basename = dict(required=False), # used to handle 'dest is a directory' via template, a slight hack
original_basename = dict(required=False), # used to handle 'dest is a directory' via template, a slight hack
content = dict(required=False, no_log=True),
dest = dict(required=True),
backup = dict(default=False, type='bool'),
Expand Down
2 changes: 1 addition & 1 deletion library/files/template
Original file line number Diff line number Diff line change
Expand Up @@ -58,5 +58,5 @@ EXAMPLES = '''
- template: src=/mytemplates/foo.j2 dest=/etc/file.conf owner=bin group=wheel mode=0644

# Copy a new "sudoers file into place, after passing validation with visudo
- action: template src=/mine/sudoers dest=/etc/sudoers validate='visudo -c %s'
- action: template src=/mine/sudoers dest=/etc/sudoers validate='visudo -cf %s'
'''