[ARVADOS] updated: da7eabf017b1f61c207d3beb861333eec7792db8

git at public.curoverse.com git at public.curoverse.com
Tue Feb 2 17:23:07 EST 2016


Summary of changes:
 services/fuse/arvados_fuse/command.py  | 3 +--
 services/fuse/tests/mount_test_base.py | 7 +++----
 2 files changed, 4 insertions(+), 6 deletions(-)

       via  da7eabf017b1f61c207d3beb861333eec7792db8 (commit)
      from  03ab37585a5f624f2c25a8a3ab8e1d864fc4d846 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit da7eabf017b1f61c207d3beb861333eec7792db8
Author: Tom Clegg <tom at curoverse.com>
Date:   Tue Feb 2 16:46:35 2016 -0500

    8288: Do not call operations.destroy() as a last resort, just abandon the llfuse thread.

diff --git a/services/fuse/arvados_fuse/command.py b/services/fuse/arvados_fuse/command.py
index ae0c920..6d53004 100644
--- a/services/fuse/arvados_fuse/command.py
+++ b/services/fuse/arvados_fuse/command.py
@@ -123,9 +123,8 @@ class Mount(object):
         if self.llfuse_thread.is_alive():
             self.logger.warning("Mount.__exit__:"
                                 " llfuse thread still alive %fs after umount"
-                                " -- resorting to operations.destroy()",
+                                " -- abandoning and exiting anyway",
                                 self.args.unmount_timeout)
-            self.operations.destroy()
 
     def run(self):
         if self.args.exec_args:
diff --git a/services/fuse/tests/mount_test_base.py b/services/fuse/tests/mount_test_base.py
index 91a4929..c79daf8 100644
--- a/services/fuse/tests/mount_test_base.py
+++ b/services/fuse/tests/mount_test_base.py
@@ -68,12 +68,11 @@ class MountTestBase(unittest.TestCase):
         del self.pool
 
         subprocess.call(["fusermount", "-u", "-z", self.mounttmp])
-        self.llfuse_thread.join(timeout=0.1)
+        self.llfuse_thread.join(timeout=1)
         if self.llfuse_thread.is_alive():
             logger.warning("MountTestBase.tearDown():"
-                           " llfuse thread still alive 100ms after umount"
-                           " -- resorting to operations.destroy()")
-            self.operations.destroy()
+                           " llfuse thread still alive 1s after umount"
+                           " -- abandoning and exiting anyway")
 
         os.rmdir(self.mounttmp)
         if self.keeptmp:

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list