| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
fixes #51025
added test cases
|
|
|
|
|
|
|
| |
* Allow restricting config values to enumerated list
* dont document internal entries
* also ignore private defs for ansible-config
* remove invalid value from tests
* added porting entry
|
|
|
|
| |
collection verify' (#73229)
|
|
|
|
|
|
|
|
| |
When the pause module is run in the background and seconds parameter is provided,
do not warn.
* Add tests
* Fix existing tests
The test wasn't failing when it should have.
|
| |
|
|
|
|
|
| |
* Fix connection_paramiko_ssh test for macOS 11
* Update Azure Pipelines config
* Add changelog
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Do the right thing on Linux when password lock and a password hash are provided by writing
out the password hash prepended by the appropriate lock string rather than using -U and -L.
This is the correct way to set and lock the account in one command.
On BSD, run separate commands as appropriate since locking and setting the password cannot
be done in a single action.
FreeBSD requires running several commands to get the account in the desired state. As a result,
the rc, output, and error from all commands need to be combined and evaluated so an accurate
and complete summary can be given at the end of module execution.
* Improve integration tests to cover this scenario.
* Break up user integration tests into smaller files
* Properly lock account when creating a new account and password is supplied
* Simplify rc collection in FreeBSD class
Since the _handle_lock() method was added, the rc would be set to None, which could make
task change reporting incorrect. My first attempt to solve this used a set and was a bit too
complicated. Simplify it my comparing the rc from _handle_lock() and the current value of rc.
* Improve the Linux password hash and locking behavior
If password lock and hash are provided, set the hash and lock the account by using a password
hash since -L cannot be used with -p.
* Ensure -U and -L are not combined with -p since they are mutually exclusive to usermod.
* Clarify password_lock behavior.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change:
- Internally, use dnf.subject.Subject#get_best_query for state: absent
- Add a bunch of tests for removing packages, given a bunch of different
pkg specs (nv, nvr, nvra, wildcard, etc.)
Test Plan:
- New tests
- Local experiments with DNF API via PDB.
Tickets:
- Fixes #72809
Signed-off-by: Rick Elrod <rick@elrod.me>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change:
- Changes to make F33 work with current tests
- Add F33 to docker.txt
- Add F33 to AZP config
Test Plan:
- ci_complete
Signed-off-by: Rick Elrod <rick@elrod.me>
|
|
|
|
|
|
|
| |
* Add ansible-galaxy-collection-scm tests for requirements.yml format
* Add a test for --no-deps
Fix assertion
|
|
|
|
|
|
| |
The recent update of ptyprocess to 0.7.0 is incompatible with Python 2.6 and
is causing test failures.
* Add setup_pexpect role to expect test
|
|
|
|
|
|
|
|
| |
Fix get_url failure in check mode (--check) when using the checksum
format <algorithm>:<url>.
Regression introduced in (#20532)
Fixes: #61369
|
|
|
|
|
|
|
| |
* Ensure we actually fallback to unsafe_writes when set to true
add integration test
add fix for get_url not passing the parameter from args
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* systemd - do not overwrite unit name when searching
PR #72702 introduced a bug that changed the unit name when splitting it up for the purpose
of searching for the unit. This only happens on unit file templates on systems that have a 5.8
or newer kernel and a version of systemd that does not contain a bugfix that causes systmed
to fail to parse dbus.
* Use facts rather than a manual probe to determine if systmed is present
* Remov unnecessary block
* Use vars files instead of set_fact
* Add tests for using a templated unit file
* Update changelog fragment
* Use template to get correct path to sleep binary
|
|
|
|
|
|
|
| |
* import_playbook - change additional params to deprecation
I incorrectly recommended this be set as a warning when it should have been a deprecation.
* Fix deprecation sanity test to not required a collection name when not inside a collection
|
| |
|
|
|
|
|
|
|
| |
* Update ansible-test pylint Python support.
* Python 3.8 is now officially supported.
* Python 3.9 is now skipped with a warning.
|
|
|
| |
Test installing a collection to a directory containing other collections without any metadata (#72971)
|
|
|
|
|
|
|
|
|
| |
* saner path dir management
fixes #72628
ensure we always store paths w/o a_c
Co-authored-by: Sloane Hertel <19572925+s-hertel@users.noreply.github.com>
|
|
|
|
|
|
| |
Having the trap exit with a specific code will override
the exit code that caused the trap to run, which could
mask errors
|
|
|
|
|
|
|
|
|
| |
* Change role argspec file used by ansible-doc
This changes the file used for role argument specs from meta/argument_specs.yml
to meta/main.yml. The argument specs are expected to be in that file under the
top-level entry of `argument_spec`.
* Switch to argument_specs
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* ansible-galaxy collection list and verify now utilize collections in site-packages.
This is a short term fix for #70147. The long term fix needs to handle
install (but that discussion is also bound up in how upgrade is going to
work and where things can get installed so it's deferred for 2.11.)
* Add test for ansible-galaxy collection list with site-packages
* Fix sanity issue
Co-authored-by: David Moreau Simard <moi@dmsimard.com>
Co-authored-by: Jordan Borean <jborean93@gmail.com>
|
|
|
|
|
|
|
| |
* Test installing a dependency compatible with multiple collection requirements
Test downloading a collection without dependencies
* Move cleanup to the end of the test suite
|
|
|
|
|
|
|
|
|
| |
* Ensure we only reset the connection when one has been previously established. Fixes #65812
* Ensure psrp doesn't trace
* winrm too
* Indentation fix
|
|
|
|
|
|
|
|
|
| |
* Fix async interpreter parsing. Fixes #70690
* Target localhost instead of remote host
* Don't forget inventory
* Address shellcheck issue
|
|
|
|
|
| |
Co-authored-by: Matt Martz <matt@sivel.net>
Fixes #69848
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This should allow users to extract specific files from an archive as
desired.
Fixes #16130, #27081.
* Rebase and make a few minor changes
* Add changelog
* Improve tests
- move to separate tasks file
- change assertions to check for exactly one file
- use remote_tmp_dir for output dir
* Make exclude and include mutually exclusive
* Don't remove files needed by other tasks
* Fix sanity tests
* Improve feature documentation
* Skip tests that use map() on CentOS 6
* Use fnmatch on include for zip archives
This matches the behavior of exclude
Co-authored-by: Sam Doran <sdoran@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
(#45198)
* file: add symlink is in a sticky directory tests
* file: handle symlink in a sticky directory
Co-Authored-By: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua>
* Add changelog and fix unit test
The builtins import was removed since it was unused, but it is now needed.
|
|
|
|
|
|
|
|
|
|
| |
Change:
- Use a version specific vars file to specify archive url for EPEL.
Test Plan:
- ci_complete
Signed-off-by: Rick Elrod <rick@elrod.me>
|
| |
|
|
|
| |
Signed-off-by: Yadnesh Kulkarni <ykulkarn@redhat.com>
|
|
|
|
|
| |
* ansible-galaxy - source deps from all servers and not just parent
* Added integration tests for this scenario
|
|
|
|
|
|
|
|
| |
We could not find reason for the docutils install so trying out removing it.
Also bumping to latest version of ansible-runner
Skip all python2 versions because next ansible-runner drops it
Co-authored-by: Elijah DeLee <kdelee@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Support listing roles in text and JSON
* Change tests for unfrack'd playbook_dir var
These tests were using '/tmp' for testing the setting of the playbook_dir
var. Now that we unfrack that var, MacOS will change this to '/private/tmp'
causing the tests to fail. We can choose a path that does not exist (since
unfrack does not validate existence) so that we can guarantee unfracking
will not change the value.
|
|
|
|
|
| |
* actual min changes needed for rhel8
- specifiy releaserver as dnf/vars is ignored
|
|
|
|
|
|
|
|
|
|
| |
Change:
- Initial set of changes for renaming to ansible-core
- Includes changelog fragment changes from base -> core
- Does NOT include docs changes
- Modifies detection stuff in setup.py to support ansible<2.9 and ansible-base
Test Plan:
- ci_complete
|
|
|
|
|
|
|
|
|
| |
* Consolidate logic for determining whether or not session is interactive
into a single function, is_interactive()
* Increase test coverage
I wasn't able to find a good way of simulating running a backgrounded test with CI since the
whole test is essentially run not in a TTY, which is similar enough to cause the new is_interactive()
function to always return false.
|
| |
|
| |
|
| |
|
|
|
| |
* avoid fatal exception on invalid collection name used in ansible-doc filter
|
|
|
|
|
|
| |
* allow any type of unsafe data
dont limit to strings
|
| |
|
|
|
|
|
|
|
|
| |
* ensure local exposes correct user
avoid corner case in which delegation relied on
playcontext fallback which was removed
fixes #72541
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Added caching mechanism for Galaxy API requests
* Add cache options and split up code
* Added unit tests
* Fix sanity test
* Use modified date and fix up caching for explicit servers
* Make sure credentials are not in cached server name
* Added test for getting updated cache version
* Changes from review
|
|
|
|
|
|
|
|
| |
* Ensure blockinfile correctly returns backupfile
Fixes #27626
based on #27859
Co-authored-by: Giovanni Sciortino (@giovannisciortino)
|
|
|
|
| |
Signed-off-by: Rick Elrod <rick@elrod.me>
|