-
Notifications
You must be signed in to change notification settings - Fork 53
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
Conversation
Since ansible/awx#15087 is merged, this can be taken out of draft... right after a rebase. |
4b21fb8
to
a466b66
Compare
|
@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. |
|
Note: I wanted to see that the "Testing DAB Consumers" check (introduced after this was created) was passing after a rebase, and it is, so it looks like we're good there. #435 includes the commit from this, and now also proposes a new replacement system. My focus is going to be on that, so I'm going to close this PR for now, with the note that these changes are included completely there. |
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.