Change in vdsm[master]: tests: janitorial: cleanup temporary directories

danken at redhat.com danken at redhat.com
Tue Dec 24 15:39:41 UTC 2013


Dan Kenigsberg has posted comments on this change.

Change subject: tests: janitorial: cleanup temporary directories
......................................................................


Patch Set 5: Code-Review+2

I'd call it "NamedTemproraryDir", with a leading uppercase, to look more like tempfile. But I see why you opted to keep with the silly Vdsm convention.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: If6581771479de9f72360ea537de32abcdad496b3
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list