Skip to content

Conversation

@alqu
Copy link
Contributor

@alqu alqu commented Apr 16, 2020

...without having to rename roles.

@Klaas-
Copy link
Contributor

Klaas- commented Apr 16, 2020

I tried to use the same variables as in awx installer when adding functionality from the kubernetes deployment of the awx installer, that's why they are not prefixed :D

@alqu
Copy link
Contributor Author

alqu commented Apr 17, 2020

Yep, I see.

I use this role together with a bunch of other roles, so it is very likely that those commonly named variables might clash with others - that was the reasoning behind it.

@ikke-t
Copy link
Owner

ikke-t commented Apr 17, 2020

What is this now? I just squashed and merged these changes, did you create two PR for the same changes?

@alqu
Copy link
Contributor Author

alqu commented Apr 17, 2020

I accidentally committed 974d8e3 into this branch...
This branch fixes the problem that occurs if you use the galaxy role.
Then ansible complains about not finding role "podman_container_systemd", since it only knows "ikke_t.podman_container_systemd" from galaxy.

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.

3 participants