Change in vdsm[master]: Don't depend on storage module
danken at redhat.com
danken at redhat.com
Sun Jun 23 17:44:41 UTC 2013
Dan Kenigsberg has submitted this change and it was merged.
Change subject: Don't depend on storage module
......................................................................
Don't depend on storage module
There is no real need for hooking module to depend on the storage
module, except for the execCmd which is being delegated to
utils.execCmd, so importing the execCmd directly from utils instead.
Change-Id: Ib36940cda2a07e2898e26e6e7de4d4f3b1292e8f
Signed-off-by: Mike Kolesnik <mkolesni at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/16003
Reviewed-by: Giuseppe Vallarelli <gvallare at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Tested-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/hooking.py
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Giuseppe Vallarelli: Looks good to me, but someone else must approve
Dan Kenigsberg: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/16003
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib36940cda2a07e2898e26e6e7de4d4f3b1292e8f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Giuseppe Vallarelli <gvallare at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the vdsm-patches
mailing list