Add support of bootloaders that follows BLS

[aplanas]

No description provided.

[aplanas]

Depends on openSUSE/sdbootutil#92

[aplanas]

cc @lnussel

[mwilck]

So this supersedes #101? To my understanding, #101 was still an RFC.

[mwilck]

For reasons unknown, the obs_scm server-side service failed, showing this package as broken. osc rebuild doesn't help. Retriggering the services doesn't help, either. The other current PRs (#101, #102, #104) don't exhibit this problem.

@aplanas, maybe just re-push, perhaps after applying a trivial commit msg change?

[mwilck]

Both patches are rather large. @aplanas and @lnussel, please indicate whether #101 should be closed in favor of this one, and confirm that you've done at least basic testing.

[aplanas]

So this supersedes #101? To my understanding, #101 was still an RFC.

Yes, it does.

[aplanas]

@aplanas, maybe just re-push, perhaps after applying a trivial commit msg change?

Re-pushed

[aplanas]

Both patches are rather large. @aplanas and @lnussel, please indicate whether #101 should be closed in favor of this one, and confirm that you've done at least basic testing.

The #101 commit is included here, and later changed and updated by the second one. #101 can be ignored.

[mwilck]

Re-pushed

Hm, weird, the OBS project is still broken.

[aplanas]

can the issue be reproduced locally?

[aplanas]

I squashed the commit jic

[mwilck]

Hm the OBS project is hosed somehow, no idea why. I'll try to fix it manually.

[mwilck]

@aplanas, the problem is that you changed the revision in the _service file to your suse-module-tools clone, and your github clone doesn't have any tags:

$ git ls-remote
From https://github.com/aplanas/suse-module-tools
2e7447774e720050841ea03eb9f8447191eeffde	HEAD
5ba5b2324176ec88db514bad2f1abb19207d2ab5	refs/heads/fix_sdboot
2e7447774e720050841ea03eb9f8447191eeffde	refs/heads/master

If you just push the 16.0.43 tag to github.com/aplanas/suse-module-tools, the OBS build should be fine.

[aplanas]

rebased and pushed 16.0.43 tag

[mwilck]

rebased and pushed 16.0.43 tag

Yep, build succeeds now. Thanks!

[aplanas]

Also depends on: openSUSE/transactional-update#122

@mwilck I changed the code a bit in the regenerate initrd post-transaction script. It is using now a new candidate feature for tukit (plugins), that will avoid the use of /dev/shm.

This tukit extension has not been merged, but what do you think about the rest of the changes?

[mwilck]

Sorry I haven't had time for this yet. It's high on my todo list but I'm busy with lots of bug reports.

[aplanas]

Update the code. In rpm-script the sdbootutil detection checks also that there is no bootloader installed. If this is the case assume that sdbootutil will be installed.

This is of use when generating an image, as when the kernel is installed there is still no bootloader in /boot

[mwilck]

Hi Alberto, thanks! I've got a few remarks.

[aplanas]

@mwilck I pushed a new commit with the changes for review.

[aplanas]

ping?

[aplanas]

@mwilck should we continue with the review?

[TobiPeterG]

I think this would fix openSUSE/sdbootutil#22

[mwilck]

LGTM, except for the script_rc treatment (which you can revert to the 1st version of the patch set).

When this is done, I'll give the set a final review pass, and merge.

[aplanas]

I will squash the commits once all is reviewed

[mwilck]

LGTM.

[aplanas]

squashed