summaryrefslogtreecommitdiff
path: root/conf
diff options
context:
space:
mode:
authorPeter Rajnoha <prajnoha@redhat.com>2015-07-14 10:03:19 +0200
committerPeter Rajnoha <prajnoha@redhat.com>2015-07-14 10:13:41 +0200
commitac3143c0936d00a0a1f4a865192040d3791afd66 (patch)
treee519e9e22d1df24694d7df2d880c79302c6d8b36 /conf
parentd41bab402805c8813bb8eb0b18f3da97bb79fdf7 (diff)
downloadlvm2-ac3143c0936d00a0a1f4a865192040d3791afd66.tar.gz
config: {thin,cache}_{check,repair}_options are never undefined
Require global/{thin,cache}_{check,repair}_options to be always defined. If not defined directly by user in the configuration and if there's no concrete default option to use, make "" (empty string) the default one - it's then clearly visible in the "lvmconfig --type default" (and generated lvm.conf) and also it makes its handling in the code more straightforward so we don't need to handle undefined values. This means, if there are no default values for these settings defined, we end up with this generated now: {thin,cache}_{check,repair}_options = [ "" ] So the value is never undefined and if it is, it's an error. (The cache_repair_options is actually not used in the code at the moment, but once the code using this setting is in, it will follow the same logic as used for thin_repair_options.)
Diffstat (limited to 'conf')
-rw-r--r--conf/example.conf.in4
1 files changed, 2 insertions, 2 deletions
diff --git a/conf/example.conf.in b/conf/example.conf.in
index 320795566..3900ad41b 100644
--- a/conf/example.conf.in
+++ b/conf/example.conf.in
@@ -876,7 +876,7 @@ global {
# Configuration option global/thin_repair_options.
# List of options passed to the thin_repair command.
- # This configuration option does not have a default value defined.
+ # thin_repair_options = [ "" ]
# Configuration option global/thin_disabled_features.
# Features to not use in the thin driver.
@@ -924,7 +924,7 @@ global {
# Configuration option global/cache_repair_options.
# List of options passed to the cache_repair command.
- # This configuration option does not have a default value defined.
+ # cache_repair_options = [ "" ]
# Configuration option global/system_id_source.
# The method LVM uses to set the local system ID.