New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix ansible.builtin.include_vars - depth #80995
Conversation
@tachyontec this PR contains the following merge commits: Please rebase your branch to remove these commits. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you be able to add a test for this scenario to avoid any future regressions. It also requires a changelog fragment to document this bugfix https://docs.ansible.com/ansible/latest/community/development_process.html#creating-a-changelog-fragment.
@tachyontec this PR contains the following merge commits: Please rebase your branch to remove these commits. |
@tachyontec this PR contains the following merge commits: Please rebase your branch to remove these commits. |
Tests passed, I think we are ready to merge ;) |
@tachyontec this PR contains the following merge commits: Please rebase your branch to remove these commits. |
fe67e74
to
fa8df21
Compare
* Changes as suggested by sivel * Add changelog fragment and tests Co-authored-by: Matt Martz <matt@sivel.net> Co-authored-by: s-hertel <19572925+s-hertel@users.noreply.github.com> (cherry picked from commit 48bed1e)
* Changes as suggested by sivel * Add changelog fragment and tests Co-authored-by: Matt Martz <matt@sivel.net> Co-authored-by: s-hertel <19572925+s-hertel@users.noreply.github.com> (cherry picked from commit 48bed1e) Co-authored-by: tachyontec <92679798+tachyontec@users.noreply.github.com>
SUMMARY
Changed vars.py, so it starts from depth 1 as root, as suggested by @sivel
Fixes #80987
ISSUE TYPE
COMPONENT NAME
vars.py
ADDITIONAL INFORMATION