Bala.FA has submitted this change and it was merged. Change subject: gluster:fix NoneType error for some devices does not contain sysfsPath ......................................................................
gluster:fix NoneType error for some devices does not contain sysfsPath Currently devices list returns the list of available disks and its fs type properly. However it fails for some devices which does not contain sysfsPath and the dev details become None. This is a workaround patch. This fixes this error by assigning empty dictonary if the dev is empty or None. This will be removed once the dependent bugs fixed Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1225417 Change-Id: I54efdf3958519e2ae038b527a03e0b57569c2e8f Signed-off-by: Timothy Asir Jeyasingh <[email protected]> Reviewed-on: https://gerrit.ovirt.org/41262 Reviewed-by: Piotr Kliczewski <[email protected]> Reviewed-by: Ramesh N <[email protected]> Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-on: https://gerrit.ovirt.org/41466 Tested-by: Ramesh N <[email protected]> Reviewed-by: Bala.FA <[email protected]> Continuous-Integration: Bala.FA <[email protected]> --- M vdsm/gluster/storagedev.py 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Timothy Asir: Verified Bala.FA: Looks good to me, approved; Passed CI tests Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/41466 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I54efdf3958519e2ae038b527a03e0b57569c2e8f Gerrit-PatchSet: 6 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] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
