diff options
| author | Brian Coca <bcoca@users.noreply.github.com> | 2022-09-28 22:54:48 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-09-28 22:54:48 +0300 |
| commit | 0b678d5036f64f95d50d73d3d27a523a2b264050 (patch) | |
| tree | 0ff1d8d16e96a7c29ede89529f5724e5307d07c5 /changelogs | |
| parent | 3423a6609f3bb2f0057f3b1bf04f1bfafaa066f3 (diff) | |
avoid roles exporting vars: (#69040)
- correct 'vars:' precedence to allow phasing out of include_params
- actually merge vars and always include role_vars
- avoided dupe deps from giving wrong vars
- use 'first' instance of dep as others are from previous instances/invocations
and can have diff values for vars
- ensured deps only provide exportable vars themselves
- added COMMENTS
- added tests
- apply export restrictions setting to defaults
- use 'public' as cutoff
Co-authored-by: Sloane Hertel <19572925+s-hertel@users.noreply.github.com>
ci_complete
Diffstat (limited to 'changelogs')
| -rw-r--r-- | changelogs/fragments/role_vars_scope_fix.yml | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/changelogs/fragments/role_vars_scope_fix.yml b/changelogs/fragments/role_vars_scope_fix.yml new file mode 100644 index 00000000000..8f45cbbc0ea --- /dev/null +++ b/changelogs/fragments/role_vars_scope_fix.yml @@ -0,0 +1,2 @@ +bugfixes: + - Fix bug in `vars` applied to roles, they were being incorrectly exported among others while only vars/main.yml was meant to be. Also adjusted the precedence to act the same as inline params. |