Change in vdsm[master]: lib: Add vdsm.concurrent library

danken at redhat.com danken at redhat.com
Tue Mar 24 14:17:50 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: lib: Add vdsm.concurrent library
......................................................................


lib: Add vdsm.concurrent library

Move misc.tmap to new module. We will probably need to move some other
utilities to the new module later.  The tmap utility will be used to
simplify fc-scan and multipath-resize helpers.

tmap() was moved as is from misc for easier review. The next patch will
clean it up a little bit. The tests were rewritten in a simpler way.

Change-Id: Iee90f4c201293c3df048cc4ad7123a837d236a38
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/38464
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Allon Mureinik <amureini at redhat.com>
Reviewed-by: Adam Litke <alitke at redhat.com>
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M debian/vdsm-python.install
M lib/vdsm/Makefile.am
A lib/vdsm/concurrent.py
M tests/Makefile.am
A tests/concurrentTests.py
M tests/miscTests.py
M vdsm.spec.in
M vdsm/storage/misc.py
8 files changed, 117 insertions(+), 71 deletions(-)

Approvals:
  Adam Litke: Looks good to me, but someone else must approve
  Nir Soffer: Verified
  Yaniv Bronhaim: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Allon Mureinik: Looks good to me, but someone else must approve
  Francesco Romani: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/38464
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iee90f4c201293c3df048cc4ad7123a837d236a38
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list