Project

General

Profile

action #152681

Updated by tinita 11 months ago

## 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 Confirm that no information is lost if CASEDIR is overwritten~~ 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 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 casedir~~ casedir 
 * ~~Then Then confirm that openqa-investigate can read the updated CASEDIR~~ CASEDIR 
 * ~~*If* *If* it turns out that CASEDIR can not or should not be overwritten then we could introduce a new TEST_GIT/NEEDLES_GIT variable~~ `TEST_GIT_URL/NEEDLES_GIT_URL` was added variable

Back