Red Hat Bugzilla – Bug 1263917
implement the new job page design
Last modified: 2018-02-05 19:41:31 EST
Implement the new job page design as described here:
New HTTP APIs:
New job page implementation:
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.
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).
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:
After some discussions on the mailing list  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:
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.
This bug was accidentally moved from POST to MODIFIED via an error in automation, please see firstname.lastname@example.org with any questions
The results-reporting-improvements-take3 branch has been merged to develop.
*** Bug 1013445 has been marked as a duplicate of this bug. ***
I'm assuming this was meant to go to VERIFIED not MODIFIED..
Beaker 23.0 has been released.