Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FISH-7132 : config refs list being handled #6480

Conversation

luiseufrasio
Copy link
Contributor

Description

In general, JMS Service availability doesn’t work when setting up a JMS cluster in LOCAL mode when using deployment groups unlike traditional clusters (generally known as Shoal clusters).

Important Info

Blockers

None

Testing

New tests

None

Testing Performed

Steps:

  1. build Payara 6 Community
  2. run it
  3. Run the following commands using asadmin:
copy-config default-config mqc-config
set configs.config.mqc-config.availability-service.jms-availability.availability-enabled=true
set configs.config.mqc-config.jms-service.type=LOCAL

create-node-ssh --sshport=22 --nodehost=localhost --installdir=${com.sun.aas.productRoot} --sshuser=${user.name} remote-node-1
create-node-ssh --sshport=22 --nodehost=localhost --installdir=${com.sun.aas.productRoot} --sshuser=${user.name} remote-node-2

create-deployment-group dg1
create-instance --deploymentgroup=dg1 --node=remote-node-1 --config=mqc-config i1
create-instance --deploymentgroup=dg1 --node=remote-node-2 --config=mqc-config i2
start-deployment-group dg1

  1. open server.log files for the 2 instances: i1 and i2
  2. verify that it is exposing JMS Service in the 2 hosts like done in traditional clusters:
ADDRESSLIST in setJmsServiceProvider : mq://localhost:27677/,mq://localhost:27677/]]
JMS Service Connection URL is : mq://localhost:27677/,mq://localhost:27677/]] 

Testing Environment

Zulu JDK 11 on Windows 11 with Maven 3.8.4

@luiseufrasio
Copy link
Contributor Author

jenkins test please

Copy link
Contributor

@jGauravGupta jGauravGupta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Worked as described in PR:

[2023-11-14T00:01:58.596+0530] [Payara 6.2023.11-SNAPSHOT] [INFO] [AS-JMS-CORE-00002] [javax.enterprise.resource.jms] [tid: _ThreadID=48 _ThreadName=RunLevelControllerThread-1699900298800] [timeMillis: 1699900318596] [levelValue: 800] [[
  ADDRESSLIST in setJmsServiceProvider : mq://localhost:27676/,mq://localhost:27677/]]

[2023-11-14T00:01:58.599+0530] [Payara 6.2023.11-SNAPSHOT] [INFO] [AS-JMS-CORE-00001] [javax.enterprise.resource.jms] [tid: _ThreadID=48 _ThreadName=RunLevelControllerThread-1699900298800] [timeMillis: 1699900318599] [levelValue: 800] [[
  JMS Service Connection URL is : mq://localhost:27676/,mq://localhost:27677/]]

@breakponchito
Copy link
Contributor

already tested, LGTM

i1

`
[2023-11-13T19:40:37.742-0600] [Payara 6.2023.11-SNAPSHOT] [INFO] [AS-JMS-CORE-00002] [javax.enterprise.resource.jms] [tid: _ThreadID=156 _ThreadName=Recovery Helper Thread] [timeMillis: 1699926037742] [levelValue: 800] [[
ADDRESSLIST in setJmsServiceProvider : mq://localhost:27676/,mq://localhost:27677/]]

[2023-11-13T19:40:37.743-0600] [Payara 6.2023.11-SNAPSHOT] [INFO] [AS-JMS-CORE-00001] [javax.enterprise.resource.jms] [tid: _ThreadID=156 _ThreadName=Recovery Helper Thread] [timeMillis: 1699926037743] [levelValue: 800] [[
JMS Service Connection URL is : mq://localhost:27676/,mq://localhost:27677/]]

`

i2

`
[2023-11-13T19:40:35.961-0600] [Payara 6.2023.11-SNAPSHOT] [INFO] [AS-JMS-CORE-00002] [javax.enterprise.resource.jms] [tid: _ThreadID=154 _ThreadName=Recovery Helper Thread] [timeMillis: 1699926035961] [levelValue: 800] [[
ADDRESSLIST in setJmsServiceProvider : mq://localhost:27677/,mq://localhost:27676/]]

[2023-11-13T19:40:35.961-0600] [Payara 6.2023.11-SNAPSHOT] [INFO] [AS-JMS-CORE-00001] [javax.enterprise.resource.jms] [tid: _ThreadID=154 _ThreadName=Recovery Helper Thread] [timeMillis: 1699926035961] [levelValue: 800] [[
JMS Service Connection URL is : mq://localhost:27677/,mq://localhost:27676/]]

`

Copy link
Contributor

@breakponchito breakponchito left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants