Skip to content
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

Put our ansible roles in standard location #324

Merged
merged 1 commit into from
Aug 20, 2021

Conversation

kbrock
Copy link
Member

@kbrock kbrock commented Aug 19, 2021

related: ManageIQ/manageiq-rpm_build#159

using standard ansible configuration and removed special
configuration file.

we removed this to get away from a root specific configuration
which is necessary as we go towards running this as manageiq user

using standard ansible configuration and removed special
configuration file.

we removed this to get away from a root specific configuration
which is necessary as we go towards running this as manageiq user
@NickLaMuro
Copy link
Member

@kbrock can we mark this as [WIP] until the RPM build PR is in place, and mark that PR as a prerequisite for this one? Otherwise code that relies on our custom roles will not work on the interim if a build goes out without.

I don’t expect there to be much of a hold up with the RPM PR once we can validate that, but they should be merged in close proximity and marked as dependent on case they are flagged for backport.

@kbrock kbrock assigned NickLaMuro and unassigned NickLaMuro Aug 20, 2021
@NickLaMuro NickLaMuro merged commit 76186ee into ManageIQ:master Aug 20, 2021
@kbrock kbrock deleted the ansible-path branch August 20, 2021 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants