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

Skip systemd workarounds in other testsuites #12151

Merged
merged 1 commit into from
Mar 17, 2021

Conversation

ge0r
Copy link
Member

@ge0r ge0r commented Mar 17, 2021

@ge0r ge0r requested a review from punkioudi March 17, 2021 08:44
Copy link
Contributor

@punkioudi punkioudi left a comment

Choose a reason for hiding this comment

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

Nice, lgtm!

@ge0r ge0r merged commit 0e65139 into os-autoinst:master Mar 17, 2021
@michaelgrifalconi
Copy link
Contributor

Hello, seems like this PR did break few other tests:

So we would revert this and check if these issues get fixed.

@ge0r
Copy link
Member Author

ge0r commented Mar 18, 2021

Hello, seems like this PR did break few other tests:

* https://openqa.suse.de/tests/5688812

* https://openqa.suse.de/tests/5688813

* https://openqa.suse.de/tests/5688831

So we would revert this and check if these issues get fixed.

It looks like I m not done with this yet :/
This "feature" where qa_run.pm has different names for different testsuites does not really help to track dependencies.
Thank you Michael for the ping! I ll try to fix it asap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants