Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 892628

Summary: RESTAPI: Internal server error 500 on wrong CPU-pinning format
Product: Red Hat Enterprise Virtualization Manager Reporter: Michael Pasternak <mpastern>
Component: ovirt-engine-restapiAssignee: Laszlo Hornyak <lhornyak>
Status: CLOSED NOTABUG QA Contact: Ido Begun <ibegun>
Severity: high Docs Contact:
Priority: unspecified    
Version: 3.2.0CC: bazulay, bdagan, dfediuck, dyasny, ecohen, iheim, mpastern, oramraz, Rhev-m-bugs, sgrinber, ykaul
Target Milestone: ---   
Target Release: 3.2.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: sla
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-02-07 13:27:25 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: SLA RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
patch none

Description Michael Pasternak 2013-01-07 13:12:03 UTC 
Description of problem:

if CPU-pinning format is incorrect will be thrown "Internal server error 500"

Expected results:

400 (bad request) + "CPU-pinning format is incorrect" detail.
Comment 1 Laszlo Hornyak 2013-01-09 17:53:46 UTC 
Michael, for that I need to add resteasy-jaxrs as dependency to restapi-types. Is that acceptable for you?
Comment 2 Laszlo Hornyak 2013-01-09 20:26:00 UTC 
Created attachment 675814 [details]
patch

gerrit is broken at the moment, so here is a patch before I delete the branch
Comment 3 Laszlo Hornyak 2013-01-09 20:55:49 UTC 
meanwhile, gerrit is back

http://gerrit.ovirt.org/10838
Comment 4 Michael Pasternak 2013-01-10 11:35:51 UTC 
(In reply to comment #1)
> Michael, for that I need to add resteasy-jaxrs as dependency to
> restapi-types. Is that acceptable for you?

no, this will create circular dependency + JAXRS ain't relevant for mapping, please see my reply in gerrit.
Comment 5 Laszlo Hornyak 2013-01-15 16:39:10 UTC 
for QE: reproduce by creating a VM and directly in the DB execute

update vm_static set cpu_pinning='FAIL';

after this just fetch the VM resources. When trying to set a wrong format either from restapi or UI, the engine is validating, so this may be the only way to break it.
Comment 6 Laszlo Hornyak 2013-01-15 16:46:10 UTC 
http://gerrit.ovirt.org/11059
Comment 7 Laszlo Hornyak 2013-02-07 13:27:25 UTC 
fixed by another bug
Comment 8 Oded Ramraz 2013-02-07 14:31:18 UTC 
Please set bug resolution to WONTFIX or duplicate ( with the original bug number )

(In reply to comment #7)
> fixed by another bug