Bug 872094

Summary:	avoid using cookies to store visible state of recipe results on recipe/job page
Product:	[Retired] Beaker	Reporter:	Amit Saha <asaha>
Component:	web UI	Assignee:	Dan Callaghan <dcallagh>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	0.9	CC:	dcallagh, dowang, ebaak, rjoost, tools-bugs
Target Milestone:	23.0	Keywords:	FutureFeature, Patch
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	ImplementationQuality
Fixed In Version:		Doc Type:	Enhancement
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-07-07 23:10:41 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:	1263918
Bug Blocks:

Description Amit Saha 2012-11-01 08:17:51 UTC

Description of problem:

The current code for displaying the job details (/jobs/jobid) uses cookies (one per recipe) to maintain information regarding what results to display -- all,failed or none. 

While working on the solution to https://bugzilla.redhat.com/show_bug.cgi?id=674025, it was felt that a cookie-less solution may result in a better implementation to not only this bug, but the overall logic used in fetching the results for recipes. For example, if only a particular task's result is requested, it will only fetch the results of that recipe and not all the other recipes in the job.

The alternative suggested by Dan was using the History API.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:

See above.

Additional info:

Comment 4 Dan Callaghan 2016-03-23 02:59:36 UTC

This is implemented for the new recipe page design.

Expanded/collapsed tasks are reflected in the URL anchor, as in /recipes/123#task4,task5 with the usual fallback to localStorage in case the user re-opens the page with no anchor.

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

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:55:25 UTC

The results-reporting-improvements-take3 branch has been merged to develop.

Comment 9 Dan Callaghan 2016-07-07 23:10:41 UTC

Beaker 23.0 has been released.