systemd: lvm2-activation-generator: report only error otherwise be silent

Do not print success status for lvm2-activation-generator:

  "LVM: Activation generator successfully completed."
  "LVM: Logical Volume autoactivation enabled." (if use_lvmetad=1)

Though this information is quite useful during boot, it may
be confusing for users if it happens anytime later and it
actually happens if systemd reloads. This is usually on package
update to update the systemd state and load any new units that are
newly installed in the system. The systemd reload is global and
so any existing generators are rerun at that moment too.

diff --git a/WHATS_NEW b/WHATS_NEW
index 6c0514f2c5bc48b80ab2a286f0d43194618a9d45..76c2ebbaac2a6f3f98cdd669b3052631cdc2e89f 100644 (file)
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,5 +1,6 @@
 Version 2.02.101 - 
 ===================================
+  Make lvm2-activation-generator silent unless it's in error state.
   Remove "mpath major is not dm major" msg for mpath component scan (2.02.94).
   Prevent cluster mirror logs from being corrupted by redundant checkpoints.
   Fix ignored lvmetad update on loop device configuration (2.02.99).

diff --git a/scripts/lvm2_activation_generator_systemd_red_hat.c b/scripts/lvm2_activation_generator_systemd_red_hat.c
index 9d4b581c04ff4d20eeb3ff527692946a6cea25cc..9cec0a9b75f89de5012401b84c12082f6084d9f8 100644 (file)
--- a/scripts/lvm2_activation_generator_systemd_red_hat.c
+++ b/scripts/lvm2_activation_generator_systemd_red_hat.c
@@ -180,10 +180,8 @@ int main(int argc, char *argv[])
        }
 
        /* If lvmetad used, rely on autoactivation instead of direct activation. */
-       if (lvm_uses_lvmetad()) {
-               kmsg(LOG_DEBUG, "LVM: Logical Volume autoactivation enabled.\n");
+       if (lvm_uses_lvmetad())
                goto out;
-       }
 
        dir = argc > 1 ? argv[1] : DEFAULT_UNIT_DIR;
 
@@ -192,8 +190,8 @@ int main(int argc, char *argv[])
            !generate_unit(dir, UNIT_NET))
                r = EXIT_FAILURE;
 out:
-       kmsg(r ? LOG_ERR : LOG_DEBUG, "LVM: Activation generator %s.\n",
-            r ? "failed" : "successfully completed");
+       if (r)
+               kmsg(LOG_ERR, "LVM: Activation generator failed.\n");
        if (kmsg_fd != -1)
                (void) close(kmsg_fd);
        return r;