Prefix all role vars in a consistent way with "awx_"

ikke-t / awx_pod

Ansible role to install Ansible AWX into pod using podman.

55 stars 19 forks source link

Prefix all role vars in a consistent way with "awx_" #13

Closed alqu closed 4 years ago

alqu commented 4 years ago

In order to make clear that all the role-depending vars belong to that role I prefixed them all with a "awx_".

Additionally I added the awx_server_email variable.

ikke-t commented 4 years ago

Good idea, thanks. However, please fix one issue. You renamed the container_state. That should not be done, as it get's passed to included role, which then sets the desired state. Please change that back, and I'll merge.

alqu commented 4 years ago

Done.

ikke-t commented 4 years ago

I squashed and merged it. thanks.