Project

General

Profile

Actions
Copy link

action #93065

closed

QA - coordination #91646: [saga][epic] SUSE Maintenance QA workflows with fully automated testing, approval and release

coordination #91914: [epic] Make reviewing openQA results per squad easier

"parallel_failed" jobs show up on /tests/overview?todo=1 but these do not need a label

Added by okurz almost 3 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Low
Assignee:
okurz
Category:
Feature requests
Target version:
Ready
Start date:
2021-05-25
Due date:
% Done:

0%

Estimated time:

Description

Motivation

As visible in
Screenshot_20210525_101708_parallel_failed_on_todo_list.png
the "todo"-view on /tests/overview can show up parallel_failed jobs but the build receives the "all reviewed" black certificate badge regardless as it should. These parallel_failed jobs should not show up on the todo view

Acceptance criteria

  • AC1: parallel_failed show not up on /tests/overview on todo=1
  • AC2: parallel_failed still show up if the todo flag is not provided
  • AC3: the evaluation for the black certificate badge is not changed

Suggestions

  • Research where the state and results of jobs are evaluated for todo=1 and compare the evaluation for the case when results are prepared for display

Files

Screenshot_20210525_101708_parallel_failed_on_todo_list.png (23 KB) Screenshot_20210525_101708_parallel_failed_on_todo_list.png okurz, 2021-05-25 08:19
Actions
Copy link
 #1

Updated by okurz almost 3 years ago

  • Parent task set to #91914
Actions
Copy link
 #2

Updated by okurz over 2 years ago

  • Status changed from Workable to Feedback
  • Assignee set to okurz

We hacked something together. Let's see how far we come: https://github.com/os-autoinst/openQA/pull/4316

Actions
Copy link
 #3

Updated by okurz over 2 years ago

  • Target version changed from future to Ready

the topic was brought up by @Xiaojing_liu in #92857 . So as this was affecting here and it showed that it's important we come together for a quick mob-programming session and came up with the above PR. Let's see if we can solve it quickly. Hence putting it on the backlog.

Actions
Copy link
 #4

Updated by okurz over 2 years ago

  • Status changed from Feedback to Resolved

https://github.com/os-autoinst/openQA/pull/4316 merged and active on production. A quick look into https://openqa.opensuse.org/tests/overview?arch=&flavor=&machine=&test=&modules=&module_re=&todo=1&distri=microos&distri=opensuse&version=Tumbleweed&build=20211115&groupid=1# looks exactly as it should. There are two "parallel_failed" jobs shown but in both cases test modules have actually failed in there independant of the job being cancelled in the next module after the failing one.

Actions
Copy link

Also available in: Atom PDF