On Fri, Oct 01, 2010 at 04:30:05PM +0200, Michael Hanselmann wrote: > Am 1. Oktober 2010 15:31 schrieb Iustin Pop <[email protected]>: > > On Fri, Oct 01, 2010 at 03:19:33PM +0200, Michael Hanselmann wrote: > >> This will be used by LUXI client programs to display the VG name. > >> > > > > Ah, only now I see this is needed for 2/2. Mmm… it's a hardcoding of the > > fact that we use *one* VG group. > > > > What do you think about simply fixing the LUs in the first place to > > return complete LV info? > > That would change the return value of OpVerifyDisks and we should > avoid that in the middle of the 2.2 series. I agree for 2.3, but not > 2.2. The latter can use this work-around as it won't get multi-VG > support.
LGTM for 2.2, but yes, we need to fix this better in 2.3… iustin
