summaryrefslogtreecommitdiffstats
path: root/test/integration (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Update triple single quotes to triple double quotes (#84099)Matt Clay2024-10-09124-471/+471
| | | | | | | | | * Update triple single quotes to triple double quotes This change was fully automated. The updated Python files have been verified to tokenize the same as the originals, except for the expected change in quoting of strings, which were verified through literal_eval. * Manual conversion of docstring quotes
* user module, avoid chmoding symlink'd home file (#83956)Brian Coca2024-10-082-1/+69
| | | | | | | also added tests --------- Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua> Co-authored-by: Sloane Hertel <19572925+s-hertel@users.noreply.github.com>
* Implicit flush_handlers inherit play tags (#83968)Martin Krizek2024-10-042-0/+14
|
* Reduce number of implicit meta tasks (#84007)Martin Krizek2024-10-034-4/+44
| | | | | | | | | | | | | | | | | This greatly reduces run time on large inventories since meta tasks are executed in the main process sequentially and just executing them is expensive. This change avoids running the following implicit meta tasks: * ``flush_handlers`` on hosts where no handlers are notified * ``noop`` for the linear strategy's lockstep, instead hosts that are not executing the current task are just not part of the current host loop A playbook consiting of two simple plays both running on ~6000 hosts runs in: devel: 37s this PR: 1.3s Co-authored-by: Sloane Hertel <19572925+s-hertel@users.noreply.github.com> Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua>
* get_url: properly parse filename in content-disposition (#83748)Martin Krizek2024-10-031-0/+20
| | | | | | | | | | Since we don't really care about the type we don't have to query for it and just retrieve the filename value. Unfortunately we cannot use module_utils.urls.get_response_filename as we don't have the response object, so just utilize email.message.Message to parse the filename instead of manually doing the work ourselves. Fixes: #83690
* debconf: set empty password value (#83217)Abhijeet Kasurde2024-10-011-28/+71
| | | | | | Fixes: #83214 --------- Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
* Prevent condor from being installed and fulfilling libfmt dependency (#84023)Matt Martz2024-09-301-1/+1
|
* Test entry points with editable install (#84002)Matt Clay2024-09-241-17/+32
| | | This allows collection of code coverage for the entry points.
* Ansible Errors, Don't hide stacked messages when yaml (#83933)Brian Coca2024-09-241-1/+3
| | | | | | Also remove redundant msg now that we fixed yaml case So no more need to %s % e. Co-authored-by: Abhijeet Kasurde <akasurde@redhat.com>
* Add basic validation for action_groups (#83965)Felix Fontein2024-09-244-0/+31
| | | Co-authored-by: Sloane Hertel <19572925+s-hertel@users.noreply.github.com>
* dnf5: re-introduce ``state: installed`` alias (#83961)Martin Krizek2024-09-231-1/+1
| | | Fixes #83960
* Do not convert floats to ints when there is truncation (#83864)Brian Coca2024-09-203-81/+1
| | | | | | | Adjusted error messages fixed tests removed py2 compat tests, since no more py2 Co-authored-by: Matt Clay <matt@mystile.com>
* Add mount_facts module (#83508)Sloane Hertel2024-09-193-0/+200
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add a mount_facts module capable of gathering mounts skipped by default fact gathering * By default, collect mount facts from standard locations including /etc/mtab, /proc/mounts, /etc/fstab, /etc/mnttab, /etc/vfstab, and on AIX, /etc/filesystems. When no file-based source for the current mounts can be found (like /proc/mounts), the module falls back to using mount as a source. This allows BSD and AIX to collect the existing mounts by default, without causing Linux hosts to use both /proc/mounts and mount output. * Non-standard locations and "mount" can be configured as a sources. * Support returning an aggregate list of mount points in addition to first found. When there are multiple mounts for the same mount point in an individual source, a warning is given if the include_aggregate_mounts option is not configured. * Add options to filter on fstypes and devices (supporting UNIX shell wildcards). * Support configuring a timeout and timeout behavior to make it easier to use the module as a default facts module without risking a hang. * Include the source and line(s) corresponding to a mount for easier debugging. Co-authored-by: Brian Coca <bcoca@users.noreply.github.com> Co-authored-by: Matt Clay <matt@mystile.com> Co-authored-by: Matt Davis <6775756+nitzmahone@users.noreply.github.com>
* timedout extended (#83953)Brian Coca2024-09-181-0/+15
| | | | | * timedout extended * add timedout test
* fact gathering, mounts, fixes for single proc code and tests (#83866)Brian Coca2024-09-104-32/+0
| | | | | | | | | * Fact gathering fix 'no shm' branhc * Use concurrent.futures instead of multiprocessing This entirely avoids the need for fallback logic since the concurrent.futures thread pool does not depend on `/dev/shm`. Co-authored-by: Matt Clay <matt@mystile.com>
* Replace binary_modules Makefile with Python script (#83925)Matt Clay2024-09-102-18/+41
| | | | | | Also update the platform list: * Remove linux ppc64le * Add darwin arm64
* loop_control "early exit" feature (#62151)Brian Coca2024-09-052-0/+19
| | | | | | | | | | | | | | | | | * add a loop_control break_when directive to break out of a loop after any item * remove loop var as normal exit would * example usage: - name: generate a random password up to 10 times, until it matches the policy set_fact: password: "{{ lookup('password', '/dev/null', chars=character_set, length=length) }}" loop: "{{ range(0, 10) }}" loop_control: break_when: - password is match(password_policy) Co-authored-by: s-hertel <19572925+s-hertel@users.noreply.github.com>
* Add location on include_tasks fail inside include (#83876)Jordan Borean2024-09-013-0/+18
| | | | | | Adds the datastore details to the parser error when attempting to include tasks that contain include_tasks without a filename set. This change will now display the exact location of the include_tasks that failed like any normal syntax error.
* Improve testing for Windows SSH and other connection plugins (#83834)Jordan Borean2024-08-286-15/+61
| | | | | | | | | Expands the test matrix used for testing on Windows to cover the three connection plugins we support for all the tasks. This change also changes how raw commands are run over SSH to avoid starting a `powershell.exe` process that was uneeded in the majority of cases used in Ansible. This simplifies our code a bit more by removing extra Windows specific actions in the ssh plugin and improves the efficiency when running tasks.
* Print the name of the option being deprecated (#83761)Martin Krizek2024-08-271-2/+2
| | | Fixes #83759
* fix delegate_to integration test (#83865)Matt Davis2024-08-271-9/+28
| | | * the test was previously passing erroneously due to the `timeout` elapsing in CI, and that the `failed` test does not encompass `unreachable`
* copy, fix permissions and atime on diff partitions (#83824)Brian Coca2024-08-261-0/+42
| | | we just set time also, when on diff partitions
* runtime-metadata sanity test: do not fail deprecation version checks if ↵Felix Fontein2024-08-269-1/+62
| | | | | | | galaxy.yml has empty `version` (#83831) * Do not create invalid SemanticVersion objects. * Fix SemanticVersion.parse(). * Add basic runtime-metadata tests.
* powershell - Improve CLIXML parsing (#83847)Jordan Borean2024-08-231-0/+9
| | | | Improves the logic used when parsing CLIXML to support all escaped character sequences and not just newlines.
* Fix up raw_params for ansible.windows modules (#83830)Jordan Borean2024-08-201-0/+20
| | | | | | | | | | | * Fix up raw_params for ansible.windows modules Fixes up the logic for detecting if using ansible.windows.win_command or ansible.windows.win_shell with _raw_params. These two modules are special in that they can be referenced in 4 different ways but the ansible.windows collection specific prefix needs to be manually added to the list. * Fix up sanity issue
* Gather mount facts, fallback for when multiproc is not feasable (#83750)Brian Coca2024-08-194-0/+33
| | | | | | * fallback to 'single threaded gathering' for when multiproc fails Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua>
* ssh and psrp - Support more complex characters in fetch_file (#83753)Jordan Borean2024-08-182-5/+25
| | | | | | | | | | | | | | * ssh and psrp - Support more complex chars in fetch_file Fixes the psrp and ssh (with piped) fetch function to work with paths that contains glob like characters in the path. For Windows this was needed when using paths that contain `[]` in the path. For ssh this was a problem with FreeBSD when using the piped transfer method with similar characters. Also tidies up the psrp logic to not inject the paths and buffer size in the script but pass it as an object through an argument/parameter. * Fix sanity check
* Add explicit winrm/psrp tests for HTTP and HTTPS (#83769)Jordan Borean2024-08-162-0/+27
|
* Fix tests when running against SSH targetJordan Borean2024-08-152-1/+10
|
* ansible-test - Add Windows remote connection optionMatt Clay2024-08-152-2/+2
|
* Use a venv in more integration tests (#83799)Matt Clay2024-08-157-9/+11
| | | | | * Use venv for pause test * Use venv for debugger test * Use venv for builtin_vars_prompt test
* fix copy module update atime/mtime (#83235)dkuji2024-08-141-0/+42
| | | | | | Ensure we force mtime/atime update when using copystat Co-authored-by: Sloane Hertel <19572925+s-hertel@users.noreply.github.com> Co-authored-by: Brian Coca <bcoca@users.noreply.github.com>
* Add end_role meta task (#83263)Martin Krizek2024-08-1310-0/+100
| | | ci_complete
* Fix ansible-vault integration test for missing vault ids (#83777)Sloane Hertel2024-08-131-11/+12
| | | | | | | * Fix broken, circumvented test for missing vault ids * verify the command returns a non-zero exit code Co-authored-by: Matt Clay <matt@mystile.com>
* Ensure skipped loop iteration register var is available (#83756)Martin Krizek2024-08-131-0/+17
| | | Fixes #83619
* Fix meta tasks breaking host/fork affinity with host_pinned (#83438)Martin Krizek2024-08-135-0/+114
| | | | Fixes #83294
* ini lookup: add new interpolation option (#83773)Martin Krizek2024-08-134-0/+14
| | | Fixes #83755
* atomic_move - fix creating file in directory with setgid bit (#83718)Sloane Hertel2024-08-123-0/+41
| | | | | | | * fix creating file in directory with setgid bit * add a test using the copy module's content option to create a file in a directory with setgid bit Co-authored-by: Martin Krizek <martin.krizek@gmail.com>
* Enable validation of subkeys in rpm key module (#83716)Kellin2024-08-121-1/+62
| | | | | | | | | | | | | | | | | | * Enable validation of subkeys in rpm key module A gpg subkey may change while the primary key remains the same. Due to this behavior, there are situations where validation of the primary gpg key fingerprint is not sufficient because the desired target is actually the gpg subkey. This change allows the user to validate against either the fingerprint of the primary gpg key or its subkey. Signed-off-by: Kellin <kellin@retromud.org> * Improve tests, add multi-fingerprint - Improve tests to cover all cases - add multi fingerprint validation Signed-off-by: Kellin <kellin@retromud.org>
* use diff intermediate var to preserve functionality (#83738)Brian Coca2024-08-082-0/+28
| | | | add tests
* Add vaulted_file test (#83717)Brian Coca2024-08-084-0/+20
| | | | | | | * Add vaulted_file test * fix is_encrypted_file while we are here Co-authored-by: Martin Krizek <martin.krizek@gmail.com>
* Fix csvfile test - quote file argument (#83751)Sloane Hertel2024-08-081-1/+1
| | | file was intended to be a string, not an undefined variable
* try disabling negative values win_reboot test (#83735)Matt Davis2024-08-071-4/+5
| | | * hoping to improve CI stability
* Fix task-adjacent search path in roles (#83621)Sloane Hertel2024-08-064-0/+12
| | | | | * Restore search path in the current task file’s directory for roles
* csvfile lookup - fix giving an error when no search term is provided (#83710)Sloane Hertel2024-08-061-0/+9
| | | Fixes #83689
* Fix display to log severity mapping (#83712)Brian Coca2024-08-062-3/+18
| | | | | | | | add caplevel to display to pass through also reverse dict order as 'last update wins' added tests ... and also log severity to log Co-authored-by: Sloane Hertel <19572925+s-hertel@users.noreply.github.com>
* Remove EXTERNALLY-MANAGED marker in apt test (#83706)Matt Clay2024-08-011-0/+13
| | | | | | | The marker is removed in ansible-test managed environments, but the apt test restores it by installing/upgrading packages. To avoid breaking later tests, the marker needs to be removed again. ci_complete
* ansible-doc: handle on_fail (#83676)Abhijeet Kasurde2024-08-011-0/+6
| | | | | Handle errors raised when role doc has errors Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
* Add UID and GID min/max keys (#81770)skupfer2024-08-014-0/+154
| | | Fixes: #72183
* add error handling when parsing values in ini files (#82718)Karl G2024-08-012-12/+16
| | | | | Fixes: #82717 Co-authored-by: Karl A. Grindley <kgrindley@ll.mit.edu>