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

Process registration screen for SLES distribution #11819

Merged
merged 2 commits into from Jan 22, 2021

Conversation

rwx788
Copy link
Member

@rwx788 rwx788 commented Jan 21, 2021

We now have registration module being added by default to the first boot
profile. System is already registered, as we use Online medium for the
installation. Instead of de-registering it, we would prefer to assert
that registered system is recognized as such, as registration code is
tested in other installations.

See poo#87904.

Rodion Iafarov added 2 commits January 21, 2021 10:21
We now have registration module being added by default to the first boot
profile. System is already registered, as we use Online medium for the
installation. Instead of unregistering it, we would prefer to assert
that registered system is recognized as such, as registration code is
tested in other installations.

See [poo#87904](https://progress.opensuse.org/issues/87904).
@jknphy
Copy link
Contributor

jknphy commented Jan 22, 2021

does it need 'test_data/yast/firstboot/yast2_firstboot.yaml' to have as well registration step 'firstboot_registration' for the failed verification?
Other than that LGTM.

@rwx788
Copy link
Member Author

rwx788 commented Jan 22, 2021

does it need 'test_data/yast/firstboot/yast2_firstboot.yaml' to have as well registration step 'firstboot_registration' for the failed verification?
Other than that LGTM.

I have created separate test_data file which will be set in the JobGroup. Tumbleweed doesn't have registration step. I also thought of removing test_data from the schedule and define it also in the job group on o3, but decided to keep that one as a default one. Somehow I had wrongly triggered VR, but the one before it worked: https://openqa.suse.de/tests/5322884
I've cleaned this up.

@jknphy jknphy merged commit 0df15c6 into os-autoinst:master Jan 22, 2021
@rwx788 rwx788 deleted the autoyast branch January 26, 2021 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants