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

Re: PATCH: allow installing grub on the MBR if /boot is on mdraid.



----- "Hans de Goede" <hdegoede redhat com> wrote:

> Joel Granados wrote:
> > ----- "Hans de Goede" <hdegoede redhat com> wrote:
> > 
> >> Hi All,
> >>
> > 
> > But here we would be loosing the "RAID Device" message.  It would be
> replaced by "First sector of boot partition", which is not bad thing,
> but I would still prefer to tell the user that its a "RAID Device".
> > 
> > I would prefer to just add a line inside the if that adds the mbr
> and then returns.
> > 
> 
> Then we miss / skip all the special cases for PPC, what if someone
> wants to do 
> this on ppc ?
> 
> Also we are not installing grub to the RAID Device, we are installing
> grub to 
> the first sector of the first partition of the RAID set, so the "First
> sector 
> of boot partition" is arguably more correct.

^^^^ This being the case. I mean if we are not really using that raid device to do bootloader installation.  I *would* leave the the patch as you originally posted it.

> 
> Last I like patches which remove code better then those which add code
> :)
> 
> Regards,
> 
> H
> 
> >> diff --git a/fsset.py b/fsset.py
> >> index c9cd098..b726985 100644
> >> --- a/fsset.py
> >> +++ b/fsset.py
> >> @@ -1522,10 +1522,6 @@ MAILADDR root
> >>              ret['boot'] = (bootDev.device, N_("EFI System
> >> Partition"))
> >>              return ret
> >>  
> >>         if bootDev.getName() == "RAIDDevice":
> >>             ret['boot'] = (bootDev.device, N_("RAID Device"))
> > =======Add this line to the if:========
> >               ret['mbr'] = (bl.drivelist[0], N_("Master Boot Record
> (MBR)"))
> > 
> >>             return ret
> >>
> >>          if iutil.getPPCMacGen() == "NewWorld":
> >>              ret['boot'] = (bootDev.device, N_("Apple Bootstrap"))
> >>              n = 1
> >>
> > 
> > regards
> > 
> 
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list redhat com
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

-- 
Joel Andres Granados
Red Hat / Brno Czech Republic


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