-
Notifications
You must be signed in to change notification settings - Fork 122
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
Assign a data path and serial number to each test in discover #1876
Conversation
d7a8c5c
to
e49d0ad
Compare
Hm, when the test name is |
Yeah, I'm adding some tests, I'll add this case as well. No idea how it happens though, but I'll look into it. |
e49d0ad
to
98dd070
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.
LGTM to the code, still running tests/full in TJ#7580157
Fails in tests/full: All failed with test assuming the old path |
To support both multihost scenarios and/or running a test multiple times, tmt needs to distinguish test data directories of different test instances. To allow this, each test instance is given a "serial number" which we can use in a data directory name. Note: there can be more elaborate ways for picking the serial numbers, the patch starts with something really trivial yet functioning. Another note: the patch does not isolate tests on a phase level, a single test, executed on different guests, may still land in the same data directory. Different patch will solve this issue.
8ec2aca
to
359930d
Compare
359930d
to
142b5c3
Compare
Tests are green \o/ So, looking for reviews & merge, then:
|
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.
LGTM
To support both multihost scenarios and/or running a test multiple times, tmt needs to distinguish test data directories of different test instances. To allow this, each test instance is given a "serial number" which we can use in a data directory name.
Note: there can be more elaborate ways for picking the serial numbers, the patch starts with something really trivial yet functioning.
Another note: the patch does not isolate tests on a phase level, a single test, executed on different guests, may still land in the same data directory. Different patch will solve this issue.
Part of the effort in #1614 and #1790.