Skip to content

WIP: Fix agama_auto expect timeout #22471

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

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

Conversation

mdoucha
Copy link
Contributor

@mdoucha mdoucha commented Jun 25, 2025

The boot_agama module switches away from the installation console. Make sure that RebootPage::expect_is_shown() looks for the finish screen on the right console.

Fixes #22437

The boot_agama module switches away from the installation console.
Make sure that RebootPage::expect_is_shown() looks for the finish
screen on the right console.
@czerw czerw requested a review from jknphy June 25, 2025 14:33
@jknphy
Copy link
Contributor

jknphy commented Jun 25, 2025

Looks reasonable if the initial console is not the same for all archs/backend, is that the case, right?
@okynos please sync here for more verification in other archs/backend, thanks!
#22473

@mdoucha mdoucha changed the title Fix agama_auto expect timeout WIP: Fix agama_auto expect timeout Jun 25, 2025
@okynos
Copy link
Contributor

okynos commented Jun 25, 2025

Yes let me do some VRs

@mdoucha
Copy link
Contributor Author

mdoucha commented Jun 25, 2025

So apparently, installation is not the right console to get back to the graphical installer either...
https://openqa.suse.de/tests/18206418#step/agama_auto/2

@okynos
Copy link
Contributor

okynos commented Jun 25, 2025

It is because installation is not saved as needle take a look at this VR https://openqa.suse.de/tests/18206556 (I added installation tag in the needle)

This is because there is no previous console saved in a needle

@okynos
Copy link
Contributor

okynos commented Jun 25, 2025

I will revert the original PR because it seems to fail in another places like https://openqa.suse.de/tests/18206546

@mdoucha
Copy link
Contributor Author

mdoucha commented Jun 25, 2025

It is because installation is not saved as needle take a look at this VR https://openqa.suse.de/tests/18206556 (I added installation tag in the needle)

This is because there is no previous console saved in a needle

That's nice, but on the PPC it switches to a third console instead of the first one. Adding a needle tag to the installer screenshot won't help when you'll get a console with login prompt and QR code instead.

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.

3 participants