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

Remove yast2_i from QSF-U group #10317

Merged
merged 2 commits into from May 20, 2020
Merged

Conversation

SergioAtSUSE
Copy link
Member

@SergioAtSUSE SergioAtSUSE commented May 19, 2020

@SergioAtSUSE SergioAtSUSE changed the title Minimal x Remove yast2_i from QSF-U group May 19, 2020
@SergioAtSUSE SergioAtSUSE marked this pull request as ready for review May 19, 2020 13:21
- console/zypper_lr
- console/zypper_ref
- console/ncurses
- console/yast2_lan
Copy link
Member

Choose a reason for hiding this comment

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

I guess yast2_lan can also be removed

Copy link
Member

Choose a reason for hiding this comment

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

This one you can remove for sure, as of now we had to add this module to EXCLUDE_MODULES in https://gitlab.suse.de/qsf-u/qa-sle-functional-userspace/-/blob/master/functional_sle15_jobgroup.yaml#L15 because main.pm uses it and we didn't want to change scope of QAM tests.

Copy link
Member

Choose a reason for hiding this comment

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

This one you can remove for sure, as of now we had to add this module to EXCLUDE_MODULES in https://gitlab.suse.de/qsf-u/qa-sle-functional-userspace/-/blob/master/functional_sle15_jobgroup.yaml#L15 because main.pm uses it and we didn't want to change scope of QAM tests.

@foursixnine
Copy link
Member

Now I wonder if keyboard_layout_gdm should also be scheduled (and reworked so it covers both: minimalx and gdm)

Copy link
Member

@rwx788 rwx788 left a comment

Choose a reason for hiding this comment

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

We have https://progress.opensuse.org/issues/66209
In case you are in a hurry, go ahead and disable all of those.

@SergioAtSUSE
Copy link
Member Author

We have https://progress.opensuse.org/issues/66209
In case you are in a hurry, go ahead and disable all of those.

What about yast2_booloader?

@rwx788
Copy link
Member

rwx788 commented May 19, 2020

yast2_booloader

https://openqa.suse.de/tests/overview?arch=&machine=&modules=yast2_booloader&distri=sle&version=15-SP2&build=195.1&groupid=110# doesn't show any test suite running it. So what's about yast2_bootloader?

@SergioAtSUSE
Copy link
Member Author

@rwx788
Copy link
Member

rwx788 commented May 19, 2020

yast2_booloader

https://openqa.suse.de/tests/overview?arch=&machine=&modules=yast2_booloader&distri=sle&version=15-SP2&build=195.1&groupid=110# doesn't show any test suite running it. So what's about yast2_bootloader?

I spotted a typo booloader.
https://openqa.suse.de/tests/overview?arch=&machine=&modules=yast2_bootloader&distri=sle&version=15-SP2&build=195.1&groupid=110#

Ah, ok. I will create another ticket, sure

@SergioAtSUSE
Copy link
Member Author

Ready to be merged

@foursixnine foursixnine merged commit 8f82be9 into os-autoinst:master May 20, 2020
@SergioAtSUSE SergioAtSUSE deleted the minimal_x branch March 8, 2021 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants