summaryrefslogtreecommitdiff
path: root/lib/format1
diff options
context:
space:
mode:
authorAlasdair G Kergon <agk@redhat.com>2014-01-14 03:27:45 +0000
committerAlasdair G Kergon <agk@redhat.com>2014-01-14 03:27:45 +0000
commit3813cd7a3c3e763c346405664fba97edf7110d13 (patch)
treefe509f82e1df1a56d2afb0e1eee7694f56d797e4 /lib/format1
parent5a450eab6a9fef7793066864ff58857d827b903f (diff)
downloadlvm2-3813cd7a3c3e763c346405664fba97edf7110d13.tar.gz
format1: Mark obsolete and do not use with lvmetad.
DO NOT USE LVMETAD IF YOU HAVE ANY LVM1-FORMATTED PVS. You may continue to use it without lvmetad, but do please schedule an upgrade to the lvm2 format (with 'vgconvert'). Sending the original LVM1 formatted metadata to lvmetad is breaking assumptions made by the code, so I am marking the format as obsolete for now and no longer sending it to lvmetad. This means that if you are using lvmetad, lvm1 volumes will usually appear invisible - though not always: it depends on exactly what sequence of commands you run! The current situation is not satisfactory. We'll either fix lvmetad and reenable this or we'll fix the code to issue appropriate warning messages when lvm1 PVs are encountered to avoid accidents. (The latest unfixed problem is that lvmetad assumes metadata sequence numbers exist and always increase - but the lvm1 format does not define or store any sequence number, confusing both the daemon and client when default values get passed to-and-fro.)
Diffstat (limited to 'lib/format1')
-rw-r--r--lib/format1/format1.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/format1/format1.c b/lib/format1/format1.c
index 509167a56..16746e3fc 100644
--- a/lib/format1/format1.c
+++ b/lib/format1/format1.c
@@ -603,7 +603,7 @@ struct format_type *init_format(struct cmd_context *cmd)
fmt->alias = NULL;
fmt->orphan_vg_name = FMT_LVM1_ORPHAN_VG_NAME;
fmt->features = FMT_RESTRICTED_LVIDS | FMT_ORPHAN_ALLOCATABLE |
- FMT_RESTRICTED_READAHEAD;
+ FMT_RESTRICTED_READAHEAD | FMT_OBSOLETE;
fmt->private = NULL;
dm_list_init(&fmt->mda_ops);