become sudo no longer working with cached credentials for localhost in 2.19.0 — please document this in the porting guide
Summary
See https://github.com/ansible/ansible/issues/85536. Since this was an intentional change that is breaking users, it should be listed in the changelog/porting guide.
docs/docsite/rst/porting_guides/porting_guide_12.rst:- Task Execution / Forks - Forks no longer inherit stdio from the parent ``ansible-playbook`` process. ``stdout``, ``stderr``, and ``stdin`` within a worker are detached from the terminal, and non-functional. All needs to access stdio from a fork for controller side plugins requires use of ``Display``.
currently exists, but it should also explain the implications on sudo caching.
This issue was brought up in the context of https://pagure.io/fesco/issue/3518, so I figured I'd file an issue here.
Issue Type
Bug Report
Component Name
general
Ansible Version
$ ansible --version
2.19
Configuration
# if using a version older than ansible-core 2.12 you should omit the '-t all'
$ ansible-config dump --only-changed -t all
OS / Environment
local connection plugin
Steps to Reproduce
Expected Results
n/a
Actual Results
n/a
Code of Conduct
- [x] I agree to follow the Ansible Code of Conduct
✔️ Linting Passed
All linting checks passed. Your pull request is in excellent shape! ☀️
Thank you for the feedback! I agree that the diff is really hard to read.
I had to fiddle around with the code a bit to make it easy to switch between real dependencies and a minimal test (because the code does not only check if the versions are the same, but also if the specified packages are the same (which makes sense), so I had to create matching minimal test samples for both min_dependencies and pyproject).
I tried to reduce the differences, and comparing this version to main in VSCode (using GitLens) actually doesn't look too bad, but here on GitHub it tries to align the wrong overlapping parts... :disappointed: I'll try to add some more explanations in the code, let me know if you can work with that.