Revert "libdm: trace missing settings"

This reverts commit 8fd886f735125de4b3e32fc0a188af0192a0d173.

This was a deliberate omission because logging token-by-token metadata
parsing greatly increases the amount of logging for hardly any benefit.

In general, only LVM config file settings need to be logged, and in
places where it's considered important to log particular elements of
metadata that should be done using specific log_* lines.

This area can be revisited.

diff --git a/WHATS_NEW_DM b/WHATS_NEW_DM
index 165cb90ad852dff47d54c57452ae89185c9c7a71..d4469970c91e58c6bfc1c38ed821a7db238312f5 100644 (file)
--- a/WHATS_NEW_DM
+++ b/WHATS_NEW_DM
@@ -12,7 +12,6 @@ Version 1.02.124 - 30th April 2016
 
 Version 1.02.123 - 23rd April 2016
 ==================================
-  Trace more settings in very verbose mode for libdm-config.
   Do not strip LVM- when debug reporting not found uuid.
 
 Version 1.02.122 - 9th April 2016

diff --git a/libdm/libdm-config.c b/libdm/libdm-config.c
index 86955c51d01ff67f9e77f058ec7911f79f9f1633..dcb7c3f81a7cf7cd084381cc47c171af5bae08b1 100644 (file)
--- a/libdm/libdm-config.c
+++ b/libdm/libdm-config.c
@@ -1154,11 +1154,8 @@ int dm_config_get_uint32(const struct dm_config_node *cn, const char *path,
        if (!n || !n->v || n->v->type != DM_CFG_INT)
                return 0;
 
-       if (result) {
-               log_very_verbose("Setting %s to %u", path, (uint32_t) n->v->v.i);
+       if (result)
                *result = n->v->v.i;
-       }
-
        return 1;
 }
 
@@ -1172,11 +1169,8 @@ int dm_config_get_uint64(const struct dm_config_node *cn, const char *path,
        if (!n || !n->v || n->v->type != DM_CFG_INT)
                return 0;
 
-       if (result) {
-               log_very_verbose("Setting %s to " FMTu64 , path, (uint64_t) n->v->v.i);
+       if (result)
                *result = (uint64_t) n->v->v.i;
-       }
-
        return 1;
 }
 
@@ -1190,11 +1184,8 @@ int dm_config_get_str(const struct dm_config_node *cn, const char *path,
        if (!n || !n->v || n->v->type != DM_CFG_STRING)
                return 0;
 
-       if (result) {
-               log_very_verbose("Setting %s to %s", path, n->v->v.str);
+       if (result)
                *result = n->v->v.str;
-       }
-
        return 1;
 }