action #93522
closed[tools][qem] auto-approval of kgrafts and live-patches
0%
Description
kgraft / live-patch updates are tested exclusively in openQA, but are getting qam-kernel approval group. That means someone has to download, update and upload test reports that no one is ever reading. With around 60 at once, it's a waste of time. These updates could get qam-openqa group and (if green) be approved by bot based on results from their runs.
So we need:
- switch the assigned group
- configure bot to approve if all is good
Updated by okurz over 3 years ago
- Subject changed from [qem] auto-approval of kgrafts and live-patches to [tools][qem] auto-approval of kgrafts and live-patches
- Target version set to Ready
@osukup this something that 1. you can do very quickly yourself or teach others how to do?
https://gitlab.suse.de/search?project_id=3530&search=kgraft shows some references but https://gitlab.suse.de/search?project_id=3530&search=qam-kernel does not reveal any hits. So where is this configured that qam-kernel is assigned? Or is this coming automatically from OBS/IBS because the kernel package is involved?
Updated by osukup over 3 years ago
groups are assigned by SMELT,
if kgraft has only qam-openqa group and is green it should be approved automaticaly
Updated by okurz over 3 years ago
right, so
- configure bot to approve if all is good
is already covered.
So for
switch the assigned group
maybe changing
https://gitlab.suse.de/tools/smelt/-/blob/master/smelt/smeltapp/agents/assignrrs.yml#L494
is enough? @hurhaj I guess you could try to apply that change yourself and ask smelt experts in a merge request if this is correct.
@osukup if you agree then assign to hurhaj or whoever you think is better fitting.
Updated by okurz over 3 years ago
- Status changed from New to In Progress
osukup created https://gitlab.suse.de/tools/smelt/-/merge_requests/683 which looks good
Updated by openqa_review over 3 years ago
- Due date set to 2021-06-23
Setting due date based on mean cycle time of SUSE QE Tools
Updated by okurz over 3 years ago
https://gitlab.suse.de/tools/smelt/-/merge_requests/683 merged.
I assume we have to wait for a more recent smelt release than the current 3.9.4 : https://gitlab.suse.de/tools/smelt/-/releases
Updated by livdywan over 3 years ago
- Due date changed from 2021-06-23 to 2021-07-02
@osukup are we still waiting on a release? Can you clarify what this is blocking on?
Updated by livdywan over 3 years ago
- Due date changed from 2021-07-02 to 2021-07-07
Moving due date due to public holidays
Updated by okurz over 3 years ago
- Due date changed from 2021-07-07 to 2021-07-02
Same as in #91356#note-28 we should not fool ourselves by acting "surprised" that suddenly there is a public holiday :D That can not be the reason for having missed the deadline. Also you already bumped the deadline in before and there was no action.
Updated by okurz over 3 years ago
- Status changed from Blocked to Resolved
We assume that this is fixed because there is a more recent release from smelt, the ticket description does not state how we could check if the original request is fulfilled. You are always welcome to tell us if something is missing.
Updated by msmeissn over 3 years ago
btw, this is probably ALSO affecting the real kernel updates ... not sure if you intended it this way.
(as real kernel updates have an initial livepatch)
Updated by hurhaj over 3 years ago
msmeissn wrote:
btw, this is probably ALSO affecting the real kernel updates ... not sure if you intended it this way.
(as real kernel updates have an initial livepatch)
no, it was not intended. and although it might be worthy of a discussion, definitely not like this.
@osukup @okurz could you readjust this? also, it will need SMELT update to fix this, correct?
Updated by okurz over 3 years ago
- Due date changed from 2021-07-02 to 2021-08-06
- Status changed from Resolved to Feedback
@osukup can you please look into this and state if it's feasible to achieve? If the original benefit can not be achieved with side-effect I suggest to revert and plan differently for the future.
Updated by osukup over 3 years ago
(as real kernel updates have an initial livepatch)
no, it was not intended. and although it might be worthy of a discussion, definitely not like this.
and unfotuanetly LTSS updates have same product/repo for kernel and ltss --> so this is simple for full supported lp but not for LTSS lp without code changes in smelt.
Updated by osukup over 3 years ago
Updated by osukup over 3 years ago
merged , so for now should be LP in own projects only with 'qam-openqa' and LTSS with 'qam-kernel' + 'qam-openqa'
Updated by osukup over 3 years ago
- Status changed from Feedback to Resolved
unfortunately its solvable only for general supported LP, (which works ... as by complaints by some colleagues too much fast:))