Merge pull request #308 from droidben/fix-prerequisites

Fix prerequisites in cephadm-preflight
ceph · Jul 10, 2024 · 8c7e6b9 · 8c7e6b9
2 parents 40ce4c8 + 3be357e
commit 8c7e6b9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/ceph_defaults/defaults/main.yml b/ceph_defaults/defaults/main.yml
@@ -20,5 +20,6 @@ ceph_client_pkgs:
 infra_pkgs:
   - chrony
   - podman
+  - lvm2
   - sos
 client_group: clients
diff --git a/cephadm-preflight.yml b/cephadm-preflight.yml
@@ -171,7 +171,7 @@
               command: dnf config-manager --set-enabled "{{ 'powertools' if ansible_facts['distribution_major_version'] == '8' else 'crb' }}"
               changed_when: false
 
-            - name: install package
+            - name: install epel package
               package:
                 name: epel-release
                 state: present
@@ -205,7 +205,7 @@
             state: "{{ (upgrade_ceph_packages | bool) | ternary('latest', 'present') }}"
           register: result
           until: result is succeeded
-          when: group_names == [client_group]
+          when: client_group in group_names
 
 
         - name: ensure chronyd is running
-Original file line number
+Diff line change
@@ Expand Up / @@ -20,5 +20,6 @@ ceph_client_pkgs: @@
     infra_pkgs:
       - chrony
       - podman
+      - lvm2
       - sos
     client_group: clients