gluster-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Gluster-devel] layout is missing when add-brick is done, new created fi


From: Jiale Gao
Subject: [Gluster-devel] layout is missing when add-brick is done, new created files only locate on old bricks
Date: Wed, 05 Mar 2014 09:50:30 +0800
User-agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0

Hi all,
Root's layout is missing when add-brick id done,and the new created files only locate on old bricks.
    See the bug:https://bugzilla.redhat.com/show_bug.cgi?id=1070573
I want to kown why the layout->list[i].err in dht_layout_merge is set to 0 if there if a missing disk layout.Whats more,Should we detect missing layout in dht_layout_anomalies to enter selfheal directory?
    So my change is the next:
diff --git a/xlators/cluster/dht/src/dht-layout.c b/xlators/cluster/dht/src/dht-layout.c
index 31d85a5..fe25568 100644
--- a/xlators/cluster/dht/src/dht-layout.c
+++ b/xlators/cluster/dht/src/dht-layout.c
@@ -366,7 +366,6 @@ dht_layout_merge (xlator_t *this, dht_layout_t *layout, xlator_t *subvol,
         }

         if (ret != 0) {
-                layout->list[i].err = 0;
                 gf_log (this->name, GF_LOG_TRACE,
                         "missing disk layout on %s. err = %d",
                         subvol->name, err);
@@ -556,7 +555,7 @@ dht_layout_anomalies (xlator_t *this, loc_t *loc, dht_layout_t *layout,
                 case ENOENT:
                 case ESTALE:
                         missing++;
-                        continue;
+                        break;
                 case ENOTCONN:
                         down++;
                         continue;
Gao

--
With kind regards,
Jiale Gao
E-mail:address@hidden




reply via email to

[Prev in Thread] Current Thread [Next in Thread]