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

Grep systemd target in installation_overview #6977

Merged
merged 1 commit into from
Mar 19, 2019

Conversation

JRivrain
Copy link
Contributor

@JRivrain JRivrain commented Mar 6, 2019

We want to verify if the right systemd target is activated at the earliest possible stage.

@JRivrain JRivrain force-pushed the inst_overview branch 4 times, most recently from 0e051de to d918708 Compare March 8, 2019 10:44
@JRivrain JRivrain changed the title [WIP] Grep systemd target in installation_overview Grep systemd target in installation_overview Mar 8, 2019
@JRivrain JRivrain force-pushed the inst_overview branch 4 times, most recently from e9ecf94 to 58435c2 Compare March 12, 2019 09:30
@JRivrain JRivrain changed the title Grep systemd target in installation_overview [WIP]Grep systemd target in installation_overview Mar 15, 2019
@JRivrain JRivrain force-pushed the inst_overview branch 9 times, most recently from b6bb310 to 56e9e85 Compare March 18, 2019 13:54
@JRivrain JRivrain force-pushed the inst_overview branch 2 times, most recently from 7c71df3 to 782ccdf Compare March 18, 2019 14:57
@JRivrain JRivrain changed the title [WIP]Grep systemd target in installation_overview Grep systemd target in installation_overview Mar 18, 2019
@rwx788 rwx788 merged commit 5bc6037 into os-autoinst:master Mar 19, 2019
@JRivrain
Copy link
Contributor Author

JRivrain commented Mar 20, 2019

os-autoinst/os-autoinst-needles-opensuse#527 has not been merged.

@rwx788
Copy link
Member

rwx788 commented Mar 20, 2019

My bad, I always forget to merge needles =(.
@JRivrain could you please re-create PR?

@JRivrain
Copy link
Contributor Author

Well, one problem was the needle, another was that in some cases it fails on kubic. We have to exclude it also on kubic. I created the new PR, but will do extra testing just in case. One very annoying thing is I had tried to find all the possible cases by searching for needles matching installation-settings-overview-loaded in https://openqa.opensuse.org/admin/needles, but those kubics tests did not come up. So I don't see any way to predict what will fail, except to test the hundreds of single SUTs out there ?
New PR #7109

@jknphy
Copy link
Contributor

jknphy commented Mar 20, 2019

I'm with @rwx788 than in previous comments mentioned to write that piece of code in a different module, but perhaps you have already found most of the cases to not opt for that at this point.

@rwx788
Copy link
Member

rwx788 commented Mar 20, 2019

Well, one problem was the needle, another was that in some cases it fails on kubic. We have to exclude it also on kubic. I created the new PR, but will do extra testing just in case. One very annoying thing is I had tried to find all the possible cases by searching for needles matching installation-settings-overview-loaded in https://openqa.opensuse.org/admin/needles, but those kubics tests did not come up. So I don't see any way to predict what will fail, except to test the hundreds of single SUTs out there ?
New PR #7109

Yeah, that's what we have to deal with, which is understandably frustrating. As for finding potential failures, you can use feature implemented by @OleksandrOrlov to look for test modules. But for this particular one, I guess it's in most of scenarios, including textmode for some maintenance, etc.

To play it safe we could make it in two iterations, to have some way to detect all usages (e.g. check_screen) and on second iteration make it strict to fail when SUT doesn't match expectations. Alternative would be to create separate module and enable it for only known scenarios, which means messing with main.pm.

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