[GLASSFISH-16962] Adding an instance to cluster causes false indication to restart other cluster instances Created: 06/Jul/11 Updated: 02/Dec/11 Resolved: 08/Jul/11
|Fix Version/s:||3.1.1_b11, 4.0|
|Remaining Estimate:||Not Specified|
|Time Spent:||Not Specified|
|Original Estimate:||Not Specified|
ogs-3.1.1-b11-07_04_2011-aix.zip, Firefox 5 on Windows XP
When a new instance is added to an existing cluster, status for all previously running instances is changed from "running" to "restart required". Adding an instance to a cluster should not require cluster restart. This is specifically problematic, since user cannot look up the reason for restart required messages for instances other than DAS in Admin Console.
Steps to reproduce:
In Admin Console:
The same status is reported by CLI:
/export/sqe/lidia/glassfish3/glassfish/nodes/localhost-domain1/cl1in1/logs % asadmin list-instances
No errors or warnings are present in DAS server.log and the following warning is present in the first clustered instance server.log (cl1in1):
, reason = The system-property, OSGI_SHELL_TELNET
|Comment by Tom Mueller [ 06/Jul/11 ]|
Confirmed the reported behavior.
The root cause of this problem is that the GenericJavaConfigListener assumes that it is receiving SystemProperty change events only for its own system properties. However, it also receives change events for other instances that are added to the configuration. To fix this, it needs to ignore those events.
|Comment by lidiam [ 06/Jul/11 ]|
Changed Summary to better reflect the problem. Also, this is a regression compared with Glassfish 3.1 b43.
|Comment by Tom Mueller [ 07/Jul/11 ]|
The reason why this bug doesn't show up in 3.1 but does show up in 3.1.1 is because there was a mistake in the implement of the fix for issue
Why fix this issue in 3.1.1?
Which is the targeted build of 3.1.1 for this fix?
Do regression tests exist for this issue?
Which tests should QA (re)run to verify the fix did not destabilize GlassFish?
|Comment by Tom Mueller [ 08/Jul/11 ]|
Fixed in revision 47933 on the trunk
|Comment by lidiam [ 12/Jul/11 ]|
Verified in promoted build ogs-3.1.1-b11-aix.zip.