[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [lvm-devel] [PATCH] Enable (already implemented?) snapshots of mirrors



On Thu, Oct 15, 2009 at 03:19:08PM -0500, Jon Brassow wrote:
> I just wanted to get a feel for where things were at WRT snapshots of
> mirrors.  Pulling out the prevention check (patch provided below),
> things seem to work just fine.  Is there something I am missing?
 
Well the check was inserted because the code was untested.

All the newly-possible combinations of commands need to be tested before
it can be removed, and other 'unsupported' messages can be placed
where things don't work correctly.

(Basically you check through all the options of all the commands
one-by-one and test every aspect which might behave differently
in respect of snapshots of mirrors.  This includes checking the dmeventd
plugins as well as 'partial' VGs, and ensuring there are no ways of
manipulating the 'in-use' mirror that conflict with the fact it is
an origin, and ensuring display tools remain comprehensible.)

Alasdair


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]