diff options
| author | Susant Palai <spalai@redhat.com> | 2016-05-23 12:09:10 +0530 |
|---|---|---|
| committer | Jeff Darcy <jdarcy@redhat.com> | 2016-05-25 07:19:27 -0700 |
| commit | 1a489f8f67a98644e6ebea652dcf96dd7425046e (patch) | |
| tree | c665ecd879d3cde16d784c1d3b5a2e3af224cc55 | |
| parent | ac2fa110ea489ca3d1b81e3872731fa1621a6e39 (diff) | |
dht/rebalance: mark hardlink failures as skipped in rebalance
Since rebalance(not remove-brick) process does not migrate hardlinks
mark them as skipped rather than failed as it creates confusion for
the users.
Change-Id: I5d469d10146274f00bb91482d0373c5235a9b8b2
BUG: 1339071
Signed-off-by: Susant Palai <spalai@redhat.com>
Reviewed-on: http://review.gluster.org/14493
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Smoke: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: N Balachandran <nbalacha@redhat.com>
| -rw-r--r-- | xlators/cluster/dht/src/dht-rebalance.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/xlators/cluster/dht/src/dht-rebalance.c b/xlators/cluster/dht/src/dht-rebalance.c index bf27e476c72..03c6d615d64 100644 --- a/xlators/cluster/dht/src/dht-rebalance.c +++ b/xlators/cluster/dht/src/dht-rebalance.c @@ -2018,6 +2018,14 @@ gf_defrag_migrate_single_file (void *opaque) defrag->skipped += 1; } UNLOCK (&defrag->lock); + } else if (op_errno == ENOTSUP) { + gf_msg_debug (this->name, 0, "migrate-data skipped for" + " hardlink %s ", entry_loc.path); + LOCK (&defrag->lock); + { + defrag->skipped += 1; + } + UNLOCK (&defrag->lock); } else if (op_errno != EEXIST) { gf_msg (this->name, GF_LOG_ERROR, 0, DHT_MSG_MIGRATE_FILE_FAILED, |