opendev/system-config
Code Issues Proposed changes

Revert "Cap ruamel.yaml install for ARA"

Browse Source 
This reverts commit a77eebe911.

Ruamel.yaml 0.18.2 converted the error assocaited with the use of this
deprecated method from a sys.exit(1) to a raised Exception. It is
believed that this will allow Ara to run in some capacity and we don't
need to pin this dependency anymore.

More details in the upstream bug here:

  https://github.com/ansible-community/ara/issues/524

Change-Id: I694b8a016755d828490f0bcf4c6ceb812edf43d9
This commit is contained in:
Clark Boylan 2023-10-25 07:56:07 -07:00
parent a045d7590a
commit 5aec9da11e
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
playbooks/roles/install-ansible/tasks/main.yaml
View File

@ -13,9 +13,7 @@
- name: Add ARA to defaults if enabled
when: install_ansible_ara_enable
set_fact:
# Ruamel.yaml 0.18.0 and newer removed a bunch of deprecated functions
# that ARA depends on. Cap the version until ARA updates.
_install_ansible_requirements: '{{ _install_ansible_requirements + ["ara[server]", "ruamel.yaml<0.18.0"] }}'
_install_ansible_requirements: '{{ _install_ansible_requirements + ["ara[server]"] }}'
- name: Set variable
# NOTE(ianw) the block when: statement is calcuated for each task