Bug 641476

Summary: devicemapper UUID field cannot be assigned after map creation
Product: [Fedora] Fedora Reporter: James Laska <jlaska>
Component: kernelAssignee: Alasdair Kergon <agk>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: high    
Version: 14CC: agk, aquini, awilliam, dcantrell, dlehman, dougsland, gansalmon, itamar, jonathan, jturner, kernel-maint, madhu.chinakonda, mbroz, pjones, prajnoha, stickster
Target Milestone: ---   
Target Release: ---   
Hardware: i386   
OS: Linux   
Whiteboard: AcceptedBlocker
Fixed In Version: 2.6.35.6-44.fc14 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 584328 Environment:
Last Closed: 2010-10-19 05:02:10 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 584328    

Description James Laska 2010-10-08 19:57:22 UTC
In order to properly resolve an issue originally discovered in bug#584328, a fix to kernel and lvm2 is also required.  Please see below for details from dlehman.

+++ This bug was initially created as a clone of Bug #584328 +++

The following was filed automatically by anaconda:
anaconda 13.37.2 exception report
Traceback (most recent call first):
  File "/usr/lib/anaconda/iw/partition_gui.py", line 1066, in populate
    devstring = device.name
  File "/usr/lib/anaconda/iw/partition_gui.py", line 1836, in getScreen
    self.populate(initial = 1)
  File "/usr/lib/anaconda/gui.py", line 1393, in setScreen
    new_screen = self.currentWindow.getScreen(anaconda)
  File "/usr/lib/anaconda/gui.py", line 1314, in nextClicked
    self.setScreen ()
AttributeError: 'NoneType' object has no attribute 'name'

<snip>

--- Additional comment from dlehman on 2010-10-08 13:36:31 EDT ---

pjones is working on a patch to the kernel (device-mapper) to allow setting of uuid for already-existing maps, as long as they have no uuid at that time. 

Patches will also be required for libdevmapper to provide a function along the lines of the existing function dm_task_set_newname, but for uuid instead of name. I have drafted a patch to pyblock that will make use of all of the above to ensure correct uuids are set for dmraid and mpath devices created by pyblock.

--- Additional comment from awilliam on 2010-10-08 14:16:10 EDT ---

At the 2010-10-08 blocker meeting we agreed to reassign this bug to pyblock and create new bugs against kernel and libdevmapper which block this bug. Jlaska will be doing this.

Comment 1 James Laska 2010-10-11 20:16:30 UTC
Please note, this bug is on the dependency chain as blocking the release of F14 blocker.  Please help resolve this issue as quickly as possible.

Comment 2 Peter Jones 2010-10-11 20:29:58 UTC
A patch has been posted to dm-devel as of Friday, 8-Oct-2010, and has been updated a couple of times since then.

Comment 3 Milan Broz 2010-10-11 20:58:44 UTC
Seems Alasdair said on dm-devel that he will accept the kernel patch the "old way", assigning to him.

Comment 4 Alasdair Kergon 2010-10-12 11:55:04 UTC
This seems to be the proposed patch for me to review:

https://patchwork.kernel.org/patch/242631/

Comment 5 Paul W. Frields 2010-10-12 14:34:51 UTC
FYI, as seen in bug 641474: "The F14 devel schedule
(http://poelstra.fedorapeople.org/schedules/f-14/f-14-devel-tasks.html) says
that the final change deadline is 18 October."  Thanks for your prompt attention!

Comment 6 Dave Jones 2010-10-13 17:04:53 UTC
Should be fixed in 2.6.35.6-42

Comment 7 Fedora Update System 2010-10-13 23:35:00 UTC
kernel-2.6.35.6-43.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/kernel-2.6.35.6-43.fc14

Comment 8 Alasdair Kergon 2010-10-14 16:33:12 UTC
Patch reviewed and accepted into the upstream device-mapper queue for 2.6.37.  I have done some refactoring, but there is no difference of any significance to its functionality.

Comment 9 Alasdair Kergon 2010-10-15 13:54:58 UTC
Peter Rajnoha tested this thoroughly this morning and was able to make the dm remove ioctl hang.  A one-line fix to the patch is required.

Comment 10 Adam Williamson 2010-10-15 16:35:30 UTC
Discussed at the 2010-10-15 blocker review meeting. pjones is working on this whole set of bugs and expects it to be cleaned up today.



-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

Comment 11 Fedora Update System 2010-10-18 05:40:22 UTC
kernel-2.6.35.6-43.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 12 James Laska 2010-10-18 16:41:59 UTC
(In reply to comment #11)
> kernel-2.6.35.6-43.fc14 has been pushed to the Fedora 14 stable repository.  If
> problems still persist, please make note of it in this bug report.

Moving into ON_QA as an updated kernel is available with the fix.

https://admin.fedoraproject.org/updates/kernel-2.6.35.6-43.fc14

Comment 13 Jesse Keating 2010-10-19 05:02:10 UTC
That kernel is in stable now.