Project

General

Profile

Actions

action #152681

closed

coordination #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

Added by tinita about 1 year ago. Updated 11 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2023-12-15
Due date:
% Done:

0%

Estimated time:

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 casedir
  • Then confirm that openqa-investigate can read the updated CASEDIR
  • 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
Actions

Also available in: Atom PDF