Ensure adding Storage Support Software Packages for MicroOS [master]

[shundhammer]

Target Branch / Product

This is the merge to master / Factory / TW of #1350 .

Bugzilla

https://bugzilla.suse.com/show_bug.cgi?id=1212452

Trello

https://trello.com/c/T9nqET80/

Problem

On an system with a multipath storage setup, the reboot after the installation failed. It turned out that the multipath-tools package was not installed.

Cause

A regular SLE-15-SPx or Leap 15.x or Tumbleweed installation uses the _inst_diskproposal client in the installation workflow which at its end determines which storage features (e.g. Btrfs, XFS, LVM, multipath) are used and adds the software packages that are needed for those technologies to the pool of packages that are going to be installed.

But MicroOS has a different installation workflow, and it doesn't use that default _inst_diskproposal client at all; instead, it uses a custom _partitionsproposal client to handle the storage proposal. And in that client, the packages handling was missing.

Fix

Added the packages handling to that _partitionsproposal client.

To avoid code duplication, that handling was factored out from the _inst_diskproposal client: It is now done by a new PackageHandler::set_proposal_packages_for(devicegraph) method which is now called from both clients.

Test

• New unit tests for that new method and for fringe cases

• Manual test in a Leap 15.5 VM with the changed files to test against regressions

• Manual test in a MicroOS 5.3 VM to check if the support packages are added; changed the storage setup between the default Btrfs to a nonstandard XFS and observed the log file if the packages were added correctly.

Related PRs

• Original PR for SLE-15-SP4: #1346
• Merge to SLE-15-SP5: #1350

[coveralls]

coverage: 97.754% (+0.002%) from 97.752% when pulling 200b66ef569dbe15492370ac217b5fb9d2ff457c on huha-pkg-master into c243464f4932eee869e5110060f425676ba8cf45 on master.

[yast-bot]

:heavy_check_mark: Public Jenkins job #468 successfully finished :heavy_check_mark: Created OBS submit request #1098088

[yast-bot]

:heavy_check_mark: Internal Jenkins job #1139 successfully finished :heavy_check_mark: Created IBS submit request #304846