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.

Sign up for GitHub

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

Jump to bottom

Split upstream testsuites from extratests #8713

Merged
merged 1 commit into from Oct 18, 2019
Merged

Split upstream testsuites from extratests #8713

merged 1 commit into from Oct 18, 2019

Conversation

dzedro
Copy link
Contributor

@dzedro dzedro commented Oct 17, 2019 •
edited

@coolo
Copy link
Contributor

coolo commented Oct 17, 2019

Note that https://openqa.suse.de/tests/3486877 has EXCLUDE_MODULES set. I assume the goal is to have it run without this setting?

@coolo
Copy link
Contributor

coolo commented Oct 17, 2019

Is it really necessary to duplicate mau-bind.yaml between service packs? Or files that are all the same in general. WDYT @rwx788 ?

@coolo
Copy link
Contributor

coolo commented Oct 17, 2019

incomplete: https://openqa.suse.de/tests/3486877/file/autoinst-log.txt

@dzedro
Copy link
Contributor Author

dzedro commented Oct 17, 2019 •
edited

Is it really necessary to duplicate mau-bind.yaml between service packs? Or files that are all the same in general. WDYT @rwx788 ?

I don't like it too, I would just put yamls that are same for all versions outside VERSION directory

@coolo
Copy link
Contributor

coolo commented Oct 17, 2019

So create a qam/common ?

@dzedro dzedro force-pushed the yaml branch 2 times, most recently from 6eae655 to ade3f67 Compare October 17, 2019 07:27
@dzedro
Copy link
Contributor Author

dzedro commented Oct 17, 2019

So create a qam/common ?

done

@dzedro
Copy link
Contributor Author

dzedro commented Oct 17, 2019

Note that https://openqa.suse.de/tests/3486877 has EXCLUDE_MODULES set. I assume the goal is to have it run without this setting?

removed EXCLUDE_MODULES

@dzedro dzedro force-pushed the yaml branch 2 times, most recently from b3cfbeb to 22a7b19 Compare October 17, 2019 08:28
@dzedro
Copy link
Contributor Author

dzedro commented Oct 17, 2019

incomplete: https://openqa.suse.de/tests/3486877/file/autoinst-log.txt

verification runs fixed

@rwx788
Copy link
Member

rwx788 commented Oct 17, 2019

Is it really necessary to duplicate mau-bind.yaml between service packs? Or files that are all the same in general. WDYT @rwx788 ?

I need to make diff of those to see how different those are. But this is again main challenge here is to understand why we have different set of test modules and act on the root cause.

@dzedro dzedro force-pushed the yaml branch 3 times, most recently from dbf3c7f to d0aed1c Compare October 17, 2019 09:04
@dzedro dzedro added the Ready Ready for review label Oct 17, 2019
@dzedro
Copy link
Contributor Author

dzedro commented Oct 18, 2019 •
edited

Is it really necessary to duplicate mau-bind.yaml between service packs? Or files that are all the same in general. WDYT @rwx788 ?

I need to make diff of those to see how different those are. But this is again main challenge here is to understand why we have different set of test modules and act on the root cause.

yaml files in common (schedule/qam/common) are same for all versions

@dzedro dzedro merged commit 57c9d73 into os-autoinst:master Oct 18, 2019
@dzedro dzedro deleted the yaml branch October 18, 2019 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Ready Ready for review
Projects
None yet
Milestone
No milestone
3 participants
@dzedro @coolo @rwx788