From: Milan Broz <mbroz@redhat.com>
To: lvm-devel@redhat.com
Subject: [PATCH 2/7] Never set mirror log and images directly visible in metadata.
Date: Sun, 10 May 2009 20:42:53 +0200 [thread overview]
Message-ID: <4A07202D.1010903@redhat.com> (raw)
In-Reply-To: <87my9ku7t9.fsf@eriador.mornfall.net>
Petr Rockai wrote:
> Milan Broz <mbroz@redhat.com> writes:
>> Mirror creation and conversion force set mirror log and images
>> to be visible. This can violate VG max_lv restriction on LV.
>>
>> There is no need to set these volumes visible, even during
>> mirror images conversion.
> Well, I believe the use-case is that when the operation fails halfway (which
> happens from time to time -- more often than would be comfortable, even), you
> can clean up by hand, since those offending LVs became visible. If there is a
> reasonable procedure that the admins can follow to get rid of the junk LVs in
> those scenarios, I'm fine.
Maybe we should display these "invisible" LVs to user (in reporting commands)
if no visible LV references them (I mean _mimage, _mlog & Co. devices)
Admin can remove these insvible LV even now (lvremove works), but must
run "lvs -a" to found them first...
Milan
next prev parent reply other threads:[~2009-05-10 18:42 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-06 14:42 [PATCH 0/7] Fix lv_count & max_lv problems in lvm2 Milan Broz
2009-05-06 14:42 ` [PATCH 1/7] Fix snapshot segment import to not use duplicate segments & replace Milan Broz
2009-05-11 6:27 ` Petr Rockai
2009-05-11 11:29 ` Alasdair G Kergon
2009-05-06 14:42 ` [PATCH 2/7] Never set mirror log and images directly visible in metadata Milan Broz
2009-05-10 18:33 ` Petr Rockai
2009-05-10 18:42 ` Milan Broz [this message]
2009-05-10 19:24 ` Petr Rockai
2009-05-11 1:30 ` Alasdair G Kergon
2009-05-11 1:39 ` Alasdair G Kergon
2009-05-11 1:54 ` Alasdair G Kergon
2009-05-12 12:34 ` Milan Broz
2009-05-06 14:43 ` [PATCH 3/7] Remove snapshot_count from VG and use function instead Milan Broz
2009-05-10 18:41 ` Petr Rockai
2009-05-11 11:48 ` Alasdair G Kergon
2009-05-11 11:42 ` Alasdair G Kergon
2009-05-06 14:43 ` [PATCH 4/7] Introduce vg_add_lc and vg_remove_lv functions Milan Broz
2009-05-10 19:06 ` Petr Rockai
2009-05-12 13:37 ` Milan Broz
2009-05-06 14:43 ` [PATCH 5/7] Introduce lv_set_visible & lv_set_hidden and use lv_is_visible always Milan Broz
2009-05-10 19:23 ` Petr Rockai
2009-05-12 15:12 ` Milan Broz
2009-05-06 14:43 ` [PATCH 6/7] Merge lv_is_displayable and lv_is_visible Milan Broz
2009-05-11 6:09 ` Petr Rockai
2009-05-06 14:43 ` [PATCH 7/7] Remove import paramater from lv_create_empty Milan Broz
2009-05-11 6:15 ` Petr Rockai
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A07202D.1010903@redhat.com \
--to=mbroz@redhat.com \
--cc=lvm-devel@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.