Edward Haas has posted comments on this change. Change subject: tests: Fix loop device leak in mount tests ......................................................................
Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/51614/1/tests/mountTests.py File tests/mountTests.py: Line 131: m.mount(mntOpts="loop") Line 132: try: Line 133: self.assertTrue(m.isMounted()) Line 134: finally: Line 135: m.umount(force=True, freeloop=True) The underling umount operation is synchronous (and blocking) as far as I can understand. storage.mount module is calling it asynchronously (sync=False) and if a timeout argument is given, it will wait on it or fail (making it kind of synchronous again). It may also help if the lazy flag is raised, so a busy mount point can be released later on. Line 136: # TODO: Use libudev to wait for specific event Line 137: with stopwatch("Wait for udev events"): Line 138: udevadm.settle(5) Line 139: -- To view, visit https://gerrit.ovirt.org/51614 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iada8f3d962778c53ada4a6f0a7f3e75b80c07239 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Edward Haas <edwa...@redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabi...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
