diff options
author | Aravinda VK <avishwan@redhat.com> | 2016-09-06 12:02:02 +0530 |
---|---|---|
committer | Aravinda VK <avishwan@redhat.com> | 2016-09-13 04:58:09 -0700 |
commit | bc000f046cb7f3614b2b6b640176ae41e0fcb633 (patch) | |
tree | c8cacc9a5d148861e21354e1d4a25f2e260ce5ff /geo-replication | |
parent | f443a73b6088298da13a2ca1fee9db72215c8f84 (diff) |
geo-rep: Fix ESTALE/EINVAL issue during set_{xtime,stime}
Setfattr may get ESTALE/EINVAL if a file is being unlinked.
To prevent worker crashing, added retry for these error messages.
On second retry it will get ENOENT and that error is handled by
ignoring.
> Reviewed-on: http://review.gluster.org/15404
> Reviewed-by: Saravanakumar Arumugam <sarumuga@redhat.com>
> 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>
> Smoke: Gluster Build System <jenkins@build.gluster.org>
BUG: 1374628
Change-Id: Ic660fa13208366d57c8d3d492bbef611475e45b7
Signed-off-by: Aravinda VK <avishwan@redhat.com>
Reviewed-on: http://review.gluster.org/15451
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Smoke: Gluster Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Saravanakumar Arumugam <sarumuga@redhat.com>
Diffstat (limited to 'geo-replication')
-rw-r--r-- | geo-replication/syncdaemon/resource.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/geo-replication/syncdaemon/resource.py b/geo-replication/syncdaemon/resource.py index 6341de5ea79..b759ed665e9 100644 --- a/geo-replication/syncdaemon/resource.py +++ b/geo-replication/syncdaemon/resource.py @@ -545,7 +545,8 @@ class Server(object): [path, '.'.join([cls.GX_NSPACE, uuid, 'stime']), struct.pack('!II', *mark)], - [ENOENT]) + [ENOENT], + [ESTALE, EINVAL]) @classmethod @_pathguard @@ -555,7 +556,8 @@ class Server(object): [path, '.'.join([cls.GX_NSPACE, uuid, 'xtime']), struct.pack('!II', *mark)], - [ENOENT]) + [ENOENT], + [ESTALE, EINVAL]) @classmethod @_pathguard |