Bug 619135
Summary: | manually adding a resource with bad plugin config hoses inventory for that resource | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Other] RHQ Project | Reporter: | John Mazzitelli <mazz> | ||||||
Component: | Agent | Assignee: | Jay Shaughnessy <jshaughn> | ||||||
Status: | CLOSED CURRENTRELEASE | QA Contact: | Mike Foley <mfoley> | ||||||
Severity: | high | Docs Contact: | |||||||
Priority: | low | ||||||||
Version: | 3.0.0 | CC: | jshaughn | ||||||
Target Milestone: | --- | ||||||||
Target Release: | --- | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2013-09-02 07:16:10 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: | |||||||||
Attachments: |
|
Description
John Mazzitelli
2010-07-28 16:29:08 UTC
I've recreated this. The issue seems to occur when the manually added resource component can't be started. The new resource itself passes the manual discovery, meaning the plugin config seems valid (if not correct). The new resource is committed to agent inventory at this point. Then we make an attempt to start the resource component. If this throws an exception due to incorrect plugin config, inability to create a connection (underlying resource is down, for example), or whatever, we exit the manual add code prior to syncing the new resource with the server. This leaves the ghost entry in the agent inventory without any server side knowledge of the resource. I think the fix would be to allow a failed component start and just proceed. The manually added resource should show up on the server, in a DOWN state. The user can then fix the connection properties or uninventory the poorly defined resource. Looking at generatig a patch for this now... Created attachment 435122 [details]
proposed patch for this issue
Created attachment 435126 [details]
Same patch with a typo fixed
jshaughn -- has this patch been applied? if so, which branches It is applied to Master. It is not applied to release-3.0.0. It looks like this was actually fixed in master, so setting to on_qa. verified 10/4/2011 by following the reproduction steps. Bulk closing of issues that were VERIFIED, had no target release and where the status changed more than a year ago. |