From 5aec9da11e050b4e6b5874e2542ed70ca37082f3 Mon Sep 17 00:00:00 2001 From: Clark Boylan Date: Wed, 25 Oct 2023 07:56:07 -0700 Subject: [PATCH] Revert "Cap ruamel.yaml install for ARA" This reverts commit a77eebe911b9651575c32dec8cb5ac84e4057192. 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 --- playbooks/roles/install-ansible/tasks/main.yaml | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/playbooks/roles/install-ansible/tasks/main.yaml b/playbooks/roles/install-ansible/tasks/main.yaml index 76a9223dec..dabfba0ad2 100644 --- a/playbooks/roles/install-ansible/tasks/main.yaml +++ b/playbooks/roles/install-ansible/tasks/main.yaml @@ -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