Bug 961877 - RFE: DiscoveryBoss.manuallyAddResource CLI call should use default plugin configuration when possible
Summary: RFE: DiscoveryBoss.manuallyAddResource CLI call should use default plugin con...
Keywords:
Status: NEW
Alias: None
Product: RHQ Project
Classification: Other
Component: CLI
Version: 4.5
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: ---
Assignee: Nobody
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-05-10 15:32 UTC by Elias Ross
Modified: 2022-03-31 04:27 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
Embargoed:


Attachments (Terms of Use)
patch (for 4.5.1) (2.65 KB, application/octet-stream)
2013-05-10 15:32 UTC, Elias Ross
no flags Details
Patch for 7e512faae456 (3.22 KB, patch)
2013-06-19 18:49 UTC, Elias Ross
no flags Details | Diff

Description Elias Ross 2013-05-10 15:32:30 UTC
Created attachment 746211 [details]
patch (for 4.5.1)

DiscoveryBoss.manuallyAddResource should use default plugin configuration keys when not indicated.

Alternatively, the CLI should support obtaining the default plugin configuration, allowing the settings to be overwritten on the CLI side.

This is what I came up with (attached).

Comment 1 Elias Ross 2013-06-19 18:49:40 UTC
Created attachment 763101 [details]
Patch for 7e512faae456

Comment 2 Heiko W. Rupp 2013-06-28 13:29:43 UTC
Elias,
thanks for the patch - I still need to try the implications better -- especially in cases when there are more than one template and the user choose one of those and only supplied one of the values - here the other values would need to be taken from the other template instead of the default ?

Thanks
  Heiko

Comment 3 Elias Ross 2013-06-28 20:16:40 UTC
Yeah, I'm not sure what to do about multiple configuration templates. I forgot that RHQ had them :-)

Since the client can pull the configuration template using the CLI (which I didn't figure out), might make sense to not have this change.

Or maybe it would be good to have an overloaded method?

DiscoveryBoss.manuallyAddResource(resourceType.id, parent.id, config, /*new*/ templateId)


Note You need to log in before you can comment on or make changes to this bug.