Bug 754419 - Configuration objects required to specifically contain required properties even though those are required to have default values
Configuration objects required to specifically contain required properties ev...
Status: NEW
Product: RHQ Project
Classification: Other
Component: Core Server (Show other bugs)
Unspecified Unspecified
medium Severity medium (vote)
: ---
: ---
Assigned To: RHQ Project Maintainer
Mike Foley
: FutureFeature
Depends On:
Blocks: 734866
  Show dependency treegraph
Reported: 2011-11-16 07:49 EST by Lukas Krejci
Modified: 2011-12-15 10:35 EST (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of:
Last Closed:
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Lukas Krejci 2011-11-16 07:49:53 EST
Description of problem:
Throughout our codebase, we have this basic assumption - if a configuration property is required, we assume it exists in the supplied configuration object. 

Now from a usability point of view, there should be no need to supply those unless their values differ from the default value specified in the configuration definition. 

This is of course not a problem when accessing RHQ through GUI, because GUI takes care of that, but there is a number of remote API methods that either directly or indirectly use a Configuration class in their parameters. A remote user shouldn't have to be required to slavishly repeat the default values on required properties.

The "impact surface" of this bug is quite large - basically the whole remote API needs to be checked if the arguments of all methods don't specify Configuration (or objects having configuration as one of their fields, recursively).

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

How reproducible:

Steps to Reproduce:
See bug https://bugzilla.redhat.com/show_bug.cgi?id=734866
Actual results:
NPEs and other unexpected effects

Expected results:
our impls should be clever enough to replace the missing properties with their default values

Additional info:
Comment 1 Lukas Krejci 2011-11-16 08:17:12 EST
Just a note that this is sort of an umbrella bug for the individual issues yet to be discovered in the individual remote APIs. The bug 734866 mentioned in the repro steps is merely the first BZ raised that falls under this umbrella.

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