action #103701
closedcoordination #91646: [saga][epic] SUSE Maintenance QA workflows with fully automated testing, approval and release
Resubmited incident (ID) with new release request (RR) inherits incident test results from previous RR
Description
When RR is declined it can be resubmitted with new RR.
Problem is that it's still same incident ID so it will have old results.
If the incident results passed then it can be auto-approved.
Regressions can pass trough.
e.g. S:M:21592:259811 http://dashboard.qam.suse.de/incident/21592
Files
Updated by okurz almost 3 years ago
- Project changed from openQA Project to QA
- Category deleted (
Regressions/Crashes) - Target version set to future
When you say "Concrete Bugs", meaning a regression, can you reference a "last good"? Or is it actually a new feature request?
Updated by dzedro almost 3 years ago
I'm not aware of any regression, but it's flaw in bot which can cause regressions.
It's confusing, people can think result is based on latest submission, but must be not.
Can be called feature request.
Updated by dzedro almost 3 years ago
Another example of old results being inherited from previous RR, there are just old test results or results don't exist anymore just the result statistics.
This is just annoying for reviewer and blocking auto-approve.
https://smelt.suse.de/incident/21598/
Updated by osukup almost 3 years ago
yes, this is result of too simplified naming of INCIDENT by bot ( technical deept from past) ...
we need check where is used and change all places. (probadly MTUI + bot-ng + qem-dashboard
Updated by osukup almost 3 years ago
another possibility is add variable ( for examle RRID) with RRID and then use this in approver part of bot, which unfortuanetly can work only for incidents part, not aggregates
Updated by osukup almost 3 years ago
after code review .. it needs changes in bot-ng ( few lines) + in mtui (also should be simple)
Updated by osukup almost 3 years ago
ahh , and I also forget about Template Generator (teregen)
Updated by dzedro almost 3 years ago
If we can't use RR then I don't know what options we have. Somehow drop previous results or filter results since day of new RR being submitted ?
Updated by osukup almost 3 years ago
- Status changed from New to In Progress
- Assignee set to osukup
Updated by osukup almost 3 years ago
- Status changed from In Progress to Feedback
.. deployed, need check if it works as intended...
Updated by okurz over 2 years ago
- Related to action #108869: Missing (re-)schedules of SLE maintenance tests size:M added
Updated by okurz over 2 years ago
- Related to action #109310: qem-bot/dashboard - mixed old and new incidents size:M added
Updated by okurz about 2 years ago
- Related to action #119161: Approval step of qem-bot says incident has failed job in incidents but it looks empty on the dashboard size:M added
Updated by mgrifalconi almost 2 years ago
- Related to action #123286: Bot and dashboard reference to wrong data and block update approval size:M added
Updated by jbaier_cz 10 months ago
- Related to action #155206: [qem-bot] re-release update can miss repo and thus not schedule updates added