Project

General

Profile

action #88223

[openqa-in-openqa] Remove special-case on load_osautoinst_tests with OPENQA_FROM_GIT

Added by cdywan 6 months ago. Updated 5 months ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Concrete Bugs
Target version:
Start date:
2021-01-26
Due date:
% Done:

0%

Estimated time:
Difficulty:

Description

We have a new test running with OPENQA_CONTAINERS=1 in addition to the existing cases. And during a review of the job template YAML we came across this special-case since the new scenario wouldn't inherit the settings from the openqa_from_git scenario. The main.pm of os-autoinst-distri-openQA has this condition:

# testing from git only tests webui so far
load_osautoinst_tests() unless check_var('OPENQA_FROM_GIT', 1);

From the comment it seems like we should enable more tests to run load_osautoinst_tests. For now the openQA in openQA job group contains OPENQA_FROM_GIT=1 in the scenario definition in YAML.

Acceptance Criteria

  • AC1:

Suggestions

  • Drop OPENQA_FROM_GIT=1 special-case from main.pm
  • Extend use cases for load_osautoinst_tests and rename the flag accordingly

History

#1 Updated by okurz 6 months ago

you told me that tickets with links are much more helpful, so I suggest you do the same :) Please reference the ticket that adds the new scenario and if you can an openQA test that shows the problem you mean … assuming that you actually see a problem? Please select the category for ticket accordingly as well. I assume "Concrete Bugs"? However so far it's not really that "concrete".

#2 Updated by okurz 6 months ago

  • Category set to Concrete Bugs
  • Target version set to Ready

#3 Updated by cdywan 6 months ago

  • Description updated (diff)

okurz wrote:

you told me that tickets with links are much more helpful, so I suggest you do the same :) Please reference the ticket that adds the new scenario and if you can an openQA test that shows the problem you mean … assuming that you actually see a problem? Please select the category for ticket accordingly as well. I assume "Concrete Bugs"? However so far it's not really that "concrete".

I filed it during a call with ilausuch and left it as New because we weren't completely sure of the intention. Apparently I closed the tab without adding more details. Thank you for asking!

#4 Updated by okurz 5 months ago

  • Target version changed from Ready to future

Also available in: Atom PDF