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