New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adapt partitioning_lvm for *activate_existing* tests to storage_ng #4354
Conversation
lib/partition_setup.pm
Outdated
@@ -196,7 +196,7 @@ sub unselect_xen_pv_cdrom { | |||
sub enable_encryption_guided_setup { | |||
my $self = shift; | |||
send_key $cmd{encryptdisk}; | |||
if (!get_var('ENCRYPT_ACTIVATE_EXISTING')) { | |||
if (!get_var('ENCRYPT_ACTIVATE_EXISTING') || is_storage_ng) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
better add a comment above this line
if ($numdisks <= 1) { | ||
die "Guided workflow does not skip disk selection when only one disk!" | ||
unless match_has_tag('inst-select-disk-to-use-as-root'); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, remove empty line
if (is_storage_ng) { | ||
if ($numdisks <= 1) { | ||
die "Guided workflow does not skip disk selection when only one disk!" | ||
unless match_has_tag('inst-select-disk-to-use-as-root'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should fail unconditionally here, as match_has_tag('inst-select-disk-to-use-as-root') is already above
} | ||
else { | ||
die "Guided workflow does skip disk selection when more than one disk! (NUMDISKS=$numdisks)" | ||
unless match_has_tag('inst-partition-guided'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On the line 89 we already have match, so if will always die unless same needle has both tags, which is not expected, I guess.
@@ -127,15 +100,14 @@ sub run { | |||
send_key "tab"; | |||
type_password; | |||
send_key "ret"; | |||
assert_screen "partition-cryptlvm-summary"; | |||
assert_screen "partition- | |||
cryptlvm-summary"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does not look correct
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tidy issue, I'm sure
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed this manually, please check.
- adapt workflow at guided setup for storage_ng - clean up record_soft_failure for issue resolved - verification run sles 15: http://e13.suse.de/tests/326#step/partitioning_lvm - verification run sles 12 sp4: http://e13.suse.de/tests/330#step/partitioning_lvm - add inst-encrypt-password-prompt in enable_encryption_guided_setup - see poo#30637 for more details - needle PR: https://gitlab.suse.de/openqa/os-autoinst-needles-sles/merge_requests/687
needles merged, LGTM. Waiting for checks |
https://gitlab.suse.de/openqa/os-autoinst-needles-sles/merge_requests/687