L3 Veritas Volume Manager: Allow to Ignore Probe Errors (SLE-15-SP2)

[shundhammer]

Bugzilla

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

This merges the original SLE-15-SP1 PR to SLE-15-SP2

Trello

https://trello.com/c/rjFULQ97/

Problem

Some third-party storage software like the Veritas Volume Manager use a disk label (partitioning scheme) that we don't support in libstorage-ng and yast-storage-ng. In this particular case, they use a Sun disk label for disks below 2 TB, and GPT above that size.

When starting the YaST partitioner, libstorage-ng detects the unsupported disk label and reports it as an error. That error is displayed in an error pop-up dialog, and the user can choose to continue anyway (in which case that disk is treated as if it were empty, i.e. no partitions are shown).

In this particular case, the customer had 29 disks and found the sheer number of reported errors overwhelming. They wanted to have a mechanism to speed this up; even more so since this happens every time the YaST partitioner is started.

Solution

This PR introduces a new environment variable $LIBSTORAGE_IGNORE_PROBE_ERRORS. If set, all errors found during probing are suppressed and treated as if the user had chosen "Continue" in the error pop-up. No error pop-up is shown.

Usage

xhost +
sudo LIBSTORAGE_IGNORE_PROBE_ERRORS=1 yast2 disk

or

export LIBSTORAGE_IGNORE_PROBE_ERRORS=1
sudo yast2 disk

Test

Unit Test

cd yast-storage-ng/src/test/lib/callbacks
rspec probe_test.rb
cd ../../..
rake test:unit

Manual Test

• In the VM manager, add a new virtual disk

• Start the VM

• Log in

• Create a Sun disk label on that disk:

  sudo parted /dev/sdb mklabel sun
  

• Start the YaST partitioner normally; it will complain about that disk.

  xhost +
  sudo yast2 disk
  

• Start it with the new environment variable; it will not complain:

  sudo LIBSTORAGE_IGNORE_PROBE_ERRORS=1 yast2 disk
  

Related PRs

• Original: SLE-15 SP1: L3 Veritas Volume Manager: Allow to Ignore Probe Errors (SLE-15-SP1) #1153
• Port to SLE-15 SP2: This PR
• Port to master: L3 Veritas Volume Manager: Allow to Ignore Probe Errors (master) #1152

[coveralls]

Coverage increased (+0.001%) to 97.655% when pulling 0e98db5 on shundhammer:huha-ignore-probe-errors-sp2 into 9284a23 on yast:SLE-15-SP2.