Bug 834353

Summary: [as5-plugin] BRMS is not recognized as a product type
Product: [Other] RHQ Project Reporter: Ian Springer <ian.springer>
Component: PluginsAssignee: Lukas Krejci <lkrejci>
Status: CLOSED CURRENTRELEASE QA Contact: Mike Foley <mfoley>
Severity: high Docs Contact:
Priority: unspecified    
Version: 4.4CC: ccrouch, hrupp, jshaughn, lkrejci, loleary
Target Milestone: ---   
Target Release: RHQ 4.5.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 4.5 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 834357 EMBJOPR-373 (view as bug list) Environment:
Last Closed: 2013-09-01 09:59:40 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 834357, 834360    
Attachments:
Description Flags
brms
none
brms none

Description Ian Springer 2012-06-21 15:53:46 UTC
We end up efaulting the product type to "AS", and then the default Resource name, description, and version erroneously contain "AS", rather than "BRMS".

Comment 1 Ian Springer 2012-06-21 15:58:20 UTC
Fixed in master:

http://git.fedorahosted.org/git/?p=rhq/rhq.git;a=commitdiff;h=cb31626

Name, description, and version now get discovered as follows for BRMS 5.3.0 ER8:

Name: BRMS jetengine:1099 default
Description: JBoss Business Rules Management System 5
Version: BRMS 5.3.0.BRMS-ER8

Comment 2 Armine Hovsepyan 2012-08-02 15:23:59 UTC
Created attachment 601985 [details]
brms

reopened.

clean rhq-server.4.5 (last build) server was installed with only BRMS plugin. BRMS server started. Inventored as EWP server, while in description it is said that it should be BRMS.

Comment 3 Armine Hovsepyan 2012-08-02 15:38:03 UTC
Created attachment 601987 [details]
brms


clean rhq (last build) server was installed with only BRMS plugin. BRMS server started. Inventored as EWP server, while in description it is said that it should be BRMS.

Comment 4 Simeon Pinder 2012-08-07 16:10:15 UTC
This is a problem but it seems to be specific to the version of BRMS being tested. Discovered type is correct for BRMS 5.3.0 but not for 5.1.0 where EWS is instead being detected instead.  Probably need to tweak the type lookup code a bit here.

Comment 5 Simeon Pinder 2012-08-07 16:38:05 UTC
With a little bit more research it looks like both BRMS 5.1.0, 5.2.0 and earlier were actually based on JBoss EWS implementations to the version lookup code is working but discovering that it's a customized EWS instance used for BRMS < 5.3. 

We need to figure out what the earliest BRMS version that we support is to determine if this is because of an unsupported version?

Or we need to update the discovery code to figure out when we've discovered BRMS+EWS instead of just EWS?

Comment 6 Larry O'Leary 2012-08-07 19:22:19 UTC
The plug-in must support BRMS 5.2.x and later.

Comment 7 Simeon Pinder 2012-08-08 19:42:37 UTC
So if they run with 5.1.x then it's an unsupported configuration.  If they run with 5.3.x then the plugin discovery works correctly.  If they run with BRMS server 5.2.x then we're going to be confused as to which server we should be listing: a) EWS 5.2.x or BRMS 5.2.x. 

We currently look up the JBoss release id from the jboss instance being run to determine EWS, BRMS, etc.

11:48:55,104 INFO  [ServerImpl] Starting JBoss (Microcontainer)...
11:48:55,104 INFO  [ServerImpl] Release ID: JBoss [BRMS] 5.3.0.GA (build: SVNTag=5.3.0.GA date=201206201517)
...

Release ID: JBoss [EWP] 5.1.0 (build: SVNTag=JBPAPP_5_1_0 date=201009150134)

BRMS is correctly relabelling for 5.3.x but not for earlier releases.  

So we're going to have to try to try to figure out how the BRMS instance is different from EWS alone by some other means.  Such an approach could be brittle. 

Can you weigh in on this issue charles? This could be a bigger problems for SOA, JDG, etc if they're doing the same thing with their earlier releases.

Comment 8 Lukas Krejci 2012-08-17 17:59:03 UTC
master http://git.fedorahosted.org/cgit/rhq/rhq.git/diff/?id=30c5e81759f5166873e49fd3486614c282203f47
Author: Lukas Krejci <lkrejci>
Date:   Fri Aug 17 19:20:48 2012 +0200

    [BZ 834353] - Distinguish between BRMS 5.(1|2).0 and EWP 5.1.x by looking
    for BRMS specific jars in the client directory.

Comment 10 Heiko W. Rupp 2013-09-01 09:59:40 UTC
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.