Timothy Asir has posted comments on this change. Change subject: gluster:fix NoneType error for some devices does not contain sysfsPath ......................................................................
Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/41466/1/vdsm/gluster/storagedev.py File vdsm/gluster/storagedev.py: Line 82: # lvm vg will not have sysfs path Line 83: if hasattr(udev, 'get_device'): Line 84: dev = udev.get_device(device.sysfsPath) or {} Line 85: elif hasattr(udev, 'udev_get_device'): Line 86: dev = udev.udev_get_device(device.sysfsPath) or {} > The bz in blivet should have been filed before those patches got merged. Y Bug was already there for RHEL7: https://bugzilla.redhat.com/show_bug.cgi?id=1223855. For RHEL6 they told that its fixed already. However Our QE not verified yet. May be we missed to identify / review it in the previous patches. It should of documented in the previous patches not here. Specifying anything here which related to previous patch is not a good idea i feel. Let this patch go in. Its all about a document related good to have thing. Does not block anything. I can send a separate enhancement patch to mention that "this is just a workaround till its fixed in blivet properly" later on. Line 87: else: Line 88: dev = {} Line 89: info['fsType'] = device.format.type or dev.get('ID_FS_TYPE', '') Line 90: if hasattr(device.format, 'mountpoint'): -- To view, visit https://gerrit.ovirt.org/41466 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I54efdf3958519e2ae038b527a03e0b57569c2e8f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5-gluster Gerrit-Owner: Timothy Asir <[email protected]> Gerrit-Reviewer: Bala.FA <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Darshan N <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Ramesh N <[email protected]> Gerrit-Reviewer: Sahina Bose <[email protected]> Gerrit-Reviewer: Timothy Asir <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
