Change in vdsm[master]: The return of super vdsm tests

smizrahi at redhat.com smizrahi at redhat.com
Wed Nov 28 15:03:28 UTC 2012


Saggi Mizrahi has posted comments on this change.

Change subject: The return of super vdsm tests
......................................................................


Patch Set 6: Looks good to me, but someone else must approve

(2 inline comments)

Fix spelling errors if you ever get a second iteration

....................................................
Commit Message
Line 5: CommitDate: 2012-11-22 14:22:01 +0200
Line 6: 
Line 7: The return of super vdsm tests
Line 8: 
Line 9: We couldn't run svdsm tests because imports wasn't exists under vdsm
didn't exist
under the vdsm
Line 10: installation directory when running the tests. We want to be able to run
Line 11: those tests without being depended on installing vdsm rpm.
Line 12: 
Line 13: Here we add extraPythonPath that included the current directory. It is


Line 7: The return of super vdsm tests
Line 8: 
Line 9: We couldn't run svdsm tests because imports wasn't exists under vdsm
Line 10: installation directory when running the tests. We want to be able to run
Line 11: those tests without being depended on installing vdsm rpm.
without having to install vdsm
Line 12: 
Line 13: Here we add extraPythonPath that included the current directory. It is
Line 14: useful only for tests, if we run it normally we want vdsm to be
Line 15: installed in its default directories and we don't need this additional


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I52e935662633601ce85150171c37fbf7c6196c15
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Royce Lv <lvroyce at linux.vnet.ibm.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list