From: Vishal Kanaujia <vishal.kanaujia@flipkart.com>
Date: Wed, 16 May 2018 09:58:31 +0000 (+0530)
Subject: Skip GPT header creation for lvm osd scenario
X-Git-Tag: v3.2.0beta1~86
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=ef5f52b1f36188c3cab40337640a816dec2542fa;p=ceph-ansible.git

Skip GPT header creation for lvm osd scenario

The LVM lvcreate fails if the disk already has a GPT header.
We create GPT header regardless of OSD scenario. The fix is to
skip header creation for lvm scenario.

fixes: https://github.com/ceph/ceph-ansible/issues/2592

Signed-off-by: Vishal Kanaujia <vishal.kanaujia@flipkart.com>
---

diff --git a/roles/ceph-osd/tasks/main.yml b/roles/ceph-osd/tasks/main.yml
index b61cfcbed..16220548a 100644
--- a/roles/ceph-osd/tasks/main.yml
+++ b/roles/ceph-osd/tasks/main.yml
@@ -38,6 +38,8 @@
 
 - name: include check_gpt.yml
   include: check_gpt.yml
+  when:
+    - osd_scenario != 'lvm'
 
 - name: include scenarios/collocated.yml
   include: scenarios/collocated.yml
@@ -66,6 +68,7 @@
   include: activate_osds.yml
   when:
     - not containerized_deployment
+    - osd_scenario != 'lvm'
   # Hard code this so we will skip the entire file instead of individual tasks (Default isn't Consistent)
   static: False
 
@@ -73,6 +76,7 @@
   include: start_osds.yml
   when:
     - not containerized_deployment
+    - osd_scenario != 'lvm'
   # Hard code this so we will skip the entire file instead of individual tasks (Default isn't Consistent)
   static: False