| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
* fix error if path had wildcard in it
* add test
* add changelog fragment
|
|
|
|
|
| |
also fix bug with multiple items clobbering previous settings
Co-authored-by: Sloane Hertel <19572925+s-hertel@users.noreply.github.com>
|
|
|
| |
Replace use of old `ansible.module_utils._text` and add a unit test to maintain backwards compatibility.
|
| |
|
|
|
|
|
| |
* Added missing parentheses; Fixes 80670
* Removes both parentheses.
|
| |
|
| |
|
|
|
|
|
| |
* source module compression setting directly from config
no need to proxy via play_context, which can fall out of date w/o constant variable updates.
|
|
|
|
|
|
|
|
| |
* 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
|
|
|
| |
Co-authored-by: Matt Martz <matt@sivel.net>
|
|
|
|
|
| |
only (#80370)
Co-authored-by: Sandra McCann <samccann@redhat.com>
|
|
|
| |
* update the include_vars action plugin to always follow symbolic links when traversing directories
|
|
|
|
|
|
|
|
|
| |
* Validate task attributes `run_once` and `action` with finalized attrs after individual loop results
* Validate task attribute `ignore_unreachable` using individual loop results
Once there's a way to post validate only certain fields, we can use self._task.post_validate() instead
This replaces the fix introduced in https://github.com/ansible/ansible/pull/80051.
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
| |
* Update password.py documentation with new semantic markup
|
| |
|
|
|
| |
Signed-off-by: tu1h <lihai.tu@daocloud.io>
|
| |
|
| |
|
|
|
|
|
|
|
| |
* password lookup, handle ident properly when saved
Currently we format and save ident when present but we didn't account for this when reading the saved file
Also added some more robust error handling.
|
|
|
| |
`uvault` should be `unvault`
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* 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>
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
* 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.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
* Change ansible_job_id format
...to be something that does not resemble float or other type which
value could be changed by literal_eval that jinja2_native uses.
Specifically the format of '%d.%d' is converted from str to float and
then back to float which may result in truncating the number resulting
in the job not being found because the job id does not exist.
|
|
|
|
|
| |
* password_hash - give a warning for unsupported algorithms (that raise a TypeError)
* add suggested changes, a test and changelog
|
|
|
| |
Convert these strings to comments, since they are not valid docstrings.
|
|
|
|
|
|
|
| |
* Add ignores for intentional pointless statements
* Remove redundant docstring
* Fix function docstring placement
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Update `collections.abc` imports
- Use `six.moves` for modules and module_utils
- Use `collections.abc` for controller code
This avoids using `ansible.module_utils.common._collections_compat`,
which was added before the vendored `six` was updated to provide these
imports.
* Update _collections_compat to use six.moves
Also update the custom pylint rule to reflect this change.
|
| |
|
| |
|
|
|
|
|
| |
* Clean up unused imports in core
* Add changelog fragment
|
|
|
|
|
|
|
| |
* Use arg validation in debug action. Fixes #79862
* Add clog frag
* Move the msg default into the argspec
|
|
|
| |
Remove irrelevant comment line form example code
|
| |
|
|
|
|
| |
also added tests for normal and bad cases
|
|
|
|
|
| |
Fixes #79776
ci_complete
|
|
|
| |
Fixes #79763
|
|
|
|
|
|
|
|
|
| |
* Fix notes in regex_replace doc
The documentation says that this maps to 'regex.replace' when it should say 're.replace'. The distinction is important because of the third-party 'regex' module that has more features than the built-in 're' module. https://pypi.org/project/regex/
* Fix notes in re.search docs
The documentation says that this maps to 'regex.search' when it should say 're.search'. The distinction is important because of the third-party 'regex' module that has more features than the built-in 're' module. https://pypi.org/project/regex/
|
|
|
|
|
| |
Fixes #79711
ci_complete
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Remove obsolete 'if'
The result dict returned by super.run() will never have a 'skipped' entry.
Without the if, it is more clear that wrap_async is actually defined at the end of the function.
* Remove more dead code.
The result dict will also not contain invocation.
* Remove notes.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Add support for importlib.resources
* Remove the importlib.resources imports
* return the correct data
* Some code comments, and re-order for consistency
* Disallow traversing packages below an individual collection
* Add a traversable class for namespaces
* Re-use variable
* Utilize itertools.chain.from_iterable
Co-authored-by: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua>
* Simplify logic to check for packages from ansible loaders
Co-authored-by: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua>
* Just a generator expression, instead of a generator
* docstrings
* Add comment about find_spec for our namespaces
* Add some initial unit tests for importlib.resources
* normalize
* Utilize importlib.resources for listing collections
* collections_path is already in config, just use config
* install uses a different default for collections_path
* Remove unused import
* Remove duplicate __truediv__
* Bring back TraversableResources
* Apply some small suggestions from code review
Co-authored-by: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua>
Co-authored-by: Matt Davis <6775756+nitzmahone@users.noreply.github.com>
* Remove cross contamination between plugin loader code and CLI code
* Remove unused import
Co-authored-by: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua>
Co-authored-by: Matt Davis <6775756+nitzmahone@users.noreply.github.com>
|
|
|
| |
fixes https://github.com/ansible/ansible/issues/75536
|
|
|
|
|
|
|
|
|
| |
there is a little mistake
##### ISSUE TYPE
- Docs Pull Request
+label: docsite_pr
|
| |
|