-
Notifications
You must be signed in to change notification settings - Fork 270
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
Add clients in firstboot_custom #12290
Conversation
455a6e1
to
1ca8b8a
Compare
See os-autoinst/os-autoinst-distri-opensuse#12290 For some reason, certain elements end-up (un)selected when we change the order of clients. It does not disturb change the workflow but needles are then different
1ca8b8a
to
813f766
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
promising work, just need some reshaping in similar fashion than existing code in the framework.
3e1e89e
to
588f821
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job here, small request, could you please update https://gitlab.suse.de/qsf-y/qa-sle-functional-y/-/blob/master/SLES_Integration_Level_Testplan.md so that we do not forget where which firstboot clients are covered.
588f821
to
3eb6fbf
Compare
Done |
See https://progress.opensuse.org/issues/73474:
In order to extend coverage for yast2-firstboot , we can modify the yast2-firstboot configuration files of the relevant test suite:
https://openqa.suse.de/tests/4840750
There already is customized file replacing /etc/YaST2/firstboot.xml after merge of:
#10856
We should re-use yast2_firstboot_custom and use non-default settings for the modules if those are allowed.
Enable firstboot_ntp, firstboot_keyboard and inst_lan modules.
related bugs: bug#1177797 , bug#960081 , bug#1140199 , bug#1101514
VRs (updated)
https://openqa.opensuse.org/tests/1701175
https://openqa.suse.de/tests/5836018