Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@damionparry
Copy link

  • Enables propagating the become flag and become user to external logs allowing the use of become to be audited

* Enables propagating the become flag and become user to external logs allowing the use of become to be audited
@damionparry damionparry requested a review from a team as a code owner August 27, 2024 16:19
@Tompage1994
Copy link

@ansible/executor is it possible to get some traction on this PR? It's been awaiting a review for a long time now

Thanks

@Shrews Shrews closed this Apr 2, 2025
@Shrews Shrews reopened this Apr 2, 2025
@Shrews Shrews added the needs_triage New item that needs to be triaged label Apr 2, 2025
Copy link

@Tompage1994 Tompage1994 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Found an issue with the tests

Copy link

@Tompage1994 Tompage1994 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@damionparry linting hates you

Copy link
Contributor

@Shrews Shrews left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Since this touches the awx_display callback, we have to get approval from someone on the Controller/AWX team since that is a crucial piece to that project.

@AlanCoding Can you or someone from your team have a look at this? Not sure that it needs full testing, but will leave that up to you.

@Shrews Shrews requested a review from AlanCoding April 2, 2025 15:16
@s-hertel s-hertel removed the needs_triage New item that needs to be triaged label Apr 3, 2025
@Tompage1994
Copy link

Can we get more traction on this again please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants