Bug 819612

Summary: [plugin container] add validation of the metric data reported by plugins
Product: [Other] RHQ Project Reporter: Ian Springer <ian.springer>
Component: Plugin ContainerAssignee: RHQ Project Maintainer <rhq-maint>
Status: NEW --- QA Contact: Mike Foley <mfoley>
Severity: high Docs Contact:
Priority: medium    
Version: 4.4CC: hbrock, hrupp
Target Milestone: ---Keywords: FutureFeature
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Description Ian Springer 2012-05-07 13:49:49 EDT
Some things we can check and log plugin-error warnings (or even throw exceptions) for:

- metrics in the report are a subset of the metrics that were requested 
- the MeasurementScheduleRequests fields in the XxxDataXxx objects in the report are equivalent objects to the ones that were passed to the plugin's getValues() method
- metrics in the report have the correct data type - e.g. if a numeric metric named "foo" was requested, make sure the report contains a numeric metric "foo", and not a trait "foo" or call-time metric "foo".
- the report's collectionTime field has not been modified by the plugin - this field really should not even be exposed by the plugin API...
Comment 1 Mike Foley 2012-05-08 15:33:33 EDT
per BZ triage ... crouch, loleary, foley