Change in vdsm[master]: tests: v2v: add test for commit fb4c72a

danken at redhat.com danken at redhat.com
Thu Oct 22 10:03:27 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: tests: v2v: add test for commit fb4c72a
......................................................................


tests: v2v: add test for commit fb4c72a

In commit fb4c72a we made sure to continue to extract VM information
if the call to get allocation and capacity failed.
This patch adds one test to make it unlikely to break this again.

Change-Id: Iad523dc7c34276c6811513220cd7230ca5fc6e45
Bug-Url: https://bugzilla.redhat.com/1265556
Signed-off-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/47364
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Shahar Havivi <shavivi at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M tests/v2vTests.py
1 file changed, 45 insertions(+), 1 deletion(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Shahar Havivi: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iad523dc7c34276c6811513220cd7230ca5fc6e45
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list