Actions
action #106368
closedcoordination #91646: [saga][epic] SUSE Maintenance QA workflows with fully automated testing, approval and release
coordination #98457: [epic] Handle openqa-review reminder comments on very old jobs better
openqa-review: Configurable no-reminder message pattern size:M
Start date:
2022-02-09
Due date:
% Done:
0%
Estimated time:
Description
Motivation¶
See #104577
Have a configurable blocklist message or pattern and when encountered no reminders are sent. For that configuration option we should provide a sane default, e.g. as regex "(WONTFIX|NO_REMINDER)".
Acceptance criteria¶
- AC1: No reminder comment is sent on bugzilla or progress.opensuse.org from openqa-review if the text from
record_soft_failureincludes NO_REMINDER - AC2: For openqa.suse.de and openqa.opensuse.org in particular same as AC1 but for the word "WONTFIX"
Suggestions¶
- Find examples of openQA jobs that use a custom string to the os-autoinst testapi function
record_soft_failure, e.g. https://openqa.suse.de/tests/7817940#step/cfs_bandwidth01/15 - Look up where this text is parsed in https://github.com/os-autoinst/openqa_review/blob/master/openqa_review/openqa_review.py then skip reminder comments when there is special text in there, e.g. in https://github.com/os-autoinst/openqa_review/blob/master/openqa_review/openqa_review.py#L1534 skip if the text is "NO_REMINDER" by default, but configurable
- For AC2 consider putting the configuration for WONTFIX in https://gitlab.suse.de/openqa/openqa-review/-/blob/master/review-jobs.yml#L1
Actions