Add yaboot.conf to the list of file for disk migration.

[schabrolles]

yaboot.conf is used by RHEL < 7 in POWER system.

Like lilo.conf, the variable boot= must be updated in case of disk migration.

We should also add lilo.conf & yaboot.conf files to CHECK_CONFIG_FILES. What do you think ?

[jsmeix]

The current change if fine for me
and I agree to also add those files
to CHECK_CONFIG_FILES.

[schabrolles]

@jsmeix Do I have to update CHECK_CONFIG_FILES directly in conf/default.conf ?

[jsmeix]

I think technically you could add lilo.conf and yaboot.conf files
to CHECK_CONFIG_FILES directly in default.conf
i.e. I think nothing should break if you do it this way.

But I think it would be cleaner code to add specific files
to CHECK_CONFIG_FILES via a specific script
only when really needed.

I.e. I would prefer to add PPC architecture-specific files
to CHECK_CONFIG_FILES via a PPC specific script.

In general see for example
layout/save/default/450_check_bootloader_files.sh
how specific files can be added to CHECK_CONFIG_FILES.

Interestingly layout/save/default/450_check_bootloader_files.sh
already contains (excerpts):

case $used_bootloader in
...
    (PPC)
        CHECK_CONFIG_FILES=( ${CHECK_CONFIG_FILES[@]} /etc/lilo.conf /etc/yaboot.conf)
        ;;

which seems to already do that.

@schabrolles
does layout/save/default/450_check_bootloader_files.sh
perhaps not work in practice for you?

[schabrolles]

@jsmeix Ok for me...
The only thing is :
(PPC) bootloader could be :

• lilo => SLES < 12
• yaboot => RHEL < 7
• grub2 => Ubuntu, RHEL >= 7, SLES >=12

So, We may be need to update this script with :

case $used_bootloader in
...
    (PPC)
        CHECK_CONFIG_FILES=( ${CHECK_CONFIG_FILES[@]} /etc/lilo.conf /etc/yaboot.conf /etc/grub.cfg /etc/grub2.cfg /boot/grub2/grub2.cfg /boot/grub/grub.cfg)
        ;;

[jsmeix]

@schabrolles
could you update
layout/save/default/450_check_bootloader_files.sh
plus a comment that explains why for PPC so many different
bootloader config files are added to CHECK_CONFIG_FILES
so that later also others understand the reason behind the code.