Bug 998354

Summary:	Update developer guide to reflect requirement to write new controllers for Flask
Product:	[Retired] Beaker	Reporter:	Raymond Mancy <rmancy>
Component:	Doc	Assignee:	Amit Saha <asaha>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Nick Coghlan <ncoghlan>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	develop	CC:	aigao, asaha, dcallagh, ebaak, llim, qwan, rmancy
Target Milestone:	0.15	Keywords:	Documentation
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-10-03 02:28:25 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:

Description Raymond Mancy 2013-08-19 05:55:46 UTC

Description of problem:

We can now run both cherrypy and flask controllers. However we should advise
against writing new controllers in cherrypy.

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


How reproducible:

Always

Steps to Reproduce:
1. Open  dev guide
2. Nothing mentioned about writing contorollers for flask
3.

Actual results:

It's not there

Expected results:

Should be.

Additional info:

Comment 2 Nick Coghlan 2013-08-26 02:58:07 UTC

This ties back in to needing a high level style guide in the developer guide :)

Comment 3 Raymond Mancy 2013-08-26 04:51:47 UTC

We don't even need to wait for that. We already talk about the TG stack and what does what in it. We could just add a small entry for flask, and then either
in the line for cherrypy or as an side, mention that creation of new cherrypy controllers has been deprecated for flask controllers.

Comment 4 Amit Saha 2013-09-16 07:20:21 UTC

The mailing list discussion has some points which will be helpful if made part of the style guide: https://lists.fedorahosted.org/pipermail/beaker-devel/2013-August/000676.html

Comment 5 Amit Saha 2013-09-18 05:03:16 UTC

I forgot to add the gerrit patch: http://gerrit.beaker-project.org/#/c/2258/

Comment 6 Nick Coghlan 2013-09-19 07:23:03 UTC

Guidelines published at http://beaker-project.org/dev/guide/code-guidelines.html

Comment 7 Nick Coghlan 2013-10-03 02:28:25 UTC

Beaker 0.15 has been released.