search

redhat-cop / aap_utilities

Ansible Collection for automated deployment of AAP and other objects for general use
https://galaxy.ansible.com/infra/aap_utilities
GNU General Public License v3.0
78 stars 47 forks source link

Updated to remove become and a python env #26

Closed sean-m-sullivan closed 3 years ago

sean-m-sullivan commented 3 years ago

What does this PR do?

After doing some testing in an Github runner that will be brought here soon, I found a few problems with the role.

1 the become requirement wasn't needed and could be removed.

2 the users ansible.cfg should be used to set the python environment if it is not set, not the inventory file as inventory takes precedent over config.

How should this be tested?

Has been tested on openshift 4.6.

sean-m-sullivan commented 3 years ago

We can change that to a variable, let me look at that.

sean-m-sullivan commented 3 years ago

Updated to have option for verbose install, and tested.