Bug 743499

Summary: Errors creating lab controllers are not handled properly
Product: [Retired] Beaker Reporter: Raymond Mancy <rmancy>
Component: web UIAssignee: Raymond Mancy <rmancy>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 0.7CC: bpeck, dcallagh, ebaak, mcsontos, mishin, rmancy, stl
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-11-01 05:08:25 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Raymond Mancy 2011-10-05 06:49:31 UTC 
Description of problem:
Errors creating lab controllers are not handled properly

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


How reproducible:


Steps to Reproduce:
1.go to /labcontrollers/new
2.Enter details, ensuring to use an email address that already exists
3.Save
  
Actual results:

500 ISE


Expected results:

It works


Additional info:

the save() method in bkr.model.labcontroller uses the edit()
method as an error_handler. The edit() method expects a valid LC id,
and if you fail to create an LC you don't have one.
Comment 1 Dan Callaghan 2012-11-01 05:08:25 UTC 
I think this is fixed now.

http://git.beaker-project.org/cgit/beaker/commit/?id=4cecff95