Bug 1263918

Summary: implement the new recipe page design
Product: [Retired] Beaker Reporter: Dan Callaghan <dcallagh>
Component: generalAssignee: matt jia <mjia>
Status: CLOSED CURRENTRELEASE QA Contact: tools-bugs <tools-bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 21CC: dcallagh, dowang, mjia, rjoost
Target Milestone: 23.0Keywords: FutureFeature, Patch
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-07-07 23:09:59 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 991245    
Bug Blocks: 602131, 626529, 786793, 858911, 872094    

Description Dan Callaghan 2015-09-17 05:40:18 UTC
Implement the new recipe page design, once it is finalised.

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

Comment 1 matt jia 2016-02-05 03:16:29 UTC
The first patch is on Gerrit which covers most of the features that present in the job page improvements.


    http://gerrit.beaker-project.org/#/c/4657/
   

Note: this patch does not show "Possible systems" when a recipe is Queued, so we need to find a solution for it.

Comment 2 Dan Callaghan 2016-02-05 04:03:37 UTC
We will also need to come up with something for the existing "Report a problem with this system" button.

Comment 3 matt jia 2016-03-14 06:05:55 UTC
(In reply to matt jia from comment #1)
> The first patch is on Gerrit which covers most of the features that present
> in the job page improvements.
> 
> 
>     http://gerrit.beaker-project.org/#/c/4657/
>    
> 

I have updated this patch with 

http://gerrit.beaker-project.org/#/c/4723/

And other extra supporting patches

http://gerrit.beaker-project.org/#/c/4724/ add db support for commenting on individual recipe tasks and results

http://gerrit.beaker-project.org/#/c/4725/ HTTP APIs for recipe task and result comments 

http://gerrit.beaker-project.org/#/c/4726/ recipe page: add UI for commenting on recipe tasks and results

Comment 4 Dan Callaghan 2016-03-14 06:32:46 UTC
We are getting close now :-)

Summary of remaining issues that need to be fixed on this branch before the recipe page is considered fully implemented:

* link to "possible systems" for a queued recipe
* show aborted/cancelled time for a recipe that was aborted or cancelled without starting
* show export links in the top right, matching the job page (also implement the XML generating endpoints)
* show "Clone Recipe Set" button in the page header, matching the "Clone" button on the job page
* "Netboot configured" item in installation progress is currently using installation.created timestamp, but it needs to look in installation.commands to find the 'configure_netboot' command use that command's .created timestamp instead

We also need to decide what to do about "Report problem with system" button, either find an acceptable place for it to live or drop it.

Comment 5 matt jia 2016-03-17 00:46:59 UTC
(In reply to Dan Callaghan from comment #4)
> We are getting close now :-)

I have put up patches to fix the blow issues.

> Summary of remaining issues that need to be fixed on this branch before the
> recipe page is considered fully implemented:
> 
> * link to "possible systems" for a queued recipe

  http://gerrit.beaker-project.org/#/c/4735/

> * show aborted/cancelled time for a recipe that was aborted or cancelled
> without starting
  
  http://gerrit.beaker-project.org/#/c/4739/

> * show export links in the top right, matching the job page (also implement
> the XML generating endpoints)

  http://gerrit.beaker-project.org/#/c/4736/

> * show "Clone Recipe Set" button in the page header, matching the "Clone"
> button on the job page

  http://gerrit.beaker-project.org/#/c/4737/

> * "Netboot configured" item in installation progress is currently using
> installation.created timestamp, but it needs to look in
> installation.commands to find the 'configure_netboot' command use that
> command's .created timestamp instead

http://gerrit.beaker-project.org/#/c/4738/

> 
> We also need to decide what to do about "Report problem with system" button,
> either find an acceptable place for it to live or drop it.

We have decided to not do this because there is no good place for it on the recipe page. People can still click through the link that links to the system page to report problem. We will revisit this again if we have demands for it in the future.

Comment 6 Mike McCune 2016-03-28 23:06:24 UTC
This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions

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

Comment 10 Dan Callaghan 2016-07-07 23:09:59 UTC
Beaker 23.0 has been released.