Change in vdsm[master]: tests: Fix race in misc:DdWatchCopy.testStop

danken at redhat.com danken at redhat.com
Wed Jan 30 20:45:51 UTC 2013


Dan Kenigsberg has posted comments on this change.

Change subject: tests: Fix race in misc:DdWatchCopy.testStop
......................................................................


Patch Set 3: Looks good to me, approved

(1 inline comment)

Convinced by Saggi that it's good enough, despite the minute risk of src.name being taken by something completely different after it is unlink'ed.

....................................................
File tests/miscTests.py
Line 507:         """
Line 508:         try:
Line 509:             with tempfile.NamedTemporaryFile() as src:
Line 510:                 os.unlink(src.name)
Line 511:                 os.mkfifo(src.name)
saggi: I don't want to read stderr and mess the output out on an error
Line 512:                 with tempfile.NamedTemporaryFile() as dst:
Line 513:                     ddWatchCopy(src.name, dst.name, lambda: True, 100)
Line 514:         except misc.se.ActionStopped:
Line 515:             self.log.info("Looks like it stopped!")


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I50e044ec2d949a95e52adcfbc0098a032fb7193d
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list