Commit Graph

9494 Commits (f9bc6566429fea76f6ef511dc9e35472d2bfdbd9)

Author SHA1 Message Date
Matt Davis 84aa943195
Remove ansible-galaxy login (#72288) (#72325)
* GitHub is removing the underlying API used to implement the `login` command. Since the general consensus seems to be that relatively nobody currently uses this command (in favor of explicit token passing), support was simply removed for interactive login. If a future need arises, this command should be reimplemented via OAuth Device Auth Grants.
* login or role login commands now produce a fatal error with a descriptive message
* updated 2.10 porting guide entry

* remove dead code/config, update messages and porting guides

(cherry picked from commit 83909bfa22)
5 years ago
Felix Fontein 7fe2a22f70 ansible-test ansible-doc sanity test: disable vars plugins for collections 5 years ago
Andrew Klychkov e7b2ba4ef6
postgresql_pg_hba: fix a crash when a new rule with an 'options' field replaces a rule without or vice versa (#72290) 5 years ago
Rick Elrod 2beeeb8a27 show installations/removals in check_mode (#70892)
Change:
- Previously, we only showed that something would have changed, not what
  would have changed. This allows us to show what will chang as well.

Test Plan:
- Local RHEL8 VM
- New integration tests

Tickets:
- Fixes #66132

Signed-off-by: Rick Elrod <rick@elrod.me>
5 years ago
Rick Elrod 49d5462f16 accumulate update filters (#71726)
Change:
- Previously when `security: true` and `bugfix: true` were both given,
  only security updates would get applied. Filters now accumulate so
  that both get applied in this case.

Test Plan:
- New integration tests for both check_mode and not. These tests make
  use of a contrived yum repository which is stored in S3.

Tickets:
- Fixes #70854

Signed-off-by: Rick Elrod <rick@elrod.me>
Co-authored-by: Matt Martz <matt@sivel.net>

Co-authored-by: Matt Martz <matt@sivel.net>
5 years ago
Abhijeet Kasurde 4ab20bdd76
[2.9] nmcli: Typecast parameter values to string as required (#72094)
Fixes: https://github.com/ansible-collections/community.general/issues/1023

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
5 years ago
Ruediger Pluem 3d17393f61
user - allow local users with an expiry date to be created (#72022) (#72086)
The luseradd / lusermod commands do not support the -e option. Set
the expiry time in this case via lchage after the user was
created / modified.

Fixes: #71942

In Python3 math.floor returns an integer whereas Python2 returns a float.
Hence always convert the result of math.floor to an int to ensure that
lexpires is an integer.

Move local expires tests in a separate file and import the tasks to the
main.yml to keep main.yml smaller.

(cherry picked from commit a7170da851)
5 years ago
Sam Doran 0cab929d2d
[stable-2.9] Allow changed/failed mgmt on strategy actions (#70919) (#72118)
* Allow changed/failed mgmt on strategy actions.
(cherry picked from commit f9c3c6cba6)

Co-authored-by: Brian Coca <bcoca@users.noreply.github.com>
5 years ago
Christian Loos 4a1555fb0a
fix distribution fact for SLES4SAP (#71559) (#72029)
b6b238a fixed the SLES4SAP detection, which was at this time ok.
Sadly Suse changed with SLES 15 the /etc/os-release file, so the above
change will no longer work.

This commit updates the SLES4SAP detection regarding
https://www.suse.com/support/kb/doc/?id=000019341.

The symlink realpath is matched with endswith, because in SLES 12+ the
link target is SLES_SAP.prod, but in SLES 11 the link target is
SUSE_SLES_SAP.prod.

(cherry picked from commit ea119d3089)
5 years ago
Andrew Klychkov 30e735c4d8
[2.9] postgresql_privs: fix the module mistakes a procedure for a function (#72014)
* postgresql_privs: fix the module mistakes a procedure for a function

* Fix CI
5 years ago
Matt Clay 8fc579061f [stable-2.9] [stable-2.10] Fix ansible-test handling of user-defined docker networks. (#72256)
* Fix ansible-test docker container detection.

* Attach test containers to the correct network.

* Do not assume `localhost` for accesing Docker.

* Look for containers on current network.

* Always map /var/run/docker.sock into containers.

This fixes issues when using a remote Docker host.

* Support container IP lookup from networks list.

* Fix container network attachment.

* Remove redundant container detection messages.

* Limit DOCKER_HOST parsing to TCP.

* Restore docker socket existence check.

The check is skipped if the docker hostname is not localhost.

* Correct changelog entry..
(cherry picked from commit 3c2e8b99be)

Co-authored-by: Matt Clay <mclay@redhat.com>.
(cherry picked from commit 6362232c30)

Co-authored-by: Matt Clay <mclay@redhat.com>
5 years ago
Matt Clay a02d29ea09
CI stability updates. (#72306)
* Avoid issues with user-defined docker networks.
* Increase CI timeout.
5 years ago
Matt Clay c54dc50bd7
[stable-2.9] Fix ansible-test Azure Pipelines container auth. (#72283)
(cherry picked from commit 2ef4b7e07e)

Co-authored-by: Matt Clay <matt@mystile.com>
5 years ago
Matt Clay b9fc07ef4e
[stable-2.9] Support collection constraints in ansible-test. (#72158)
This allows collections to specify requirements and constraints for packages that ansible-test has requirements or constraints for..
(cherry picked from commit 5f76bd2af7)

Co-authored-by: Matt Clay <matt@mystile.com>
5 years ago
Rick Elrod 609667c7d4
[tests] --no-use-pep517 in kubevirt_conformance (#72015)
Change:
- This enables the inventory_kubevirt_conformance test to pass again on
  freebsd.
- This was due to a google-auth version bump. The dep chain looks like
  this: openshift -> kubernetes -> google-auth -> aiohttp -> multidict

Test Plan:
- ansible-test integration inventory_kubevirt_conformance --remote
  freebsd/12.0

Signed-off-by: Rick Elrod <rick@elrod.me>
5 years ago
Matt Clay 0dba2fe0dc
[stable-2.9] CI provider fixes for ansible-test. (#71929) (#71933)
* Make Azure Pipelines resource_prefix lowercase.

* Make classification of CI files consistent.

* Update package-data sanity test for AZP.
(cherry picked from commit 92b66e3e31)

Co-authored-by: Matt Clay <mclay@redhat.com>
5 years ago
Jordan Borean 7e8ecb2e58
psrp - fix hang when copying an empty file (#71649) (#71650)
(cherry picked from commit b615789fcc)
5 years ago
Rick Elrod 9d54f13ec1 Prevent templating unused variables for {%include%} (#68749)
Fixes #68699

(cherry picked from commit ff1ba39c8a)
5 years ago
Martin Krizek a63baa3290
Provide more information in AnsibleUndefinedVariable (#71666) (#71877)
Fixes #55152

(cherry picked from commit 00b22ab55e)
5 years ago
Rick Elrod a2548ccb6c [tests] Make setup_docker clean up better
Change:
- pip packages should get removed after, not try to add them again
- Try removing containerd.io package too
- Backport of #71949

Test Plan:
- CI
- ci_complete

Signed-off-by: Rick Elrod <rick@elrod.me>
5 years ago
Rick Elrod ace7f247f2 [docker_login] Fix changed status for newer docker
Change:
- Newer docker versions report the same message whether or not a logout
  actually happened.
- Determine change status from looking at the config instead if we can.
- This also allows us to restore check_mode in logout and re-enable that
  test.

Test Plan:
- CI, re-enabled tests

Tickets:
- Refs 6248f2fb6f

Signed-off-by: Rick Elrod <rick@elrod.me>
5 years ago
Rick Elrod 39fae9320c [incidental_setup_docker] changes to upstream repo (#71897)
Change:
- The docker-ce.repo file for centos does not work on RHEL since it uses
  $releasever and on RHEL that is, e.g., "7Server".
- Instead, set up the repo manually.
- Additionally, the docker centos8 repo no longer has old versions, so
  we use the (only) version in the repo instead.

Test Plan:
- CI

Signed-off-by: Rick Elrod <rick@elrod.me>
(cherry picked from commit 31ddca4c0d)
5 years ago
Matt Clay 61d383ef23 [stable-2.9] Fix attribute testing in file integration test. (#71843)
The file test will no longer attempt to test attributes if `lsattr -vd` does not work on the system under test.
(cherry picked from commit 17765cd4e8)

Co-authored-by: Matt Clay <mclay@redhat.com>
5 years ago
Sam Doran 582cf06501
[stable-2.9] openssl_privatekey test - Add pause for macOS (#71819)
The stat time granularity on macOS is one second. We recently upgrade
to faster macOS hosts, so some tests that run closely together to
see if something changed will have the same timestamp intermittently.
5 years ago
Sam Doran 3e26c56edd
[stable-2.9] incidental_setup_openssl - Pin version of Python packages (#71777)
A recent update to cffi that was yanked is still being installed on our
Mac OS X 10.11 test image since the version of pip there is very old and
does not ignore yanked packages.

Pin the version of pyOpenSSL and its dependencies to fix this and avoid
future spontaneous failures.
(cherry picked from commit 65cdb86c8a)

Co-authored-by: Sam Doran <sdoran@redhat.com>
5 years ago
Matt Clay 417e408f59
[stable-2.9] Backport ansible-test CI provider support. (#71614)
* Add encoding.py from devel to support backports.

* Add io.py from devel to support backports.

* Update ansible-test support for CI providers. (#69522)

Refactored CI provider code to simplify multiple provider support and addition of new providers.

(cherry picked from commit d8e0aadc0d)

* Add Shippable request signing to ansible-test. (#69526)

(cherry picked from commit e7c2eb519b)

* ansible-test local change detection: use --base-branch if specified (#69508)

(cherry picked from commit 43acd61901)

* Add Azure Pipelines support to ansible-test.

(cherry picked from commit 8ffaed00f8)

* Update ansible-test remote endpoint handling. (#71413)

* Request ansible-core-ci resources by provider.
* Remove obsolete us-east-2 CI endpoint.
* Add new --remote-endpoint option.
* Add warning for --remote-aws-region option.
* Update service endpoints.
* Allow non-standard remote stages.
* Add changelog fragment.

(cherry picked from commit d099591964)

* Fix ansible-test coverage traceback. (#71446)

* Add integration test for ansible-test coverage.

* Fix ansible-test coverage traceback.

* Fix coverage reporting on Python 2.6.

(cherry picked from commit f5b6df14ab)

* Use new endpoint for Parallels based instances.

(cherry picked from commit 98febab975)

* Add pause to avoid same mtime in test.

(cherry picked from commit 3d769f3a76)

Co-authored-by: Felix Fontein <felix@fontein.de>
5 years ago
Rick Elrod 15a77b4575
[tests] fix fallout from dnf gpg fix (#71551)
Change:
- Missed needed gpg ignores

Test Plan:
- CI
- ci_complete

Signed-off-by: Rick Elrod <rick@elrod.me>
5 years ago
Sam Doran dfaadf6f33
Fix inventory_kubevirt_conformance test (#71546) 5 years ago
Rick Elrod 1d043e082b
[dnf] ensure packages are gpg-verified (#71540)
Change:
- By default the dnf API does not gpg-verify packages. This is a feature
  that is executed in its CLI code. It never made it into Ansible's
  usage of the API, so packages were previously not verified.
- This fixes CVE-2020-14365.

Test Plan:
- New integration tests

Signed-off-by: Rick Elrod <rick@elrod.me>
5 years ago
Jordan Borean 8e3b754710
powershell - fix nested CLIXML parser (#71412) - 2.9 (#71450)
* powershell - fix nested CLIXML parser (#71412)

(cherry picked from commit 8897d7e2ff)

* Remove test that shouldn't have been backported
6 years ago
Jordan Borean fe72da80a4
powershell - fix quoting values (#71411) (#71448)
* powershell - fix quoting values

* Add ignore for smart quote skip

(cherry picked from commit 72a7cb4a2c)
6 years ago
GomathiselviS cb79d9c80d
Update network integration auth timeout (#71057) (#71239)
* Update network integration auth timeout

* added changelog

Co-authored-by: Ganesh Nalawade <ganesh634@gmail.com>
6 years ago
David Shrewsbury f563365f8e
Fix play stats when rescue block is a child block (#70922) (#71335)
* check run state of current block only

* Add changelog and test

* Add test for issue 29047

(cherry picked from commit f2f6c34632)
6 years ago
Rhys b6c7598a20
[2.9] Fix issue with MongoDB 4.0.20 (#71393)
Co-authored-by: Matt Davis <nitzmahone@users.noreply.github.com>
6 years ago
Dusan Matejka aa698ba9b6
Zabbix: Handle KeyError in zabbix_host module (#65392) (#71288)
Fixes: #65304
(cherry picked from commit 7b2cfdacd0)

Co-authored-by: sky-joker <sky.jokerxx@gmail.com>
6 years ago
Ganesh Nalawade 67fab726b9
Fix commit timeout failure issue for netconf modules (#71291)
Fixes https://github.com/ansible-collections/cisco.iosxr/issues/74

*  ncclient API expects commit timeout value in either unicode
   or bytes format, hence convert the timeout value explicitly
   to string type.
6 years ago
Lorenzo Castelli 4a5aac0ac1
systemd - supports new systemctl output message for chroot (#71197) (#71329)
The message generated by systemctl has been updated in 9321e23c40, which requires a corresponding change in the systemd module.

In addition, this fixes the module when the SYSTEMD_OFFLINE environment variable is set.

(cherry picked from commit a1a50bb3cd)
6 years ago
Martin Krizek 15f3426971
_check_failed_state: always use the current/nested state (#71347) (#71456)
Fixes #71306

(cherry picked from commit 9792d631b1)
6 years ago
Rick Elrod 6b4977e3f4
Backport cron module regression fix (#71244)
Change:
- Backport of #71207

Test Plan:
- CI

Tickets:
- Fixes #71213
- Fixes #71217
- Fixes #71216
- Fixes #71226
- Refs #71207

Signed-off-by: Rick Elrod <rick@elrod.me>
6 years ago
Rick Elrod 7d61e47a0e
[stable-2.9] tests: Use `hg serve` instead of bitbucket for hg (#71398)
* tests: Use `hg serve` instead of bitbucket for hg

Change:
- Uses `hg serve` instead of a bitbucket repo for hg tests
- bitbucket no longer serves hg

Test Plan:
- CI, fixed integration tests

Signed-off-by: Rick Elrod <rick@elrod.me>
6 years ago
Rick Elrod 7779137b71
[stable-2.9] disable mongodb_* tests, fix setup_mongodb (#71388)
* Disable mongodb_replicaset tests for now

Change:
- 4.0.20 breaks tests, disable for now

Test Plan:
- CI

Tickets:
- https://github.com/ansible-collections/community.mongodb/issues/136

Signed-off-by: Rick Elrod <rick@elrod.me>

* setup_mongodb: Nix RH package installation/removal

Change:
- Our RHEL and CentOS images make these unnecessary and they were
  broken.

Test Plan:
- CI

Signed-off-by: Rick Elrod <rick@elrod.me>

* Disable mongodb_shard tests for now

Change:
- 4.0.20 breaks tests, disable for now

Test Plan:
- CI

Tickets:
- https://github.com/ansible-collections/community.mongodb/issues/136

Signed-off-by: Rick Elrod <rick@elrod.me>
6 years ago
Matt Davis 6123266c82 [stable-2.9] fix collection package root location under pytest >=6.0.0 (#70963).
(cherry picked from commit 86b6c4bbb6)

Co-authored-by: nitzmahone <nitzmahone@users.noreply.github.com>
6 years ago
Sam Doran ea8c219402
[stable-2.9] Revert "Change default file permissions so they are not world readable (#70221) (#70825)" (#71232)
This reverts commit 0199b1cf05.
6 years ago
Jordan Borean a57e6172f7
Ensure -k is set to delegated hosts without a pass (#71136) - 2.9 (#71167)
* Ensure -k is set to delegated hosts without a pass (#71136)

* Ensure -k is set to delegated hosts without a pass

* Fix up some broken tests

* Update task_executor.py

one possible fix, the other is updating winrm to normalize on 'password' like the other connection plugins

* Add alias for winrm and fix incorrect assumption

* Make sure aliases are used for keyword options

* Conditionally run test if sshpass is present, fix sanity

Co-authored-by: Brian Coca <bcoca@users.noreply.github.com>
(cherry picked from commit 3f22f79e73)

* Backport subset of https://github.com/ansible/ansible/pull/69670
6 years ago
Felix Fontein 02be2a13db
ansible-test: bump acme test container version to 2.0.0 (#71097) (#71166)
(cherry picked from commit 050841324c)
6 years ago
Rick Elrod 9cc782eebc
[tests] pin docker to <4.3.0 (#71192)
Change:
- New python docker lib wants a newer docker than we have in CI.

Test Plan:
- CI

Signed-off-by: Rick Elrod <rick@elrod.me>
6 years ago
Brian Coca 4a6a9acea8
dont clobber facts in loop (#71032) (#71096)
* dont clobber facts in loop

	fixes #71031

(cherry picked from commit f9af27c631)
6 years ago
Brian Coca 6e8adc0526
reset logging to INFO (#70878) (#70882)
* reset logging to INFO (#70878)

- due to CVE-2019-14846
 - also added comments and test to avoid 'oportunistic' reversion

(cherry picked from commit 1223ce656a)

* Update keep_log_at_info.yml

Co-authored-by: Rick Elrod <rick@elrod.me>
6 years ago
Sloane Hertel 44cca5426f
[2.9] template connection variables accessed directly before using (#70657) (#70689)
* template connection variables accessed directly before using (#70657)

* template variables accessed directly when using them instead of FieldAttributes

(cherry picked from commit 8c213c9334)

* changelog
6 years ago
Abhijeet Kasurde aa58d8c9ed
[2.9] hashi_vault: Handle equal sign in secret name value (#70169)
Fixes: ansible/ansible#55658

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
6 years ago