Allow empty activation/{auto_activation|read_only|}_volume_list config option.

In case we don't want to activate, autoactivate or have the
VG/LV read-only. Primarily targeted for the auto_activation_volume_list,
but it makes no harm for other settings (the part of the code
that reads these three settings is shared, but there's no
reason to separate it only for this change).

diff --git a/WHATS_NEW b/WHATS_NEW
index cff005df110ef4ca4dc8daf6d7c89a7f1c4d73b5..519d4ddc7f428b1e6ab2ce641b014d986c30bd8f 100644 (file)
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,5 +1,6 @@
 Version 2.02.99 - 
 ===================================
+  Allow empty activation/{auto_activation|read_only|}_volume_list config option.
   Add lvm.conf option global/thin_disabled_features.
   Add lvconvert support to swap thin pool metadata volume.
   Implement internal function detach_pool_metadata_lv().

diff --git a/lib/activate/activate.c b/lib/activate/activate.c
index 185ba5fe301aed529f66838d2f6335534b889b64..ee552b4846580cf7b169f408653a2d50c73bb8bf 100644 (file)
--- a/lib/activate/activate.c
+++ b/lib/activate/activate.c
@@ -340,6 +340,8 @@ static int _lv_passes_volumes_filter(struct cmd_context *cmd, struct logical_vol
                    config_path, lv->vg->name, lv->name);
 
        for (cv = cn->v; cv; cv = cv->next) {
+               if (cv->type == DM_CFG_EMPTY_ARRAY)
+                       return 0;
                if (cv->type != DM_CFG_STRING) {
                        log_error("Ignoring invalid string in config file %s",
                                  config_path);