Bug 1392238

Summary: [ceph-ansible]: Task "ceph-osd | fix partitions gpt header or labels of the osd disks" is failing
Product: Red Hat Storage Console Reporter: Tejas <tchandra>
Component: ceph-ansibleAssignee: Sébastien Han <shan>
Status: CLOSED ERRATA QA Contact: ceph-qe-bugs <ceph-qe-bugs>
Severity: urgent Docs Contact:
Priority: urgent    
Version: 2CC: adeza, aschoen, ceph-eng-bugs, gmeno, hnallurv, kdreyer, nthomas, sankarshan, seb, shan, vakulkar
Target Milestone: ---   
Target Release: 2   
Hardware: Unspecified   
OS: Linux   
Whiteboard:
Fixed In Version: ceph-ansible-1.0.5-43.el7scon Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-11-22 23:42:46 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
ansible playbook log
none
ansible playbook log
none
ansible playbook log with -vvvv
none
ansible playbook log none

Description Tejas 2016-11-06 16:34:54 UTC
Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 4 Tejas 2016-11-06 16:45:13 UTC
Created attachment 1217769 [details]
ansible playbook  log

Comment 9 seb 2016-11-08 10:26:08 UTC
I know where the bug is, it was introduced by a cherry pick of mine, in this commit: 3cd233fccda85126287558172b4d2548f9416fed to solve rhbz#1391468.

Fix incoming.

Comment 10 seb 2016-11-08 10:42:45 UTC
I pushed a new commit: 473d5805f4bcdcc7e4327b14247c22fae1084fad in the rhscon-2-rhel-patches branch so Ken needs to build the package I guess.

Comment 11 seb 2016-11-08 15:38:27 UTC
Thanks Ken! Tejas please test and let me know :)

Comment 15 Tejas 2016-11-08 16:19:37 UTC
Created attachment 1218609 [details]
ansible playbook  log

Comment 16 Christina Meno 2016-11-08 18:13:50 UTC
Andrew, Alfredo, Ken:

What do you guy think we should do here?

Comment 18 Alfredo Deza 2016-11-08 18:31:17 UTC
I think it should be best to drop it from this version. I don't feel confident in fixing this in time.

I would like to prevent the situation where we add these fixes to either fail further or break other things that were working.

Comment 19 Ken Dreyer (Red Hat) 2016-11-08 19:00:25 UTC
I've run ceph-ansible-1.0.5-41 through clot for RHEL and Ubuntu, so I know it passes the basic smoke tests for ceph-installer.

On the other hand, we have no automated tests for the dmcrypt code paths. It's not clear what we'll uncover next w.r.t dmcrypt support at this point.

Comment 20 Christina Meno 2016-11-08 20:31:16 UTC
Spoke with Federico today, plan is: we'll see if Sebastien comes up with a fix by Wednesday and if that doesn't work we'll revert this, ship last known good version of ceph-ansible and use the process we documented for 2.0 here https://bugzilla.redhat.com/show_bug.cgi?id=1366808

Comment 23 Tejas 2016-11-09 09:01:02 UTC
I have attached the ansible playbook logs with -vvvv enabled just in case.

Thanks,
Tejas

Comment 24 Tejas 2016-11-09 09:01:42 UTC
Created attachment 1218850 [details]
ansible playbook  log with -vvvv

Comment 25 seb 2016-11-09 10:40:52 UTC
I just pushed a fix downstream. Ken please build a new package and Tejas please test.
Thanks!

Comment 31 Tejas 2016-11-11 09:58:03 UTC
Created attachment 1219693 [details]
ansible playbook  log

Comment 33 Tejas 2016-11-11 11:33:11 UTC
Please ignore my comments no. 30, 31, 32.

Those were meant for another BZ.

Actually this BZ seems to be resolved in ceph-ansible-1.0.5-43.el7scon
so Moving to verified.

Comment 35 errata-xmlrpc 2016-11-22 23:42:46 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2016:2817