Bug 1392238 - [ceph-ansible]: Task "ceph-osd | fix partitions gpt header or labels of the osd disks" is failing
Summary: [ceph-ansible]: Task "ceph-osd | fix partitions gpt header or labels of the o...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Storage Console
Classification: Red Hat
Component: ceph-ansible
Version: 2
Hardware: Unspecified
OS: Linux
urgent
urgent
Target Milestone: ---
: 2
Assignee: Sébastien Han
QA Contact: ceph-qe-bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-11-06 16:34 UTC by Tejas
Modified: 2016-11-22 23:42 UTC (History)
11 users (show)

Fixed In Version: ceph-ansible-1.0.5-43.el7scon
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-11-22 23:42:46 UTC
Target Upstream Version:

Attachments (Terms of Use)
ansible playbook log (63.97 KB, text/plain)
2016-11-06 16:45 UTC, Tejas 		no flags Details
ansible playbook log (179.84 KB, text/plain)
2016-11-08 16:19 UTC, Tejas 		no flags Details
ansible playbook log with -vvvv (818.52 KB, text/plain)
2016-11-09 09:01 UTC, Tejas 		no flags Details
ansible playbook log (356.58 KB, text/plain)
2016-11-11 09:58 UTC, Tejas 		no flags Details
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2016:2817 0 normal SHIPPED_LIVE ceph-iscsi-ansible and ceph-ansible bug fix update 2017-04-18 19:50:43 UTC

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
Note You need to log in before you can comment on or make changes to this bug.