Sandro Bonazzola has posted comments on this change.

Change subject: packaging: setup: revert nfs config on cleanup
......................................................................


Patch Set 10:

> Still did not verify "live", instead added two test dirs - one based on 
> doctest (Thanks, Sandro, for the tip) and another one using simple files 
> (after I decided doctest is not very comfortable for this kind of tests). 
> Please choose - if noone prefers the doctest one I'll drop it.

Well, I meant to embed the doc test as part of the module, but since it seems 
to fail there, Alon is right suggesting the nose framework. Let's leave the 
test part out of this patch, we'll add it later.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I74dd4c1556bd6479fcf10f85fcbe083f215e0854
Gerrit-PatchSet: 10
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Kiril Nesenko <[email protected]>
Gerrit-Reviewer: Ofer Schreiber <[email protected]>
Gerrit-Reviewer: Ohad Basan <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to