-
Notifications
You must be signed in to change notification settings - Fork 36
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
[HOLD] Remove ANSIBLE_BASE_ROLE_PRECREATE only used by AWX #296
base: devel
Are you sure you want to change the base?
Conversation
Since ansible/awx#15087 is merged, this can be taken out of draft... right after a rebase. |
4b21fb8
to
a466b66
Compare
Quality Gate passedIssues Measures |
@newswangerd can you take a look at this and give it a thumbs up if we're good to merge? |
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.
Is there another mechanism for pre creating roles in place?
We might hold off on merging (at least for a bit), as we're still not totally sure if other apps might use this. |
Checked with @AlanCoding this is still on hold. |
Quality Gate passedIssues Measures |
Opening as a draft initially, the work is going on separately in AWX, along with the role rename.
The AWX change is a blocker for merging this, but I want to be sure everything is passing here.
eda-server did not use this, and I do not anticipate galaxy_ng will either, but I await confirmation of that. If you plan to use this, we can keep it.