Project

General

Profile

Actions
Copy link

action #166559

open

coordination #154768: [saga][epic][ux] State-of-art user experience for openQA

coordination #166556: [epic] Improved test reviewer user experience - Restart filtered jobs from /tests/overview

Restart filtered jobs from /tests/overview with comment size:M

Added by okurz about 1 month ago. Updated 1 day ago.

Status:
Workable
Priority:
Low
Assignee:
ybonatakis
Category:
Feature requests
Target version:
Ready
Start date:
2024-09-09
Due date:
% Done:

0%

Estimated time:

Description

Motivation

On test overview pages like https://openqa.opensuse.org/tests/overview?distri=opensuse&version=Tumbleweed&groupid=1 we already have a comment button which allows to add a comment on all currently shown jobs, i.e. depending on the current filter selection. With the same filter now we can also add a possibility to restart the according filtered jobs, either with or without a corresponding comment.

Let's start with an option to comment and restart the selected jobs.

Acceptance criteria

  • AC1: Users with operator level can restart all currently filtered jobs from /tests/overview
  • AC2: Jobs that are not shown on the filtered view are not restarted except for necessary dependencies
  • AC3: Users below the operator level are not able to restart jobs

Suggestions

  • Take a look on https://openqa.opensuse.org/tests/overview?distri=opensuse&version=Tumbleweed&groupid=1, ensure you are logged in and explore the existing "comment" icon
  • Change the grey comment icon on /tests/overview to visualize that the action is there to "comment and/or restart"
  • Trigger the same simple restart that is available next to individual jobs (check the code to be sure whatever it does)
  • Do not add another button on the level of /tests/overview directly
  • Pick a good color combination for the backgrounds of the button
  • Consider a drop-down menu instead of individual buttons
  • Be ready for a longer discussion in pull requests based on your screenshot proposals how it could look like :)

Out of scope

  • Being able to select individual jobs.
  • Restarting without comment. That can come later as part of the parent epic #166556

Related issues 1 (0 open — 1 closed)

Copied to openQA Project - action #167827: Restart multiple jobs with comment over APIResolvedmkittler2024-09-09

Actions
Actions
Copy link
 #1

Updated by livdywan 29 days ago

  • Subject changed from Restart filtered jobs from /tests/overview with comment to Restart filtered jobs from /tests/overview with comment size:M
  • Description updated (diff)
  • Status changed from New to Workable
Actions
Copy link
 #2

Updated by ybonatakis 24 days ago

  • Status changed from Workable to In Progress
  • Assignee set to ybonatakis
Actions
Copy link
 #3

Updated by ybonatakis 23 days ago

I am troubled a bit with the proper design.
We have a modal Elem which calls apiv1_post_comments

I have create two links:

  • Add comment on all currently shown jobs this is exactly what we have until now. onsubmit triggers addComments js with a POST on apiv1_post_comments
  • Add comment on all currently shown jobs with Restart Here either I have to find a way to reuse the addComments and then loop over ids to call restart in one form action or split the actions in two?!

Any suggestions?

Actions
Copy link
 #4

Updated by okurz 23 days ago

ybonatakis wrote in #note-3:

I am troubled a bit with the proper design.
We have a modal Elem which calls apiv1_post_comments

I have create two links:

  • Add comment on all currently shown jobs this is exactly what we have until now. onsubmit triggers addComments js with a POST on apiv1_post_comments
  • Add comment on all currently shown jobs with Restart Here either I have to find a way to reuse the addComments and then loop over ids to call restart in one form action or split the actions in two?!

Any suggestions?

I think the best way is to either extend the existing API method or create an according new API route as target for "Add comment on all currently shown jobs with Restart" and do the iteration over the currently shown jobs there. Don't iterate over the jobs in the Javascript code.

Actions
Copy link
 #5

Updated by tinita 23 days ago · Edited

IMHO restart is the more "important" operation, so I would add a route for batch restarting with comments instead of batch commenting with restart. (e.g. I wouldn't expect a job restart functionality under a /comments endpoint.)

Actions
Copy link
 #6

Updated by ybonatakis 16 days ago

Actions
Copy link
 #7

Updated by ybonatakis 16 days ago

  • Subject changed from Restart filtered jobs from /tests/overview with comment size:M to Restart filtered jobs from /tests/overview with comment
  • Status changed from In Progress to Workable

back to workable due to open discussion and change direction in the implementation https://suse.slack.com/archives/C02AJ1E568M/p1727259601332579 and https://suse.slack.com/archives/C02AJ1E568M/p1727259594106919

Actions
Copy link
 #8

Updated by ybonatakis 16 days ago

  • Assignee deleted (ybonatakis)
Actions
Copy link
 #9

Updated by okurz 15 days ago

  • Subject changed from Restart filtered jobs from /tests/overview with comment to Restart filtered jobs from /tests/overview with comment size:M
Actions
Copy link
 #10

Updated by livdywan 15 days ago · Edited

Discussing this briefly to clarify:

Actions
Copy link
 #11

Updated by ybonatakis 11 days ago

  • Status changed from Workable to In Progress
  • Assignee set to ybonatakis
Actions
Copy link
 #12

Updated by ybonatakis 10 days ago

I need some details of the feature.
Restart filtered jobs from /tests/overview with comment implies to comment the restarted jobs, right? because the feature as is comments the current jobs. or is there a scenario in this feature that the comment is written in the current list of jobs and then restarts the job?

Actions
Copy link
 #13

Updated by tinita 10 days ago

It's still the same requirement as before; comments should be written on the original jobs.
Just the backend API design suggestion has changed.

Actions
Copy link
 #15

Updated by okurz 4 days ago

  • Copied to action #167827: Restart multiple jobs with comment over API added
Actions
Copy link
 #16

Updated by okurz 4 days ago

  • Status changed from In Progress to Blocked

As discussed blocked on #167827

Actions
Copy link
 #17

Updated by mkittler 1 day ago

  • Status changed from Blocked to Workable
Actions
Copy link

Also available in: Atom PDF