yast / yast-storage-ng

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

[SP5] Guided Setup: display a hint for disks with sensible transports #1333

Closed ancorgs closed 1 year ago

ancorgs commented 1 year ago

Problem

Described and discussed at https://bugzilla.suse.com/show_bug.cgi?id=1209588

The screen of the Guided Setup used to select the target disk(s) shows some information about each disk to make it possible to identify them.

sles-bfs-1

But deciding which information to display is VERY tricky:

As a reasonable compromise, YaST displays the following information for each disk:

That was reported to not be enough in the case in which both local and remote NVMe disks are present in the system. There is nothing in the previous list that would allow to distinguish the remote from the local ones.

Solution

As a quick fix for SLE-15-SP5, just display the data transport in some special cases.

This displays the internal identifier of the transport, that is something like fcoe or tcp. Nothing too nice. Eg.

Testing

Extended unit tests

coveralls commented 1 year ago

Coverage Status

Coverage: 97.751% (+0.0002%) from 97.75% when pulling cf949f6e0296bdefac43feba8e09ee7ac2e3c709 on ancorgs:bsc1209588_sp5 into 4af0e59801545484eeef5eac139c263ef488ae8d on yast:SLE-15-SP5.

yast-bot commented 1 year ago

:heavy_check_mark: Internal Jenkins job #753 successfully finished :heavy_check_mark: Created IBS submit request #296200