debian/uca: remove the handler notification

The "update apt cache" in the ceph-handler role was never called and the
handler trigger after adding the uca repository doesn't exist at all.
Instead of using a handler for that we can just set the update_cache
parameter to true like the other apt_repository tasks.

Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>

diff --git a/roles/ceph-common/tasks/installs/debian_uca_repository.yml b/roles/ceph-common/tasks/installs/debian_uca_repository.yml
index d0267f938df3f842a33e67a2766844db1fbbddf1..7861cfd2df123c6e13074659b91fa1214a9bc350 100644 (file)
--- a/roles/ceph-common/tasks/installs/debian_uca_repository.yml
+++ b/roles/ceph-common/tasks/installs/debian_uca_repository.yml
@@ -9,5 +9,4 @@
   apt_repository:
     repo: "deb {{ ceph_stable_repo_uca }} {{ ceph_stable_release_uca }} main"
     state: present
-    update_cache: no
-  notify: update apt cache if a repo was added
+    update_cache: yes

diff --git a/roles/ceph-handler/handlers/main.yml b/roles/ceph-handler/handlers/main.yml
index 4f6b6334cc5efaa5d9bb5c4886ec01f532f878db..42d4aa5ffd18dc471e55119a5d308579cadd9b16 100644 (file)
--- a/roles/ceph-handler/handlers/main.yml
+++ b/roles/ceph-handler/handlers/main.yml
@@ -4,13 +4,6 @@
      - not rolling_update | bool
      - not docker2podman | default(False) | bool
   block:
-    - name: update apt cache
-      apt:
-        update-cache: yes
-      when: ansible_os_family == 'Debian'
-      register: result
-      until: result is succeeded
-
     - name: make tempdir for scripts
       tempfile:
         state: directory