action #152681
closedcoordination #58184: [saga][epic][use case] full version control awareness within openQA
coordination #152847: [epic] version control awareness within openQA for test distributions
openqa-investigate creates wrong CASEDIR setting for some tests size:M
0%
Description
Observation¶
https://openqa.opensuse.org/tests/3815334#comments has:
openqa_from_bootstrap:investigate:retry: https://openqa.opensuse.org/tests/3815345
openqa_from_bootstrap:investigate:last_good_tests:92866dcb2f14046a8e26a47bcb0f0ee3959920b3: https://openqa.opensuse.org/tests/3815346
openqa_from_bootstrap:investigate:last_good_build::TW.25094: https://openqa.opensuse.org/tests/3815347
openqa_from_bootstrap:investigate:last_good_tests_and_build:92866dcb2f14046a8e26a47bcb0f0ee3959920b3+:TW.25094: https://openqa.opensuse.org/tests/3815348
The last_good_tests and last_good_tests_and_build are both failing because they have:
CASEDIR https://github.com/os-autoinst/os-autoinst-distri-opensuse.git#92866dcb2f14046a8e26a47bcb0f0ee3959920b3
But it should be os-autoinst-distri-openqa instead, and the commit 92866dcb2f14046a8e26a47bcb0f0ee3959920b3 is not found.
That's because:
https://github.com/os-autoinst/scripts/blob/master/openqa-investigate#L56
repo=${casedir:-'https://github.com/os-autoinst/os-autoinst-distri-opensuse.git'}
Acceptance Criteria¶
- AC1: openQA-in-openQA investigation jobs show correct "last_good_tests" jobs using test code from os-autoinst-distri-openQA (not os-autoinst-distri-opensuse)
Suggestions¶
Confirm that no information is lost if CASEDIR is overwritten- git ref is still available
- branch name is still known
- users are not typically confused
- Try to solve it with a simple hacky solution and think about consequences later, i.e. add more ideas to the parent epic :)
In os-autoinst we already compute the TEST_GIT_HASH so probably it's easy to look up the actual git repo used if any and then overwrite CASEDIR after isotovideo read casedirThen confirm that openqa-investigate can read the updated CASEDIRIf it turns out that CASEDIR can not or should not be overwritten then we could introduce a new TEST_GIT/NEEDLES_GIT variableTEST_GIT_URL/NEEDLES_GIT_URL
was added