summaryrefslogtreecommitdiff
path: root/test/integration/targets
Commit message (Collapse)AuthorAgeFilesLines
* config lookup, new option to show origin of value (#80651)Brian Coca2023-05-111-0/+2
|
* fix error if path had wildcard in it (#74723)PRASOON KARUNAN V2023-05-101-0/+15
| | | | | | | * fix error if path had wildcard in it * add test * add changelog fragment
* first_found fix no terms option (#76550)Brian Coca2023-05-081-0/+8
| | | | | also fix bug with multiple items clobbering previous settings Co-authored-by: Sloane Hertel <19572925+s-hertel@users.noreply.github.com>
* Account for overlays when interacting with Jinja envs (#80705)Martin Krizek2023-05-043-0/+24
| | | | | | | | Instead of using Templar.environment in Templar.do_template for accessing/mutating the environment, myenv local variable should be used because it is the environment used for actual templating. It can either point to Templar.environment or newly created environment overlay. Fixes #80605
* Use ansible.module_utils.common.text.converters (#80704)Matt Clay2023-05-036-6/+6
| | | Replace use of old `ansible.module_utils._text` and add a unit test to maintain backwards compatibility.
* Better parse virtualenv_command option for pip (#80624)Dmitriy Rabotyagov2023-05-021-0/+22
| | | | | | | | | | Atthe moment if a users wants to protect virtualenv_command by using quotes around 'venv', module will fail out as literal parsing is used at the moment. In order to imrpove things, argparse is leveraged to parse out passed value to the virtualenv_command Closes-Bug: #76372 Signed-off-by: Dmitriy Rabotyagov <noonedeadpunk@gmail.com>
* _symbolic_mode_to_octal- fix raising ValueError for invalid symbolic modes ↵Sloane Hertel2023-04-271-0/+23
| | | | | | | (#80449) validate the full user and perm strings instead of just first character fixes unhelpful unarchive error for some invalid modes
* Fix gather parallel (#80597)Brian Coca2023-04-265-0/+61
| | | | | | | | * fix meaning of parallel in gather_facts * Update docs with note about parallel not always being faster * add 'smarter' usage of gahter_timeout for parallel tasks * restore async when needed, not always * added typing * parallelism tests
* Fix calling v2_playbook_on_notify callback (#80585)Martin Krizek2023-04-248-1/+163
| | | Co-authored-by: Matt Martz <matt@sivel.net>
* ansible-test - Remove deprecated features (#80558)Matt Clay2023-04-184-18/+0
|
* argspec - fix validating type for required options that are None (#79677)Sloane Hertel2023-04-173-4/+136
| | | | | * Only bypass type validation for null parameters if the default is None. A default is mutually exclusive with required. * Prevent coercing None to str type. Fail the type check instead.
* ansible-galaxy - fix installing dir sources with a trailing path separator ↵Sloane Hertel2023-04-171-1/+1
| | | | | (#79110) * strip trailing path separator from type=dir sources
* ansible-test - Replace pytest-forked (#80525)Matt Clay2023-04-144-1/+94
| | | | | - Unit tests now report warnings generated during test runs. - Python 3.12 warnings about `os.fork` usage with threads (due to `pytest-xdist`) are suppressed. - Added integration tests to verify forked test behavior.
* feature: add support for symbolic links when passing "dir" (#80460)kloud-byun2023-04-141-0/+25
| | | * update the include_vars action plugin to always follow symbolic links when traversing directories
* Last handler defined runs, fix for roles (#79558)Martin Krizek2023-04-125-0/+32
| | | | | | | | | Fixes #73643 * clear_notification method and simplify ifs * Deduplicate code * Limit number of Templar creations * Fix sanity * Preserve handler callbacks order as they were notified
* Fix the parsing integration tests (#80454)Matt Clay2023-04-079-92/+37
| | | | Most of the bad_parsing tests were no longer running, with several of them no longer being valid. The invalid tests have been removed and the valid ones rewritten.
* Replace validate-modules's semantic markup parser with antsibull-docs-parser ↵Felix Fontein2023-04-052-4/+29
| | | | (#80406)
* Extend ansible-doc test with more classic markup. (#80407)Felix Fontein2023-04-053-2/+14
|
* Fetch signatures from galaxy after the dependency resolver runs (#80334)Sloane Hertel2023-04-041-1/+2
| | | | | | | Reduce the number of Galaxy API calls made during dependency resolution by fetching remote signatures afterwards, since these are not used in backtracking. Reduce the verbosity to `-vvvv` (to match other Galaxy API calls) to see this activity. Co-authored-by: Sviatoslav Sydorenko <webknjaz@redhat.com>
* Implement semantic markup support for Ansible documentation in ↵Felix Fontein2023-03-312-0/+117
| | | | validate-modules. (#80243)
* Fix run_once by instantly tiny post_validate (#78492) (#80051)tu1h2023-03-312-0/+22
| | | Signed-off-by: tu1h <lihai.tu@daocloud.io>
* Support role extension for semantic markup. (#80305)Felix Fontein2023-03-313-3/+7
|
* Add new dnf5 module (#80272)Martin Krizek2023-03-317-15/+63
|
* fix using templated values for include/import role FA (#80320)Sloane Hertel2023-03-304-0/+41
| | | | | | | | | | | * fix using templated values for include/import role options 'public', 'allow_duplicates', and 'rolespec_validate' * pass templated values without changing the instance * Fix templating by setting always_post_validate to True and calling IncludeRole.post_validate() instead ci_complete * add changelog
* ansible-test - Fix packaging change classification (#80360)Matt Clay2023-03-292-0/+2
|
* Use `skip/macos` instead of `skip/macos/13.2` (#80343)Matt Clay2023-03-281-1/+1
| | | The reason for the skip won't be going away with future versions of macOS.
* Remove obsolete integration test aliases (#80342)Matt Clay2023-03-2818-20/+0
| | | | | | | * Remove obsolete integration test aliases * Remove obsolete `skip/osx` aliases * Use `skip/macos` instead of `skip/osx` in docs
* New deb822_repository module (#80018)Matt Martz2023-03-286-0/+283
| | | Fixes #77073
* Fix URL processing. (#80295)Felix Fontein2023-03-274-18/+26
|
* Ansible.Basic - Improve temporary file cleanup process (#80293)Jordan Borean2023-03-281-0/+162
| | | | | * Ansible.Basic - Improve temporary file cleanup process * Add comment on struct value used
* ansible-galaxy - support `resolvelib >= 0.5.3, < 1.1.0` (#80196)Wong Hoi Sing Edison2023-03-271-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * ansible-galaxy - support `resolvelib >= 0.5.3, < 1.1.0` <https://pypi.org/project/resolvelib/1.0.1> released on 2023-03-09: - <https://github.com/sarugaku/resolvelib/blob/main/CHANGELOG.rst#101-2023-03-09> - <https://github.com/sarugaku/resolvelib/releases/tag/1.0.1> Signed-off-by: Wong Hoi Sing Edison <hswong3i@pantarei-design.com> * Trigger CI by pinning resolvelib with latest version Since resolvelib is pre-installed in our test containers, we should temporarily pin the latest version allowed to force the tests to run with that version. Once the tests have passed that commit can be reverted. Please make those changes without force pushing, so that we keep the reference to the passing CI run. We can squash the commits when merging the PR so the temporary commits won't be in the final commit merged to the devel branch. Signed-off-by: Wong Hoi Sing Edison <hswong3i@pantarei-design.com> * https://github.com/ansible/ansible/pull/80196#discussion_r1136003637 Also test resolvelib with multiple supported versions. Signed-off-by: Wong Hoi Sing Edison <hswong3i@pantarei-design.com> * Revert "Trigger CI by pinning resolvelib with latest version" This reverts commit 5518e5dbca01b87d13850cee9ccdbdc1ee49188f. --------- Signed-off-by: Wong Hoi Sing Edison <hswong3i@pantarei-design.com>
* Add support for plugin field in seealso. (#80212)Felix Fontein2023-03-234-0/+145
|
* Fix url lookup test to use test container (#80284)Matt Clay2023-03-231-2/+2
|
* Don't double loop, ensuring that machine formats don't dupe collections onto ↵Matt Martz2023-03-231-12/+27
| | | | | | | all paths (#80277) * Don't double loop, ensuring that machine formats don't dupe collections onto all paths * Add tests
* Do not double calculate loops and `delegate_to` (#80171)Matt Martz2023-03-233-0/+43
|
* ansible-doc: fix broken seealso links in text output (#80280)Felix Fontein2023-03-234-2/+61
| | | | | * Fix broken URLs. * Also remove auto-generated description for modules outside ansible.builtin.
* validate-modules: check_mode attribute and compare with supports_check_mode ↵Felix Fontein2023-03-239-1/+238
| | | | | parameter (#80090) Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>
* validate-modules: support plugin see-also (#80244)Felix Fontein2023-03-231-0/+3
|
* Implement semantic markup support for Ansible documentation in ansible-doc. ↵Felix Fontein2023-03-213-20/+27
| | | | (#80242)
* Make PEP 517 mutation tests use pinned old setuptools (#80262)Sviatoslav Sydorenko2023-03-201-4/+9
| | | | | | | | | | | * Pin setuptools to lowest supported @ PEP 517 test This allows catching the behavior of builds under old setuptools. * Stop invoking `setup.py install` in tests This is not the part we care about since it involves dealing with the external runtime dependencies rather than building our source distribution.
* 📦 Switch sdist build-system to pure setuptools (#80255)Sviatoslav Sydorenko2023-03-201-9/+124
| | | | | | | | | | | | | | | | | | | This patch modifies the in-tree build backend to build sdists that swap out pointers to it in the `pyproject.toml`'s `[build-system]` section. The effect of this is that the first build from source (for example, from a Git checkout) uses our PEP 517 in-tree build backend. But the produced tarball has `build-backend` set to `setuptools.build_meta` which is the native build backend of `setuptools`. So any following builds from that sdist will skip using the in-tree build backend, calling the setuptools' one. The good news is that if the first build generated the manpages, they will be included and won't go anywhere even though, a different build system is in place. Combined with #80253, this will make sure not to modify the current source checkout on that first build. Co-authored-by: Matt Clay <matt@mystile.com>
* copy - fix check mode with remote_src=True (#78624)Sloane Hertel2023-03-081-2/+39
| | | | | | | * Don't create dest directory in check mode uncomment existing test Fix checking for file attribute changes in check mode and add a test
* Revert "🧪 Switch macOS 13.2 to 12.0 in CI (#80145)" (#80156)Sviatoslav Sydorenko2023-03-071-1/+1
| | | This reverts commit 326e70c02a2563e13a68565c31ce42477ef4874f.
* paramiko connection now uses the correct source of data (#79704)Brian Coca2023-03-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * paramiko connection now uses the correct source of data * keep orig name * added missing port to docs * Update changelogs/fragments/paramiko_config.yml Co-authored-by: Sloane Hertel <19572925+s-hertel@users.noreply.github.com> * test ansible_host templating * added missing private_key_file * removed all play_context fallbacks * Update lib/ansible/plugins/connection/paramiko_ssh.py Co-authored-by: Jordan Borean <jborean93@gmail.com> * Update lib/ansible/plugins/connection/paramiko_ssh.py Co-authored-by: Jordan Borean <jborean93@gmail.com> --------- Co-authored-by: Sloane Hertel <19572925+s-hertel@users.noreply.github.com> Co-authored-by: Jordan Borean <jborean93@gmail.com>
* 🧪 Switch macOS 13.2 to 12.0 in CI (#80145)Sviatoslav Sydorenko2023-03-061-1/+1
| | | | | | | | * 🧪 Switch macOS 13.2 to 12.0 in CI The former revealed unexpected flakiness while the latter is the previous value that was used to be stable. This is a temporary revert. * Skip lookup_url integration test under macOS 12.0
* ansible-inventory prevent duplicating host entries (#80059)Brian Coca2023-03-066-20/+200
| | | | this happened after implementing limits we introduced a bug that a host would be duplicated if it existed in a group's children
* Add macOS 13 to CI and ansible-test (#79508)Sviatoslav Sydorenko2023-03-031-1/+8
| | | | | | | | | | | | | | | | | | * ✨ Add macOS 13.2 to `ansible-test` * 🧪 Replace macOS 12.0 with 13.2 in the CI matrix * Skip `lookup_url` under macOS 13.2 This is due to https://wefearchange.org/2018/11/forkmacos.rst.html that manifests itself as follows: TASK [lookup_url : Test that retrieving a url works] *************************** objc[15394]: +[__NSCFConstantString initialize] may have been in progress in another thread when fork() was called. objc[15394]: +[__NSCFConstantString initialize] may have been in progress in another thread when fork() was called. We cannot safely call it or ignore it in t he fork() child process. Crashing instead. Set a breakpoint on objc_initializeAfterForkError to debug. ERROR! A worker was found in a dead state * 📝 Extend ansible-test change note w/ macOS 13.2
* 🐛 Make integration tests compatible w/ modern Git (#80122)Sviatoslav Sydorenko2023-03-0315-48/+176
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 🐛 Make integration tests compatible w/ modern Git This patch makes use of the `init.defaultBranch` setting to unify the test across new and old Git versions since one defaults to `master` and the other uses `main` for the default branch. Where possible, it uses the `HEAD` committish to avoid having to normalize the branch name. The change fixes the following integration tests: * `ansible-galaxy` * `ansible-galaxy-collection-scm` (recursive collection) * `git` * 🐛Replace `git-symbolic-ref` with a repo template This custom Git repository template emulates the `init.defaultBranch` setting on Git versions below 2.28. Ref: https://superuser.com/a/1559582. Other workarounds mentioned there, like invoking `git symbolic-ref HEAD refs/heads/main` after each `git init` turned out to have mysterious side effects that break the tests in surprising ways. * 🎨 Make Git integration test non-destructive This patch makes use of the `$HOME` environment variable to trick Git into using a user-global config generated in the temporary directory.
* unarchive - properly handle relative path for `dest` (#75267)Sam Doran2023-03-012-0/+27
| | | | | | | | | | | | | * unarchive - properly handle relative path for dest * Add integration test * Return output of underlying commands with increased verbosity * Revert "Return output of underlying commands with increased verbosity" This reverts commit a2790c8275cdc5697b65670a0beffdc74b741bf6. * Warn when a relative destination path was provided
* add a worker queue to get updates from the main results thread (#79886)Sloane Hertel2023-02-282-0/+48
| | | | | | | | * Create a queue per WorkerProcess to receive intra-task updates * Update `pause` action to use the worker queue * Deprecate ConnectionBase()._new_stdin * Add new `Display` convenience method `prompt_until` to manage both controller- and worker-sourced prompting without cross-fork stdin sharing, in-worker mechanism to handle request-response over new worker queue.