Dan Kenigsberg has submitted this change and it was merged. Change subject: domainMonitor: Cleanup imports ......................................................................
domainMonitor: Cleanup imports - Import only modules, except sdCache and config that we import everywhere. This should be fixed by converting them to modules since they are singeltons anyway - in another patch. - Import standard library modules before vdsm - Import vdsm library modules before package modules - Use relative imports for package imports - Sort imports in each group Change-Id: Iff024da4c259e8976e6248b5b10e1bb55128312c Signed-off-by: Nir Soffer <[email protected]> Reviewed-on: http://gerrit.ovirt.org/29015 Reviewed-by: Yoav Kleinberger <[email protected]> Reviewed-by: Federico Simoncelli <[email protected]> --- M vdsm/storage/domainMonitor.py 1 file changed, 13 insertions(+), 12 deletions(-) Approvals: Nir Soffer: Verified Federico Simoncelli: Looks good to me, approved Yoav Kleinberger: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/29015 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iff024da4c259e8976e6248b5b10e1bb55128312c Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Xavi Francisco <[email protected]> Gerrit-Reviewer: Yoav Kleinberger <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
