Bug 1263917 - implement the new job page design
Summary: implement the new job page design
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: general
Version: 21
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 23.0
Assignee: Dan Callaghan
QA Contact: tools-bugs
URL:
Whiteboard:
: 1013445 (view as bug list)
Depends On:
Blocks: 646416 853351 894137 967481 995009 1064738 1122682 1169838 1305951
TreeView+ depends on / blocked
 
Reported: 2015-09-17 05:39 UTC by Dan Callaghan
Modified: 2023-07-17 00:59 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-07-07 23:11:40 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1263918 0 unspecified CLOSED implement the new recipe page design 2021-02-22 00:41:40 UTC
Red Hat Issue Tracker BKR-3181 0 None None None 2022-05-27 00:41:40 UTC

Internal Links: 1263918

Description Dan Callaghan 2015-09-17 05:39:13 UTC
Implement the new job page design as described here:

https://beaker-project.org/dev/proposals/job-page-improvements.html

Comment 1 Dan Callaghan 2015-09-30 23:48:34 UTC
New HTTP APIs:
http://gerrit.beaker-project.org/4406

New job page implementation:
http://gerrit.beaker-project.org/4407

There is still one big missing piece of functionality: reviews (ack/nak/comments). The team has discussed this already to some length but we are still not sure what is the best way forwards, in terms of modelling the new review system and what features it should have and how the UI should behave.

The current design proposal kind of handwaves past that, but it will need some more attention, perhaps in a separate BZ.

Comment 2 Dan Callaghan 2015-10-13 01:35:48 UTC
The HTTP APIs on the server side of things are merged now, which should make it easier to work on some other related patches. The actual job page UI itself still needs some decisions about the model and UI for reviewing (ack/nak/comments).

Comment 3 Dan Callaghan 2015-10-15 03:40:43 UTC
I noticed in my devel environment that the system page was giving a 500 if a hardware scan was in progress. It was caused by a circular reference in JSON for system when a hardware scan recipe was running (system -> recipe -> system) accidentally introduced by the job API patch.

Fix is here:
http://gerrit.beaker-project.org/4441

Comment 4 Dan Callaghan 2015-11-12 02:12:56 UTC
After some discussions on the mailing list [1] we have settled on an approach for the ack/nak/commenting problem.

Recipe set comments will be separated into a fully-fledged commenting system. For this bug it will support recipe set comments only but they will be expanded to recipes/tasks in subsequent patches.

"Nak" will become "waived", "ack" will be the default. We will have a new optional mechanism for users to track whether they have reviewed a given recipe or not, inspired by Gerrit's reviewed checkboxes.

So now, with the original patch implementing the whole new UI:

http://gerrit.beaker-project.org/4407

plus some extra supporting patches:

http://gerrit.beaker-project.org/4480 extract recipe set comments into a separate table
http://gerrit.beaker-project.org/4481 support for tracking per-user recipe reviewed state
http://gerrit.beaker-project.org/4482 track per-user recipe reviewed state
http://gerrit.beaker-project.org/4486 job page: add UI for commenting on recipe sets

this is ready to go.

[1] http://thread.gmane.org/gmane.comp.systems.beaker.devel/1302

Comment 5 Mike McCune 2016-03-28 22:26:18 UTC
This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions

Comment 6 Dan Callaghan 2016-04-05 07:54:11 UTC
The results-reporting-improvements-take3 branch has been merged to develop.

Comment 8 Dan Callaghan 2016-04-18 07:32:03 UTC
*** Bug 1013445 has been marked as a duplicate of this bug. ***

Comment 10 Dan Callaghan 2016-04-26 23:31:22 UTC
I'm assuming this was meant to go to VERIFIED not MODIFIED..

Comment 11 Dan Callaghan 2016-07-07 23:11:40 UTC
Beaker 23.0 has been released.

Comment 12 Hison 2022-05-27 00:36:16 UTC Comment hidden (spam)

Note You need to log in before you can comment on or make changes to this bug.