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
74 stars 44 forks source link

Fix hostnames for install check and bump version to 2.3 #133

Closed ericzolf closed 1 year ago

ericzolf commented 1 year ago

What does this PR do?

The hostname defaults for controller and PAH were assuming a list were a dictionary was expected. Took the chance to bump the default AAP version to 2.3

How should this be tested?

Try to update/install a cluster with controller and AH, and without forcing the installation. Without this fix, it fails stating that the dictionary of hosts doesn't have a [0] index.

Is there a relevant Issue open for this?

n/a - found out while updating my own cluster

Other Relevant info, PRs, etc

The fix isn't necessary anymore according to https://issues.redhat.com/browse/AAP-4555