[rhel7/master] Skip nvram update on ppc64 image/dir installations (#1136486)

Brian C. Lane bcl at redhat.com
Wed Sep 3 15:34:08 UTC 2014


On Wed, Sep 03, 2014 at 12:55:16PM +0200, Vratislav Podzimek wrote:
> On Tue, 2014-09-02 at 12:20 -0700, Brian C. Lane wrote:
> > We don't want to modify the host's nvram.
> > 
> > Resolves: rhbz#1136486
> > ---
> >  pyanaconda/bootloader.py | 6 ++++++
> >  1 file changed, 6 insertions(+)
> > 
> > diff --git a/pyanaconda/bootloader.py b/pyanaconda/bootloader.py
> > index 839ebd9..6995214 100644
> > --- a/pyanaconda/bootloader.py
> > +++ b/pyanaconda/bootloader.py
> > @@ -1943,6 +1943,9 @@ class IPSeriesYaboot(Yaboot):
> >          super(IPSeriesYaboot, self).install()
> >  
> >      def updatePowerPCBootList(self):
> > +        if flags.imageInstall or flags.dirInstall:
> > +            log.info("Skipping updatePowerPCBootList for image/directory install.")
> > +            return
> >  
> >          log.debug("updatePowerPCBootList: self.stage1_device.path = %s", self.stage1_device.path)
> >  
> > @@ -2003,6 +2006,9 @@ class IPSeriesGRUB2(GRUB2):
> >  
> >      # This will update the PowerPC's (ppc) bios boot devive order list
> >      def updateNVRAMBootList(self):
> > +        if flags.imageInstall or flags.dirInstall:
> > +            log.info("Skipping updateNVRAMBootList for image/directory install.")
> > +            return
> >  
> >          log.debug("updateNVRAMBootList: self.stage1_device.path = %s", self.stage1_device.path)
> What about using can_touch_runtime_system(touch_live=True) here? I know
> we probably don't have to care about live installations on PPC, but who
> knows.

Bah. I always seem to forget we have the touch_live flag. I'll switch
to that. Thanks!

-- 
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)


More information about the anaconda-patches mailing list