Project

General

Profile

action #162632

Updated by okurz 6 months ago

## Motivation 
 See error report by Martchus aka. mkittler in https://github.com/os-autoinst/os-autoinst-distri-openQA/pull/177#issuecomment-2180542666 

 ## Acceptance criteria 
 * **AC1:** `openqa-clone-custom-git-refspec https://github.com/os-autoinst/os-autoinst-distri-openQA/pull/<id> https://openqa.opensuse.org/tests/<test_id>` yields passing openQA job without needing to set any needles URL specifically for `<id>` pointing to any PR in that repo and `<test_id>` pointing to any openQA-in-openQA job. Look up one of those from https://openqa.opensuse.org/group_overview/24 
 * **AC2:** Specifying NEEDLES_DIR as additional argument has precedence 
 * **AC3:** osado can still passed when triggered with `openqa-clone-custom-git-refspec` which do not use the git URL in their CASEDIR or NEEDLES_DIR without needing explict mentions of NEEDLES_DIR 

 ## Suggestions 
 * Look into https://github.com/os-autoinst/openQA/blob/master/script/openqa-clone-custom-git-refspec and how it processes NEEDLES_DIR in particular. 
 * Take a look at existing production openSUSE default tests like https://openqa.opensuse.org/tests/4300489 . Those ones have no setting for CASEDIR and no setting for NEEDLES_DIR. After processing vars.json has CASEDIR set to the value of DISTRI and no value for NEEDLES_DIR. In contrast a production openQA-in-openQA job like https://openqa.opensuse.org/tests/4301863 has both DIR variables set to git URLs and vars.json mentions a path like "/var/lib/openqa/pool/13/os-autoinst-needles-openQA" ending up in needing the workaround 


 ## Workaround 
 * `openqa-clone-custom-git-refspec https://github.com/os-autoinst/os-autoinst-distri-openQA/pull/177 https://openqa.opensuse.org/tests/4285633 NEEDLES_DIR=https://github.com/os-autoinst/os-autoinst-needles-openQA.git`

Back