| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
|
|
|
|
| |
(#73493)
(cherry picked from commit 0cdc410dce6658e93c06fa27e0100ddbb11e7015)
|
|
|
|
|
|
|
|
| |
2.9 (#73495)
* bugfix: use to_text(stdout) in psrp.Connection.put_file method (#73491)
Co-authored-by: jakegatsby <jakegatsby@example.com>
(cherry picked from commit f271d02a9fa07299ea4fccbd7554a1bcec1782dc)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change:
- Other targets might remove rpm-build as they clean up after
themselves. Ensure that it's present in setup_rpm_repo because
rpmfluff needs it.
Test Plan:
- Local experimentation with yum_repository and mysql_db (the latter of
which depends on a handler which was removing rpm-build) on
stable-2.9.
Signed-off-by: Rick Elrod <rick@elrod.me>
(cherry picked from commit aca5b0e43be4d7bd067abc6631978b1cdfa75d84)
|
|
|
|
|
|
|
|
| |
**SECURITY** - CVE-2021-20178
Hide user sensitive information like `privkey` and `authkey`
while logging in console.
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
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.
(cherry picked from commit 0e6c334115)
Co-authored-by: Sam Doran <sdoran@redhat.com>
|
|
|
|
|
|
|
| |
on bhyve (#73204). (#73234)
(cherry picked from commit df451636e74fe6f0021a5555392f84c2bf194432)
Co-authored-by: Sam Doran <sdoran@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
(#73243)
**SECURITY** - CVE-2021-20180
Hide user sensitive information which is marked as ``secured``
while logging in console.
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
|
|
|
| |
Co-authored-by: Andrew Klychkov <andrew.klychkov@gmail.com>
|
|
|
|
|
| |
(#73285)
Co-authored-by: Andrew Klychkov <andrew.klychkov@gmail.com>
|
|
|
|
|
| |
(#73286)
Co-authored-by: Andrew Klychkov <andrew.klychkov@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(#73241) (#73297)
If a YAML file fails to load due to a syntax error in a file, or there is an error in the last line of a
file, PyYAML reports the last line number of the file as the index where the error occurred.
When reading the file lines, we use that index to the get the relevant line. If the index value is out
of range, the relevant line is lost for error reporting.
Subtract one from the index value to avoid the IndexError in this specific scenario. It is possible
to still get an IndexError, which will be handled as it is currently.
* Update existing tests and add new tests
(cherry picked from commit e8d4b62b41)
Co-authored-by: Sam Doran <sdoran@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Ability to add local variables into AnsibleJ2Vars was added in
18a9eff11f0a6e51b17405ce596bd9ff7e676320 to fix #6653. Local variables
are added using ``AnsibleJ2Vars.add_locals()`` method when creating a
new context - typically when including/importing a template with
context. For that use case local template variables created using
``set`` should override variables from higher contexts - either from the
play or any parent template, or both; Jinja behaves the same way.
Also removes AnsibleJ2Vars.extras instance variable which is not used.
Also adds missing test for #6653.
Fixes #72262
Fixes #72615
ci_complete
(cherry picked from commit a2af8432f36ec8cc5368a747f1211d2b9ba01f2e)
|
|
|
|
|
|
|
| |
Changed [int] to [int64] to support larger folders. Otherwise module fails as soon as a large folder is encountered.
(cherry picked from commit 8def67939dbd5dbba84fe160f3ad187c76ebe63a)
Co-authored-by: joshinryz <josh.robinett@gmail.com>
|
|
|
| |
Co-authored-by: Andrew Klychkov <andrew.klychkov@gmail.com>
|
|
|
|
|
| |
* Fix inventory cache warning by checking if the key exists before loading it
(cherry picked from commit 840bdc1e10f1f0d3c8c0ce4109e9724b466202c0)
|
|
|
|
|
|
|
|
|
| |
Change:
- Address missing no_log change from #73489.
Tickets:
- Refs #73489
Signed-off-by: Rick Elrod <rick@elrod.me>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change:
- A number of modules were missing no_log=True where they should have
had it.
Test Plan:
- Lots of grepping.
Tickets:
- Refs https://github.com/ansible-collections/community.general/pull/1725
Signed-off-by: Rick Elrod <rick@elrod.me>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Improve setup_rpm_repo
- add handlers to remove repos
- add variable to control whethere or not repos are created
* Use local repo for all distros
* Change repo creation script to module.
(cherry picked from commit 997b2d2a1955ccb4e70f805c18dc3e227e86c678)
Co-authored-by: Sam Doran <sdoran@redhat.com>
|
|
|
| |
Co-authored-by: Andrew Klychkov <andrew.klychkov@gmail.com>
|
|
|
|
|
| |
(cherry picked from commit fc590aeb2104c2c4e6a3aacba53852da1d7a26d9)
Co-authored-by: Matt Clay <matt@mystile.com>
|
|
|
|
|
|
|
|
| |
* [stable-2.9] Update ansible-test pylint Python support. (#72972)
* Update ansible-test pylint Python support.
* Python 3.8 is now officially supported.
(cherry picked from commit 37d09f24882c1f03be9900e610d53587cfa6bbd6)
|
|
|
|
|
|
| |
(cherry picked from commit 484e4af4d0b0f8ba88f73c20f592bb107eb9396a)
Co-authored-by: Matt Clay <matt@mystile.com>
|
| |
|
| |
|
|
|
|
|
| |
This reverts commit 6ae2146d023a74b28d3d85635f4a18d308c0a19c.
Co-authored-by: Alicia Cozine <acozine@users.noreply.github.com>
|
|
|
|
|
|
|
| |
* update the docs requirements versions (#73108)
(cherry picked from commit 093662390150491a81e8762d24d96a4258b0aa0d)
* remove antsibull
|
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
combine_vars uses dict.update() to replace keys
(cherry picked from commit 5e03e322de5b43b69c8aad5c0cb92e82ce0f3d17)
* Add tests for merging and replacing vars from inventory sources (#73181)
(cherry picked from commit 9de2da8a7ee1c0219b804b6afc7b090101106743)
|
|
|
|
|
|
|
|
|
|
| |
* Use is_sequence, and Mapping throughout, add support for tuples. Fixes #65722
* Address tests
* Remove unused import
* Add changelog
* Add docstring for clarity
(cherry picked from commit f8654de851a841d33d2d2e76942c68a758535e56)
Co-authored-by: Matt Martz <matt@sivel.net>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
together (#73016) (#73178)
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..
(cherry picked from commit 264e08f21a15213a4db76339888d3dfa2f2d6abb)
Co-authored-by: Sam Doran <sdoran@redhat.com>
|
|
|
|
|
|
| |
Support for using python-kubernetes v12 has been added to the
new upstream repo:
https://github.com/ansible-collections/community.kubernetes/blob/main/plugins/module_utils/common.py#L256-L261
This PR backports that support to ansible 2.9
|
| |
|
|
|
|
| |
* fix iteration error with _transform_name_attribute function
* add information on BIG-IQ 7.x support
|
|
|
|
|
| |
* Backport of https://github.com/ansible-collections/community.docker/pull/47 to stable-2.9.
* Also fix old-options.
|
|
|
|
| |
https://github.com/ansible-collections/community.docker/commit/5eef093e994ce4bb10529b1c5da38c4992f5b3c4 (#73067)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(#73014)
* 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
* Remove 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
(cherry picked from commit 48803604cd3f8640ed8dd36357044a3fd51b6490)
|
|
|
|
|
|
|
|
| |
Having the trap exit with a specific code will override
the exit code that caused the trap to run, which could
mask errors
(cherry picked from commit 5157a92139)
Co-authored-by: Sam Doran <sdoran@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
* Ensure we only reset the connection when one has been previously established. Fixes #65812
* Ensure psrp doesn't trace
* winrm too
(cherry picked from commit a3b6485073a0bbd2574285b4087d10dcfd80d494)
Co-authored-by: Matt Martz <matt@sivel.net>
|
|
|
|
|
|
|
|
|
| |
* Fix async interpreter parsing. Fixes #70690
* Target localhost instead of remote host
* Don't forget inventory
* Address shellcheck issue
(cherry picked from commit 83764ad)
Co-authored-by: Matt Martz <matt@sivel.net>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* ansible-test - prefer venv over virtualenv on Python 3 (#73000)
Also pin virtualenv to 16.7.10 for older Mac OS X systems. This was the version being installed
anway with the previous constraint (<20).
On systems with Python 3, now prefer venv over virtualenv. Test to see if venv is functional since
some systems have a non-functional venv installation (such as Debian).
(cherry picked from commit 850a77f6391cc09b7296e831877adcf4052f5404)
(cherry picked from commit a48b3d2089807ff0ab34ddacfcc90de49903d10e)
* Make the new ansible-test venv behavior opt-in
Co-authored-by: Sam Doooran <sdoran@redhat.com>
|
|
|
| |
Backport of https://github.com/ansible-collections/community.docker/commit/117f1322138c9910add8f43adb2f08e7ee5998d8 (#73066)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change:
- module_utils.basic.is_special_selinux_path() used a string ==
bytestring comparison which returned False and made Ansible think that
certain filesystems aren't, in fact, special-cased, when they should
be. Ensure both sides of the == are bytestrings.
Test Plan:
- Added `copy` integration tests for this case.
Tickets:
- Fixes #70244
Signed-off-by: Rick Elrod <rick@elrod.me>.
(cherry picked from commit 688cd8657bcd4610aa268913be67e76778bc6940)
Co-authored-by: Rick Elrod <rick@elrod.me>
|
|
|
|
|
|
| |
(cherry picked from commit 0eb1e22cd9)
Co-authored-by: Sam Doran <sdoran@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
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
(cherry picked from commit 003a9e890d)
Co-authored-by: Sam Doooran <sdoran@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Add `--venv-system-site-packages` to ansible-test.
(cherry picked from commit 934d6b235bd04a8aef570f4e4a5d6e0abe550318)
Backport of https://github.com/ansible/ansible/pull/65615
* [stable-2.9] Add coverage exporting to ansible-test
A new `--export` option for `ansible-test coverage combine` allows multi-step aggregation of code coverage for CI pipelines..
(cherry picked from commit fa2be89cd44f0c867f24351c3ba73d5e849cb507)
Backport of https://github.com/ansible/ansible/pull/72563
|
|
|
|
|
|
|
|
| |
* Increase fetch depth to reduce the frequency of full test runs for out-of-date PRs.
* Update scripts to match those which will be used in collections.
(cherry picked from commit 3274e02fd87b93975d409ec47661b8011b1a7fbf)
Co-authored-by: Matt Clay <mclay@redhat.com>
|
| |
|