-
Notifications
You must be signed in to change notification settings - Fork 270
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
spvm: Format all disks before the test starts #9449
Conversation
@@ -117,10 +117,15 @@ sub prepare_pvm_installation { | |||
|
|||
# Delete partition table before starting installation | |||
select_console('install-shell'); | |||
foreach my $disk (split(',', get_var('DISK_DEVICES', 'sda'))) { |
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.
As we get rid of this variable, let's remove it from variables.md
too.
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.
yep, but also found better solution =)
6458a17
to
c00d1fc
Compare
f2d9b3d
to
d0a1375
Compare
lib/bootloader_pvm.pm
Outdated
|
||
my $disks = script_output('lsblk -n -p -l -o NAME -d -e 7,11'); | ||
for my $d (split('\n', $disks)) { | ||
script_run("echo 'wipefs -a $d'"); |
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.
Let's remove this line too ;)
d0a1375
to
6acbe70
Compare
my $disks = script_output('lsblk -n -p -l -o NAME -d -e 7,11'); | ||
for my $d (split('\n', $disks)) { | ||
script_run "wipefs -a $d"; | ||
create_encrypted_part("$d") if get_var('ENCRYPT_ACTIVATE_EXISTING'); |
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.
Hi, this caused s little issue here https://openqa.suse.de/tests/3909752#step/bootloader_start/36 as then in create_encrypted_part we have assert_script_run "parted -s /dev/$disk mklabel gpt";
No description provided.