Change in vdsm[master]: Refactoring storage.misc: moving watchCmd and its deps to vd...

danken at redhat.com danken at redhat.com
Thu May 16 19:29:34 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Refactoring storage.misc: moving watchCmd and its deps to vdsm.utils.
......................................................................


Refactoring storage.misc: moving watchCmd and its deps to vdsm.utils.

This refactoring is needed to break the depedency of lib/vdsm/qemuImg
from vdsm/storage/misc. Modules inside lib/vdsm should not use
anything that is not present in the python std lib or in a dependency
of the vdsm-python package.

Change-Id: I45eaf487b0260863af09087e5114fb836276a750
Signed-off-by: Giuseppe Vallarelli <gvallare at redhat.com>
---
M lib/vdsm/qemuImg.py
M lib/vdsm/utils.py
M tests/main.py
M tests/miscTests.py
M vdsm/storage/Makefile.am
M vdsm/storage/blockSD.py
M vdsm/storage/blockVolume.py
M vdsm/storage/clusterlock.py
M vdsm/storage/fileSD.py
M vdsm/storage/fileVolume.py
M vdsm/storage/image.py
M vdsm/storage/misc.py
M vdsm/storage/storage_exception.py
M vdsm/storage/volume.py
14 files changed, 105 insertions(+), 94 deletions(-)

Approvals:
  Giuseppe Vallarelli: Verified
  Dan Kenigsberg: Looks good to me, approved
  Zhou Zheng Sheng: Looks good to me, but someone else must approve


--
To view, visit http://gerrit.ovirt.org/14408
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I45eaf487b0260863af09087e5114fb836276a750
Gerrit-PatchSet: 17
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Giuseppe Vallarelli <gvallare at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Giuseppe Vallarelli <gvallare at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list