From: Zack Cerza Date: Tue, 14 Jul 2015 01:08:24 +0000 (-0600) Subject: Fix replace_repos logic X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=552827f438176ab74d6450b0c3d5596af06c8a06;p=ceph-cm-ansible.git Fix replace_repos logic The issues were with quoting of booleans, and |success vs. .rc == 0 Signed-off-by: Zack Cerza --- diff --git a/roles/common/tasks/rhel-entitlements.yml b/roles/common/tasks/rhel-entitlements.yml index 6679468..3f4e3ab 100644 --- a/roles/common/tasks/rhel-entitlements.yml +++ b/roles/common/tasks/rhel-entitlements.yml @@ -60,7 +60,7 @@ - name: Set replace_repos if newly-subscribed set_fact: replace_repos: True - when: entitled|changed and entitled|success + when: entitled|changed and entitled.rc == 0 - name: Set replace_repos if entitlement is skipped set_fact: @@ -69,14 +69,14 @@ - name: Disable all rhsm repos command: subscription-manager repos --disable '*' - when: replace_repos == "True" + when: replace_repos == True # This produces an absurd amount of useless output no_log: True - name: Enable necessary rhsm repos command: subscription-manager repos --enable {{ item }} with_items: rhsm_repos - when: replace_repos == "True" and rhsm_repos|length > 0 + when: replace_repos == True and rhsm_repos|length > 0 - name: Remove old apt-mirror repository definition. file: