Re: [PATCH] pvremove *really* means pvremove!

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

 



On Tue, Mar 10, 2009 at 10:50:03AM -0500, David Lehman wrote:
I'm going to got through some lvm testing to see if I can avoid having
to do this.  Wont commit untill I am satisfied that I is most likely the
only or the easiest answer to the issue.
> I don't like that we must do this, but we apparently do so... Ack.
> 
> On Mon, 2009-03-09 at 17:36 +0100, Joel Granados Moreno wrote:
> > * storage/devicelibs/lvm.py (pvremove): Make sure to erase the PVs with
> >   arguments "-ff"" and "-y".  The first one is to force the action, the
> >   other is to make sure lmv does ask for user interaction.
> > ---
> >  storage/devicelibs/lvm.py |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/storage/devicelibs/lvm.py b/storage/devicelibs/lvm.py
> > index 0faee10..bdb732a 100644
> > --- a/storage/devicelibs/lvm.py
> > +++ b/storage/devicelibs/lvm.py
> > @@ -130,7 +130,7 @@ def pvresize(device, size):
> >  
> >  def pvremove(device):
> >      rc = iutil.execWithRedirect("lvm",
> > -                                ["pvremove", device],
> > +                                ["pvremove","-ff", "-y", device],
> >                                  stdout = "/dev/tty5",
> >                                  stderr = "/dev/tty5",
> >                                  searchPath=1)
> 
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

-- 
Joel Andres Granados
Brno, Czech Republic, Red Hat.

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux