diff options
author | Aravinda VK <avishwan@redhat.com> | 2016-09-06 12:02:02 +0530 |
---|---|---|
committer | Aravinda VK <avishwan@redhat.com> | 2016-09-14 23:30:02 -0700 |
commit | 8ec42734d425328f8b8ea44054fc45a91dba9588 (patch) | |
tree | d207699ea5ee033da5a4dc6fb8a45f6779453349 /geo-replication | |
parent | bc1a6900be5ba9834c3744ae454f8beee242c1d4 (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: 1374626
Change-Id: Ic660fa13208366d57c8d3d492bbef611475e45b7
Signed-off-by: Aravinda VK <avishwan@redhat.com>
(cherry picked from commit 9db2826cd8dbcc0fb58f5434c21d00ca5c15c491)
Reviewed-on: http://review.gluster.org/15445
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: Saravanakumar Arumugam <sarumuga@redhat.com>
Diffstat (limited to 'geo-replication')
-rw-r--r-- | geo-replication/syncdaemon/resource.py | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/geo-replication/syncdaemon/resource.py b/geo-replication/syncdaemon/resource.py index 0223bcdb633..c66ded6e4a2 100644 --- a/geo-replication/syncdaemon/resource.py +++ b/geo-replication/syncdaemon/resource.py @@ -569,7 +569,8 @@ class Server(object): [path, '.'.join([cls.GX_NSPACE, uuid, 'stime']), struct.pack('!II', *mark)], - [ENOENT]) + [ENOENT], + [ESTALE, EINVAL]) @classmethod @_pathguard @@ -579,7 +580,8 @@ class Server(object): [path, '.'.join([cls.GX_NSPACE, uuid, 'entry_stime']), struct.pack('!II', *mark)], - [ENOENT]) + [ENOENT], + [ESTALE, EINVAL]) @classmethod @_pathguard @@ -589,7 +591,8 @@ class Server(object): [path, '.'.join([cls.GX_NSPACE, uuid, 'xtime']), struct.pack('!II', *mark)], - [ENOENT]) + [ENOENT], + [ESTALE, EINVAL]) @classmethod @_pathguard |