Bug 823558 - [as7] check that server's basedir has not changed since discovery needs to be added to server components' start() method
[as7] check that server's basedir has not changed since discovery needs to be...
Product: RHQ Project
Classification: Other
Component: Plugins (Show other bugs)
Unspecified Unspecified
high Severity high (vote)
: ---
: RHQ 4.5.0
Assigned To: Charles Crouch
Mike Foley
Depends On:
Blocks: as7-plugin 824019
  Show dependency treegraph
Reported: 2012-05-21 10:23 EDT by Ian Springer
Modified: 2015-02-01 18:28 EST (History)
3 users (show)

See Also:
Fixed In Version: 4.5
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 824019 (view as bug list)
Last Closed: 2013-09-01 06:10:00 EDT
Type: Bug
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 Ian Springer 2012-05-21 10:23:32 EDT
This check is important, because the basedir is the resource key, and so if that changes, it's no longer the same server Resource, and start() should throw a InvalidPluginConfigurationException, which will result in a yellow triangle in the GUI.

We have such a check in the as4 and as5 plugins.

We should also move the launch-type and product-name checks that are currently in getAvailablity() to start().
Comment 1 Ian Springer 2012-05-21 12:15:12 EDT
Fixed in master:

Comment 2 Ian Springer 2012-05-21 13:15:06 EDT
Adding a fix to the original fix - getAvailability() now throws InvalidPluginConfigurationException when validation of server attributes fails...

Comment 3 Ian Springer 2012-05-21 13:42:06 EDT
Adding another fix to the original fix - start() now initializes this.connection before calling getAvailability()...

Comment 4 Ian Springer 2012-05-22 18:38:56 EDT
Adding itests that test the server attribute validation that is now done in BaseServerComponent.start(), and make sure to null out this.lastAvail in BaseServerComponent.stop() so the attribute validation is guaranteed to happen during a component restart:

Comment 7 Heiko W. Rupp 2013-09-01 06:10:00 EDT
Bulk closing of items that are on_qa and in old RHQ releases, which are out for a long time and where the issue has not been re-opened since.

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