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 {} > Got it. But bz#1209493 is ovirt one. What about the bug in blivet of missi We have't think of creating a new bug in blivet for this device.format.type issue when I send a patch which got merged already in upstream and downstream because they have given this alternative way of handling this. If it is really necessary now to create one, I can log a bug for that. What is your view on this? 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
