diff options
| author | shishir gowda <sgowda@redhat.com> | 2013-07-11 13:05:55 +0530 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2013-07-11 17:43:03 -0700 |
| commit | 60d1949b00fa42e0c5d1f0a763004ca474a4645d (patch) | |
| tree | c1f00811d51b8981844970b3c36624478332a2b0 | |
| parent | 5bb136c4ca18cc4c058040ea6db312be13edb098 (diff) | |
cluster/dht: If linkfile unlink fails with ENOTCONN, do not fail
Currently if linkfile fails with ENOENT, we do not fail. We also
need to treat failures with ENOTCONN as success, as if cached subvol
is up, rm of a file should succeed. A stale linkfile will get removed
later
Change-Id: I71d136847933351ed9e2c939bda4a69bc96a3cfc
BUG: 983416
Signed-off-by: shishir gowda <sgowda@redhat.com>
Reviewed-on: http://review.gluster.org/5317
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anand Avati <avati@redhat.com>
| -rw-r--r-- | xlators/cluster/dht/src/dht-common.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/xlators/cluster/dht/src/dht-common.c b/xlators/cluster/dht/src/dht-common.c index e55c2543817..c7bbfd160f7 100644 --- a/xlators/cluster/dht/src/dht-common.c +++ b/xlators/cluster/dht/src/dht-common.c @@ -1645,7 +1645,8 @@ dht_unlink_linkfile_cbk (call_frame_t *frame, void *cookie, xlator_t *this, LOCK (&frame->lock); { - if ((op_ret == -1) && (op_errno != ENOENT)) { + if ((op_ret == -1) && !((op_errno == ENOENT) || + (op_errno == ENOTCONN))) { local->op_errno = op_errno; gf_log (this->name, GF_LOG_DEBUG, "subvolume %s returned -1 (%s)", |