diff options
| author | Aravinda VK <avishwan@redhat.com> | 2016-05-25 11:56:56 +0530 | 
|---|---|---|
| committer | Aravinda VK <avishwan@redhat.com> | 2016-06-20 23:05:29 -0700 | 
| commit | a9e77087b7f00a5cb40d6ca466a5ac6d6ccfc93b (patch) | |
| tree | 7b0b27a7d5e8265cd57d39cb6fc35ad100afff2d /geo-replication | |
| parent | d962f51fd840b5997fb5ee3a2596f28fb52e72c4 (diff) | |
geo-rep: Handle stime/xtime set failures
While setting stime/xtime, if the file or directory is already
deleted then Geo-rep will crash with ENOENT.
With this patch, Geo-rep will ignores ENOENT since stime/xtime can't
be applied on a deleted file/directory.
Change-Id: I2d90569e51565f81ae53fcb23323e4f47c9e9672
Signed-off-by: Aravinda VK <avishwan@redhat.com>
BUG: 1345883
Reviewed-on: http://review.gluster.org/14529
(cherry picked from commit 1a348bfaeb9f2a50ec8ce27e5477e9b430c58b3c)
Reviewed-on: http://review.gluster.org/14714
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Kotresh HR <khiremat@redhat.com>
Diffstat (limited to 'geo-replication')
| -rw-r--r-- | geo-replication/syncdaemon/resource.py | 16 | 
1 files changed, 10 insertions, 6 deletions
diff --git a/geo-replication/syncdaemon/resource.py b/geo-replication/syncdaemon/resource.py index 27f7e84a2ce..bc7b13c95bc 100644 --- a/geo-replication/syncdaemon/resource.py +++ b/geo-replication/syncdaemon/resource.py @@ -534,17 +534,21 @@ class Server(object):      @_pathguard      def set_stime(cls, path, uuid, mark):          """set @mark as stime for @uuid on @path""" -        Xattr.lsetxattr( -            path, '.'.join([cls.GX_NSPACE, uuid, 'stime']), -            struct.pack('!II', *mark)) +        errno_wrap(Xattr.lsetxattr, +                   [path, +                    '.'.join([cls.GX_NSPACE, uuid, 'stime']), +                    struct.pack('!II', *mark)], +                   [ENOENT])      @classmethod      @_pathguard      def set_xtime(cls, path, uuid, mark):          """set @mark as xtime for @uuid on @path""" -        Xattr.lsetxattr( -            path, '.'.join([cls.GX_NSPACE, uuid, 'xtime']), -            struct.pack('!II', *mark)) +        errno_wrap(Xattr.lsetxattr, +                   [path, +                    '.'.join([cls.GX_NSPACE, uuid, 'xtime']), +                    struct.pack('!II', *mark)], +                   [ENOENT])      @classmethod      @_pathguard  | 
