yast / yast-storage-ng

Rewrite of https://github.com/yast/yast-storage
http://yast.github.io/
GNU General Public License v2.0
13 stars 19 forks source link

Ensure adding Storage Support Software Packages for MicroOS [SLE-15-SP5] #1350

Closed shundhammer closed 1 year ago

shundhammer commented 1 year ago

Target Branch / Product

This is the merge to SLE-15-SP5 of #1346 .

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

Installer Self-Update / Maintenance Update

Related PRs

coveralls commented 1 year ago

Coverage Status

coverage: 97.749% (+0.001%) from 97.748% when pulling 985006eb068162588ec383c1a0da7a5bb97cff24 on huha-pkg-sp5 into 5884c805f1c4ea56aa8d6366bbe93b80237c3870 on SLE-15-SP5.

yast-bot commented 1 year ago

:heavy_check_mark: Internal Jenkins job #755 successfully finished :heavy_check_mark: Created IBS submit request #302935