clusterinthecloud / ansible

Ansible config for Cluster in the Cloud
https://cluster-in-the-cloud.readthedocs.io
MIT License
10 stars 27 forks source link

Ansible update fixes #128

Open GMW99 opened 2 years ago

GMW99 commented 2 years ago

Epel has now released version 5.4 of Ansible within stable with this there are a few things that broke these are the fixes.

milliams commented 2 years ago

I've been annoyed that we're stuck on Ansible 2.9 for so long so the fact that 5 is now available in EPEL is very welcome news. Given how much has changed in Ansible in the meantime, I'm surprised that this is all the changes that are needed. To give some more confidence, would you be able to also edit the CI config file (.github/workflows/main.yml) in your branch to change the version of Ansible specified from:

'ansible<2.10,>=2.9' 'ansible-lint<6.0.0'

to perhaps

'ansible>=5,<6' 'ansible-lint'

I expect there are things that will not be compatible.

Furthermore, the mgmt node bootstrap from Terraform (https://github.com/clusterinthecloud/terraform/blob/master/common-files/bootstrap.sh.tpl) runs Ansible before having installed EPEL so it will currently still be getting Ansible 2.9 (whereas the compute nodes to install EPEL before Ansible so will get the new version - https://github.com/clusterinthecloud/ansible/blob/6/roles/packer/templates/prepare_ansible.sh.j2). We might also need to install ansible-collection-community-general to get all the core community roles.