Replicator: update activate code for vgchange

Activate only the first replicator-dev LV, that activates all other
related LVs from Replicator. In case of error during this activation,
it will not retry again for other heads (less confusing error log).

diff --git a/WHATS_NEW b/WHATS_NEW
index 9f4ce4e02ecba0f91bf6867bf0dbefff0d9af5d3..4541874efc6f39ded51ac1c172163d46488400aa 100644 (file)
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,5 +1,6 @@
 Version 2.02.67 -
 ===============================
+  Activate only first head of Replicator for vgchange -ay.
   Add Replicators' LVs to dtree for activation.
   Avoid print activation message if there is a missing VG (Replicator).
   Fixed scripts/relpath.awk to work in mawk

diff --git a/tools/vgchange.c b/tools/vgchange.c
index 4f27eea972188a48686615f970a611af21c6d28a..e2fbaeb31e2d5763acd715f97b0bd8b8de31b9e1 100644 (file)
--- a/tools/vgchange.c
+++ b/tools/vgchange.c
@@ -104,6 +104,11 @@ static int _activate_lvs_in_vg(struct cmd_context *cmd,
                if ((lv->status & MIRROR_IMAGE) || (lv->status & MIRROR_LOG))
                        continue;
 
+               /* Only request activation of the first replicator-dev LV */
+               /* Avoids retry with all heads in case of failure */
+               if (lv_is_replicator_dev(lv) && (lv != first_replicator_dev(lv)))
+                       continue;
+
                /* Can't deactivate a pvmove LV */
                /* FIXME There needs to be a controlled way of doing this */
                if (((activate == CHANGE_AN) || (activate == CHANGE_ALN)) &&