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

Fix Base-RT-SelfInstall #553

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

grisu48
Copy link
Contributor

@grisu48 grisu48 commented Nov 26, 2024

Base-RT SelfInstall should use the selfinstall yaml anchors, not the image-based settings.

Base-RT SelfInstall should use the selfinstall yaml anchors, not the
image-based settings.
@grisu48 grisu48 requested a review from lkocman November 26, 2024 10:47
@lkocman
Copy link
Contributor

lkocman commented Nov 26, 2024

Absolutely

I wanted to put our Leap Micro 6.1 into a better shape so I reworked this a lot
https://paste.opensuse.org/pastes/4d80b898cb87 (backup)
https://openqa.opensuse.org/admin/job_templates/102?

The UI seem to already contain this change (I'm aware that I need to push it all to git).
I wanted to fix also the Default-qcow and Default-encrypted boot.

Basically all of these remaining faiulres https://openqa.opensuse.org/tests/overview?result=failed&result=incomplete&result=timeout_exceeded&distri=leap-micro&version=6.1&build=4.22&groupid=102

It all seems to be wrongly parametrized as I can boot these fine in virt-manager.

But let's be careful to not overwrite the jobgroup config now :)

@grisu48
Copy link
Contributor Author

grisu48 commented Nov 28, 2024

[...]
But let's be careful to not overwrite the jobgroup config now :)

About the other changes yes, but this one is clearly wrong. A SelfInstall job should use the selfinstall settings and not the image settings.

@lkocman lkocman merged commit bc0be23 into os-autoinst:master Dec 4, 2024
3 checks passed
@grisu48 grisu48 deleted the leap-micro-selfinstall branch December 4, 2024 13:20
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.

2 participants