From ed4b1af4d163ae4c969018720cf2fc0bf0657fca Mon Sep 17 00:00:00 2001
From: yatinkarel <ykarel@redhat.com>
Date: Fri, 3 May 2019 18:22:25 +0530
Subject: [PATCH] Handle lvcreate failure by ensuring dep on lvm2

[1] added lvm_create task to create a lvm device
required by ceph-nautilus. lvcreate command fails
some times with below Error:-
/dev/ceph_vg/lv_data: not found: device not cleared

To overcome this let's add a direct dependency on 'lvm2'
package as lvm_create task needs that package, in addition
adds a sleep task of 5 seconds.

[1] https://review.opendev.org/#/c/650281

Change-Id: I61b6c353cb5216265bf31d56a9f40437821b92fc
---
 manifests/ceph.pp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/manifests/ceph.pp b/manifests/ceph.pp
index b8808c806..85debd833 100644
--- a/manifests/ceph.pp
+++ b/manifests/ceph.pp
@@ -34,12 +34,15 @@ class openstack_integration::ceph (
     })
   }
 
+  ensure_packages(['lvm2'], {'ensure' => 'present', before  => Exec['lvm_create']})
+
   exec { 'lvm_create':
     command   => "/bin/true # comment to satisfy puppet syntax requirements
 truncate --size=10G /diskimage.img
 losetup /dev/loop0 /diskimage.img
 pvcreate /dev/loop0
 vgcreate ceph_vg /dev/loop0
+sleep 5
 lvcreate -n lv_data -a y -l 100%FREE ceph_vg
 ",
     unless    => "/bin/true # comment to satisfy puppet syntax requirements