summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAravinda VK <avishwan@redhat.com>2016-09-06 12:02:02 +0530
committerAravinda VK <avishwan@redhat.com>2016-09-13 04:58:09 -0700
commitbc000f046cb7f3614b2b6b640176ae41e0fcb633 (patch)
treec8cacc9a5d148861e21354e1d4a25f2e260ce5ff
parentf443a73b6088298da13a2ca1fee9db72215c8f84 (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>
-rw-r--r--geo-replication/syncdaemon/resource.py6
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