Skip to content
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

kvm-test: add option to add extra disks for install #1523

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ogayot
Copy link
Member

@ogayot ogayot commented Jan 3, 2023

I tried to reproduce https://bugs.launchpad.net/subiquity/+bug/2000066 in a VM where the original test case involves multiple disks. Sadly, I didn't manage to reproduce the crash but I think having the option in kvm-test to add extra disks would be good for future testing.

The patch introduces a new option --extra-disk-size to create additional disks in kvm-test. The option can be specified multiple times and will not affect the first disk (the one boots in presence of the --boot option).

# will create the first disk normally and will add an extra 10GiB disk
kvm-test.py --install --extra-disk-size 10G
# will create the first disk normally and will two extra disks (one of 10 and one of 20 GiB).
kvm-test.py --extra-disk-size 10G --extra-disk-size 20G

The size of the first disk (that can honor the --boot option) can still be configured using the --disksize option).

The patch introduces a new option --extra-disk-size to create additional
disks in kvm-test. The option can be specified multiple times and will
not affect the first disk (the one boots in presence of the --boot
option).

 $ kvm-test.py --install --extra-disk-size 10G

... will create the first disk normally and will add an extra 10GiB disk

 $ kvm-test.py --extra-disk-size 10G --extra-disk-size 20G

... will create the first disk normally and will two extra disks (one of
10 and one of 20 GiB).

The size of the first disk (that can be honor the --boot option) can
still be configured using the --disksize option).

Signed-off-by: Olivier Gayot <[email protected]>
@ogayot ogayot requested a review from dbungert January 3, 2023 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant