PR#4: Improve no-bootloader support -

appliance-tools

#4 Improve no-bootloader support

Closed Nov 15, 2018 by davidlt. Opened Jun 13, 2018 by davidlt.

davidlt/appliance-tools no-bootloader into master

David Abdurachmanov • Jun 13, 2018

b669c3c29cc35ea0852d09965c13b2582f968aca

davidlt commented Jun 13, 2018

In Fedora/RISC-V case kernel and bootloader (BBL - Berkeley Boot Loader)
are external to the image. The image only delivers rootfs. Thus our
kickstart has bootloader --location=none --disabled.

If user disabled bootloader:
- Allow user to pick partition layout (i.e. do not ignore it).
- Do not write GRUB Legacy config

The changes were successfully used for Fedora RISC-V (riscv64) disk
images.

Signed-off-by: David Abdurachmanov david.abdurachmanov@gmail.com

ngompa commented Jun 17, 2018

This looks good to me, but was this tested to work fine for architectures where we have the bootloader internal to the image (for example x86_64)?

Also, could you please rebase on top of current master?

ngompa commented Nov 14, 2018

I'm nearing release of the next version of appliance-tools, to release the new Python 3 support. Can you please rebase this and make sure it works with current master on Python 3?

davidlt commented Nov 15, 2018

See: https://pagure.io/appliance-tools/pull-request/6

Pull-Request has been closed by davidlt

Nov 15, 2018