| Commit message (Collapse) | Author | Files | Lines |
|
Fixes: #83608
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
|
|
Removes the docs for the environment keyword in the shell base plugins
as they are a no-op in the plugins themselves. The environment value is
provided by the action base which gets it from the action base on the
task.environment value. This should avoid confusion around how its being
set and removes some code that is not used at all.
|
|
Use secrets library instead of random.
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
|
|
While upgrade process removes a package, module should
report changed=True instead of changed=False
Fixes: #46314
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
|
|
* Update the timeout to download role archive to 60 seconds to match collections
TODO: in future make a configurable 'artifact download timeout'
|
|
Fixes: #79959
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
|
|
Co-authored-by: Abhijeet Kasurde <akasurde@redhat.com>
|
|
The VyOS remote image required for running the tests is no longer functional.
|
|
add testcase for expanding home dir names
|
|
|
|
|
|
|
|
This reverts commit 0d5460df16af2c8e082796c6a46cb075060f74c9.
|
|
|
|
|
|
* lift code that normalizes value type for boolean vtype to cover both
branches of conditional.
* remove obsolete and incomplete conversion of type in set_selection.
Fixes: #83594
Signed-off-by: Peter A. Bigot <pab@pabigot.com>
|
|
|
|
|
|
* fixed typo in description
* fixed comment in examples
|
|
Previous change overlooked 'uncommenting' the variable entry as a way to update this to keep the functionality.
Co-authored-by: Glandos <bugs-github@antipoul.fr>
Co-authored-by: Abhijeet Kasurde <akasurde@redhat.com>
|
|
|
|
Signed-off-by: Roberto Alfieri <ralfieri@redhat.com>
|
|
* dnf: follow-up on is_newer_installed arches fix
* fix for the non package object case
* prevent early bailout in dnf _is_newer_version_installed
* non-installed available arches would fail out of the check early
---------
Co-authored-by: Matt Davis <nitzmahone@redhat.com>
|
|
(#83422)
|
|
(#83420)
|
|
|
|
|
|
* Quality-of-life improvements for release tool
- Default devel releases to b1
- Default non-devel releases to rc1
- Default to release announcement to console
- Avoid auto-links in GH release annoucements for file sizes
|
|
Co-authored-by: flowerysong <junk+github@flowerysong.com>
|
|
|
|
Using this dictionary to store the return values results in
the return values showing up in the returned
`invocation['module_args']`, which is confusing. It also causes all
module arguments to be returned, which is preserved by this change but
should ideally be removed in the future.
|
|
|
|
|
|
Fixes: #82490
|
|
different options (#83530)
* Reject option/alias names equal up to casing belonging to different options.
* Update test/lib/ansible_test/_util/controller/sanity/validate-modules/validate_modules/main.py
Co-authored-by: Sloane Hertel <19572925+s-hertel@users.noreply.github.com>
|
|
* package_facts: fix warning logic
* Refactor so that warnings can work
|
|
Fixes #83019
|
|
Fixes #83447
|
|
fixes #82872
nicer implementation courtesy of nitzmahone
|
|
|
|
previouslly we recorded but did not show to avoid spam
since we could not dedup from forks, that was already
fixed in another PR so now we can show/display them.
Also:
* funcitonalize deprecation msg construct from docs
* reuse formatting func in cli
* normalize alternatives: most of the code used intended plural
but some and most data/tests used the singular
* update schemas and tests
Co-authored-by: Matt Davis <6775756+nitzmahone@users.noreply.github.com>
Co-authored-by: Felix Fontein <felix@fontein.de>
|
|
* Replace references to archived ansible/ansible-examples repository
* Update examples to reference ansible repository
* Include HTTPS among SSH references in examples
|
|
Fixes #83498
Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua>
|
|
|
|
Previously, `support_discard` simply returned the value of
`/sys/block/{device}/queue/discard_granularity`. When its value is `0`, then
the block device doesn't support discards; _however_, it being greater than
zero doesn't necessarily mean that the block device _does_ support discards.
But another indication that a block device doesn't support discards is
`/sys/block/{device}/queue/discard_max_hw_bytes` being equal to `0` (with the
same caveat as above). So if either of those are `0`, set `support_discard` to
zero, otherwise set it to the value of `discard_granularity` for backwards
compatibility.
Signed-off-by: Benoît Knecht <bknecht@protonmail.ch>
|
|
Fixes #83406
|
|
Identical branches in conditional node: 'elif name != permanent_hostname' and 'else'. First bruch is not nessesary
Co-authored-by: mar.s <marchenkov.s.d@gmail.com>
|
|
Fixes #83373
|
|
* Enable Ubuntu 24.04 group 6 in CI
* Disable rootfull Podman on Ubuntu
* Disable unix-chkpwd AppArmor profile on Ubuntu for Fedora 40 tests
* Document AppArmor and rootfull issues
|
|
we only templated in some cases but when queueing we could get an untemplated name for the 'on start' event.
|