Project

General

Profile

Actions

action #92034

closed

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

openQA Project - coordination #89062: [epic] Simplify review for SUSE QAM

Re-enable openqa-investigate options after the black certificate now only shows properly "reviewed" jobs

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

Status:
Resolved
Priority:
High
Assignee:
Category:
-
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Motivation

openqa-investigate was disabled for SUSE Maintenance job groups on special request due to special workflows however other SUSE QAM reviewers like the idea of openqa-investigate so we would like to bring it back and resolve that special exception

Acceptance criteria

  • AC1: openqa-investigate jobs are also scheduled for jobs in "Maintenance" job groups on openqa.suse.de

Suggestions

Actions #1

Updated by mkittler over 3 years ago

  • Assignee set to mkittler
Actions #2

Updated by mkittler over 3 years ago

SR containing initial discussion: https://gitlab.suse.de/openqa/salt-states-openqa/-/merge_requests/425

There were 3 points:

comments are hiding failures and reviewer does not know if comment is his or from "investigate:retry"

This point has been addressed by the mentioned change regarding the certificate icon.


MM tests are not restarted properly …

When I remember correctly, I've changed it to exclude tests with dependencies completely. So this shouldn't be an issue anymore (although a real solution for this would of course be better).


restarted jobs are renamed, thus don't belong to the same group as original and bot does still see the failed job and auto approve of RR is blocked https://openqa.suse.de/tests/5251199#comments https://openqa.suse.de/tests/5251564#comments https://openqa.suse.de/tests/5251780#comments

Not sure about this one. It doesn't seem like openqa-investigate doesn't make the problem worse - it just doesn't help for approving some PRs automatically.


Draft SR for enabling this: https://gitlab.suse.de/openqa/salt-states-openqa/-/merge_requests/484

Actions #3

Updated by okurz over 3 years ago

Draft SR for enabling this: https://gitlab.suse.de/openqa/salt-states-openqa/-/merge_requests/484

merged. I assume after deployment we still need to restart the instance. I restarted the webUI on osd with systemctl restart openqa-webui and verified that everything looks normal. I suggest you write a message, e.g. in #testing and call this done

Actions #4

Updated by okurz over 3 years ago

  • Status changed from Workable to In Progress
Actions #5

Updated by okurz over 3 years ago

  • Status changed from In Progress to Resolved

Posted in https://chat.suse.de/channel/testing?msg=d88Xbgym4W8YmPqDW . Following https://openqa.suse.de/tests?match=:investigate: we can see again investigation jobs for Maintenance jobs, e.g. https://openqa.suse.de/tests/5934726 so all good.

Actions

Also available in: Atom PDF