action #136244
closedEnsure arbitrary comments can be taken over to new jobs size:M
Description
Motivation¶
#107923-64 states that carry-over of @review:acceptable_for…
comments do not work which is expected because such comments are just "openQA comments", in openQA only bugrefs are carried over.
#107923-71 might provide valid points that should be crosschecked but effectively no one will be able to find those suggestions again buried in a ticket. Let's make the suggestions more prominent.
For example
@review:acceptable_for:incident_30146:bsc#1212271
bsc#1212271
Or
@review:acceptable_for:incident_30146:.
bsc#1212271
Note the bugref to trigger takeover.
Acceptance criteria¶
- AC1: Valid workflows from #107923-71 are covered in public upstream help text or documentation
- AC2: Arbitrary comments can be taken over with a known syntax, e.g. combined with bugref
- AC3: Plain comments without bugrefs are not taken over
Suggestions¶
- See #107923-71
- Research what this feature is, where it's implemented and how it works
- Check if the ideas are valid and work as stated
- Correct where needed
- Optional: Improvements to help text within openQA comment editor
- Add to upstream openQA documentation
- Optional: Spread the ideas more, e.g. workshop
Updated by okurz about 1 year ago
- Target version changed from Tools - Next to Ready
Updated by livdywan about 1 year ago
- Subject changed from Ensure ideas from #107923-71 are reachable from open.qa/docs to Ensure arbitrary comments can be taken over to new jobs size:M
- Description updated (diff)
- Status changed from New to Workable
Updated by dheidler about 1 year ago
- Status changed from Workable to In Progress
Updated by openqa_review about 1 year ago
- Due date set to 2023-11-14
Setting due date based on mean cycle time of SUSE QE Tools
Updated by livdywan about 1 year ago
- Related to action #139055: Comments mentioning bugrefs as part of a sentence are treated like bug refs and taken over size:S added
Updated by dheidler about 1 year ago
Updated by livdywan about 1 year ago
- Due date changed from 2023-11-14 to 2023-11-17
Not expecting this to be worked on this week, hence bumping the due date
Updated by okurz about 1 year ago
Updated by dheidler about 1 year ago
- Status changed from In Progress to Feedback
Let's see if it will look fine after next deployment.
Updated by okurz about 1 year ago
Already deployed, see https://openqa.opensuse.org/changelog , ready to verify :)
Updated by dheidler about 1 year ago
- Status changed from Feedback to Resolved
Looks good so far.
Opened https://github.com/os-autoinst/openQA/pull/5368 for some minor styling improvements.