| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
| |
* keep unsafe .. unsafe
fixes #23734, which was broken in previous fix that allowed non string types to be templated
use new 'is_template' function vs bastardizing others
refactored clean_data to allow for arbitrary data structures to clean
fixed/removed some tests
* deal with complex data for is_template
* typos
|
|
|
|
|
|
| |
output. (#23832)
(cherry picked from commit 3f5b304fc28b6c34df9ea6a4c3531dc422ce198b)
rebased for @yannig
|
|
|
|
|
|
|
|
|
|
| |
We have a list of specific messages that we scree-scrape and flag
them as legit errors.
However, we also have a catch-all regex that matches everything
starting with %.
That can cause issues on commands that return lines with that
character, like for example the 'crypto key generate'.
Fixes #23770
|
| |
|
|
|
|
| |
* Add junos_command module unit test
|
|
|
|
|
|
|
|
|
|
|
| |
In current stable (2.2), ansible galaxy install --force do erase
a role, even if the version is not set. This commit should restore
that specific behavior, in accordance to people reports:
https://github.com/ansible/ansible/issues/11266#issuecomment-273801480
It was also the behavior planned in the initial discussion:
"if you're not fixing versions in your roles file, then it's fine
to expect that the role will be reinstalled each time you run
ansible-galaxy install.", cf https://github.com/ansible/ansible/pull/12904
|
|
|
|
| |
We hit bug #23737 due to bad coverage on test_iosxr_facts, we
were not checking memory facts at all.
|
|
|
|
|
|
| |
We were not calling match.group, plus we were lacking a ':' from
the expected output of 'show memory summary'.
Fixes #23737
|
|
|
| |
Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
|
| |
|
|
|
| |
see #23729
|
|
|
|
| |
* fixes #23586
* temporary solution until connection-specific vars are handled by play_context
|
|
|
|
| |
(cherry picked from commit 20d9f79a5250e764ffeddb046aeb82b7824ed952)
|
|
|
|
|
|
|
|
|
|
|
| |
* Revert "fixes play context connection user (#21776)"
This reverts commit 58ee661437f123fdd8cf516bac053fe593910ced.
fixes #23530
* preserve original in copy
updated comment to clarify wtf is going on here
|
|
|
| |
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
|
|
|
| |
make pep8
|
|
|
|
|
|
|
|
| |
(#23802)
* Add example for using route53 next_marker option
* make route53_facts pep8 and remove from legacy files
|
|
|
|
|
|
|
|
| |
* Hook port back up for eos cli transport
* The default values have moved
* Why not have conditional defaults?
|
| |
|
|
|
| |
* Give a default to eos transport
|
|
|
| |
Missed that D...
|
| |
|
|
|
| |
Thanks @adamchainz!
|
| |
|
|
|
|
|
|
|
|
|
|
| |
fixes #17382
alternate to #22979
deal with cases in which group/host have . in name
updated as per feedbck
only be strict about extension when doing dirs
also avoid ~ endings
|
|
|
|
| |
fixes #23729
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
* Add guide for Packet host
* incorporate feedback from Packet
* Format headings according to Python doc guide
* fixed rstcheck issues in packet guide
|
| |
|
| |
|
|
|
| |
Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
|
| |
|
|
|
| |
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
without this patch, ansible-galaxy will mangle files containing
the archive parent directory name, eg 'owncloud/files/owncloud.cron'
will become 'owncloud/files/.cron'.
The previous code could affect the entire path and even filenames.
If a file path has the top level dir name as a substring, galaxy
replaces it with ''. In one example, the archive top level dir
is 'go', so 'files/go-bin.sh' becomes 'files/-bin.sh'.
Fixes #22572, #23694, #23623
|
| |
|
|
|
|
|
|
|
| |
d0e3d75381292fc7bf417598f9f6542a7fcd2d07
* Use a generator expression instead of a list comprehension
* Use copy() to create a new set instead of the constructor.
|
| |
|
|
|
| |
Update the Ansible 2.3 New Module list to include the cisco_spark notification module.
|
|
|
|
|
|
|
| |
Miscallenous fixes added to make gconftool-2 work.
Fixes https://github.com/ansible/ansible/issues/23670
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
|
| |
|
| |
|
|
|
| |
No docs were included when https://github.com/ansible/ansible/pull/9927 was merged.
|
|
|
|
|
|
|
|
| |
* Removing irrelevant s3 call
Fix exception handling
Make s3_sync pep8 and remove from legacy file
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Fix vault reading from stdin (avoid realpath() on non-links)
os.path.realpath() is used to find the target of file paths that
are symlinks so vault operations happen directly on the target.
However, in addition to resolving symlinks, realpath() also returns
a full path. when reading from stdin, vault cli uses '-' as a special
file path so VaultEditor() will replace with stdin.
realpath() was expanding '-' with the CWD to something like
'/home/user/playbooks/-' causing errors like:
ERROR! [Errno 2] No such file or directory: u'/home/user/ansible/-'
Fix is to specialcase '-' to not use realpath()
Fixes #23567
* to_text decrypt output when writing to stdout
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Make warning logs consistent
Arguments outside provider with default
value should not log as warning in case
it is not mentioned in play.
* Make nxos timeout default consistent and add comments
* Make comments more verbose
|
|
|
|
|
|
|
|
|
| |
update module to support more standard state=present/absent syntax
update module to use required_if, required_together, mutually_exclusive functions where possible
per ryansb review: make documentation section more clear, fix some extra quotes, remove FIXME comment
pre willthames review: force private_zone to True if vpc_id is set and fix word wrap
|