X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/8dfb0e3f8ca8154f4c16ab26e03aa6c6b114b5ad..28e50cc9480fdad416404542511a172cdc7253c7:/services/fuse/arvados_fuse/__init__.py diff --git a/services/fuse/arvados_fuse/__init__.py b/services/fuse/arvados_fuse/__init__.py index 72ccdb5f94..0944a31875 100644 --- a/services/fuse/arvados_fuse/__init__.py +++ b/services/fuse/arvados_fuse/__init__.py @@ -50,6 +50,16 @@ an object that is live in the inode cache, that object is immediately updated. """ from __future__ import absolute_import +from __future__ import division +from future.utils import viewitems +from future.utils import native +from future.utils import listvalues +from future.utils import listitems +from future import standard_library +standard_library.install_aliases() +from builtins import next +from builtins import str +from builtins import object import os import sys import llfuse @@ -73,8 +83,7 @@ import collections import functools import arvados.keep from prometheus_client import Summary - -import Queue +import queue # Default _notify_queue has a limit of 1000 items, but it really needs to be # unlimited to avoid deadlocks, see https://arvados.org/issues/3198#note-43 for @@ -82,10 +91,10 @@ import Queue if hasattr(llfuse, 'capi'): # llfuse < 0.42 - llfuse.capi._notify_queue = Queue.Queue() + llfuse.capi._notify_queue = queue.Queue() else: # llfuse >= 0.42 - llfuse._notify_queue = Queue.Queue() + llfuse._notify_queue = queue.Queue() LLFUSE_VERSION_0 = llfuse.__version__.startswith('0') @@ -205,7 +214,7 @@ class InodeCache(object): def cap_cache(self): if self._total > self.cap: - for ent in self._entries.values(): + for ent in listvalues(self._entries): if self._total < self.cap or len(self._entries) < self.min_entries: break self._remove(ent, True) @@ -262,10 +271,10 @@ class Inodes(object): self._entries[key] = item def __iter__(self): - return self._entries.iterkeys() + return iter(self._entries.keys()) def items(self): - return self._entries.items() + return viewitems(self._entries.items()) def __contains__(self, k): return k in self._entries @@ -303,12 +312,12 @@ class Inodes(object): if entry.has_ref(False): # Only necessary if the kernel has previously done a lookup on this # inode and hasn't yet forgotten about it. - llfuse.invalidate_entry(entry.inode, name.encode(self.encoding)) + llfuse.invalidate_entry(entry.inode, native(name.encode(self.encoding))) def clear(self): self.inode_cache.clear() - for k,v in self._entries.items(): + for k,v in viewitems(self._entries): try: v.finalize() except Exception as e: @@ -526,7 +535,7 @@ class Operations(llfuse.Operations): entry.st_size = e.size() entry.st_blksize = 512 - entry.st_blocks = (entry.st_size/512)+1 + entry.st_blocks = (entry.st_size // 512) + 1 if hasattr(entry, 'st_atime_ns'): # llfuse >= 0.42 entry.st_atime_ns = int(e.atime() * 1000000000) @@ -567,7 +576,7 @@ class Operations(llfuse.Operations): @lookup_time.time() @catch_exceptions def lookup(self, parent_inode, name, ctx=None): - name = unicode(name, self.inodes.encoding) + name = str(name, self.inodes.encoding) inode = None if name == '.': @@ -715,8 +724,7 @@ class Operations(llfuse.Operations): # update atime self.inodes.touch(p) - - self._filehandles[fh] = DirectoryHandle(fh, p, [('.', p), ('..', parent)] + list(p.items())) + self._filehandles[fh] = DirectoryHandle(fh, p, [('.', p), ('..', parent)] + listitems(p)) return fh @readdir_time.time() @@ -772,6 +780,7 @@ class Operations(llfuse.Operations): @create_time.time() @catch_exceptions def create(self, inode_parent, name, mode, flags, ctx=None): + name = name.decode(encoding=self.inodes.encoding) _logger.debug("arv-mount create: parent_inode %i '%s' %o", inode_parent, name, mode) p = self._check_writable(inode_parent) @@ -789,6 +798,7 @@ class Operations(llfuse.Operations): @mkdir_time.time() @catch_exceptions def mkdir(self, inode_parent, name, mode, ctx=None): + name = name.decode(encoding=self.inodes.encoding) _logger.debug("arv-mount mkdir: parent_inode %i '%s' %o", inode_parent, name, mode) p = self._check_writable(inode_parent) @@ -803,6 +813,7 @@ class Operations(llfuse.Operations): @unlink_time.time() @catch_exceptions def unlink(self, inode_parent, name, ctx=None): + name = name.decode(encoding=self.inodes.encoding) _logger.debug("arv-mount unlink: parent_inode %i '%s'", inode_parent, name) p = self._check_writable(inode_parent) p.unlink(name) @@ -810,6 +821,7 @@ class Operations(llfuse.Operations): @rmdir_time.time() @catch_exceptions def rmdir(self, inode_parent, name, ctx=None): + name = name.decode(encoding=self.inodes.encoding) _logger.debug("arv-mount rmdir: parent_inode %i '%s'", inode_parent, name) p = self._check_writable(inode_parent) p.rmdir(name) @@ -817,6 +829,8 @@ class Operations(llfuse.Operations): @rename_time.time() @catch_exceptions def rename(self, inode_parent_old, name_old, inode_parent_new, name_new, ctx=None): + name_old = name_old.decode(encoding=self.inodes.encoding) + name_new = name_new.decode(encoding=self.inodes.encoding) _logger.debug("arv-mount rename: old_parent_inode %i '%s' new_parent_inode %i '%s'", inode_parent_old, name_old, inode_parent_new, name_new) src = self._check_writable(inode_parent_old) dest = self._check_writable(inode_parent_new)