Bug 537191

Summary:	Id field of Configuration not serialized when using reflective serialization
Product:	[Other] RHQ Project	Reporter:	John Sanda <jsanda>
Component:	Core Server	Assignee:	Simeon Pinder <spinder>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Corey Welton <cwelton>
Severity:	medium	Docs Contact:
Priority:	low
Version:	1.4	CC:	cwelton
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:	2.4	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2010-08-12 16:50:04 UTC	Type:	---
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:
Bug Blocks:	557793

Description John Sanda 2009-11-12 18:37:31 UTC

Description of problem:
The Configuration class delegates to EntitySerializer to handle the reflective serialization strategy. Configuration.id does not specify the @Column annotation, only the @Id annotation. EntitySerializer does not look for the @Id annotation, and therefore, Configuration.id is not properly serialized with the reflective strategy which I believe is used with the CLI.

I have already modified EntitySerializer to look for the @Id annotation in addition to the other entity annotations it looks for. This fixes Configuration.id as well as any others that declare @Id without also declaring @Column.

The hash for the commit is 68534e1f50d3c054e2dbddfd8e622818db45139b and has been pushed to the raw-config branch.


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


How reproducible:


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


Expected results:


Additional info:

Comment 1 Simeon Pinder 2010-01-18 16:08:36 UTC

Reproduce instructions:

1: Login to CLI as rhqadmin and locate resource id for the root platform(Ex. Linux Platform ... 10001)
2: var platform = ProxyFactory.getResource(10001);
3: Exectue platform.viewProcessList()
* Note the id field of Configuration is always [0] and not serialized before fix.

Error:
Invoking operation viewProcessList
Configuration [0] - null
  processList [249] {

Correct:
Invoking operation viewProcessList
Configuration [12077] - null
  processList [249] {


Fix:
Svn CP commit revision 5275

Comment 2 Corey Welton 2010-01-22 23:58:56 UTC

QA Verified, the Process list is correctly returned.

Comment 3 Corey Welton 2010-08-12 16:50:04 UTC

Mass-closure of verified bugs against JON.