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

Modify existing scenario agama_auto_dolomite for selecting another installation device #17719

Merged
merged 1 commit into from Sep 28, 2023

Conversation

tinawang123
Copy link
Contributor

@tinawang123 tinawang123 commented Sep 7, 2023

Modify existing scenario agama_auto_dolomite for selecting another installation device

@rfan1
Copy link
Contributor

rfan1 commented Sep 7, 2023

I can see the second HDD image in your job setting, I would like suggest you adding it, and then use lsblk command to show them.
Then you can define your root disk in json file IMO :)

@tinawang123 tinawang123 changed the title Modify existing scenario agama_auto_dolomite for selecting another installation device [WIP]Modify existing scenario agama_auto_dolomite for selecting another installation device Sep 7, 2023
@tinawang123 tinawang123 force-pushed the selectdev branch 3 times, most recently from a83a567 to 8d974a1 Compare September 7, 2023 09:00
sub run {
select_console 'root-console';

my $installed_disk_expected = get_test_suite_data()->{installed_disk};
Copy link
Contributor

@JRivrain JRivrain Sep 7, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think validate vda is not useful as its the default, so we don't know if the disk was selected thanks to the jsonnet code, or just because it's the default.
Please use a second disk, then you could try to use console/validate_first_disk_selection with this test data:
guided_partitioning:
disks:
- vdb
unused_disks:
- vda

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It already has 'console/validate_first_disk_selection' test module to check the disk.
I will delete this test module, use the existed one.

@tinawang123 tinawang123 force-pushed the selectdev branch 11 times, most recently from d181ec1 to 57e56ae Compare September 12, 2023 07:53
@tinawang123 tinawang123 force-pushed the selectdev branch 2 times, most recently from 623057e to b394f00 Compare September 19, 2023 02:09
@tinawang123 tinawang123 changed the title [WIP]Modify existing scenario agama_auto_dolomite for selecting another installation device Modify existing scenario agama_auto_dolomite for selecting another installation device Sep 19, 2023
@rakoenig rakoenig merged commit 2c0368e into os-autoinst:master Sep 28, 2023
7 checks passed
@tinawang123 tinawang123 deleted the selectdev branch October 8, 2023 03:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants