Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1139102 - The node-identifier default value uses invalid dash char '-'
The node-identifier default value uses invalid dash char '-'
Status: VERIFIED
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: Transaction Manager (Show other bugs)
6.3.0
Unspecified Unspecified
unspecified Severity unspecified
: DR12
: EAP 6.4.0
Assigned To: tom.jenkinson
Hayk Hovsepyan
:
Depends On: 1143956
Blocks:
  Show dependency treegraph
 
Reported: 2014-09-08 01:11 EDT by Takayoshi Kimura
Modified: 2017-10-09 20:06 EDT (History)
4 users (show)

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


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
JBoss Issue Tracker JBTM-1341 Major Closed Reject setting node identifier on CoreEnvironmentBean unless it can be successfully used by TxControl 2018-09-19 11:40 EDT

  None (edit)
Description Takayoshi Kimura 2014-09-08 01:11:27 EDT
The node-identifier doesn't accept dash char '-' but generated default value contains that char becuase the Utility.getpid() returns signed int. See the following inconsistent error message.

> 11:40:01,575 WARN  [com.arjuna.ats.arjuna] (ajp-/0.0.0.0:8009-1) ARJUNA012132: Supplied name of node contains reserved character -. Using Arjuna:-2044534771

Workaround exists, set correct node-identifier value manually.
Comment 1 tom.jenkinson 2014-09-08 03:08:15 EDT
Hi Takayoshi,

In production we should always recommend setting the node-identifier manually. What configuration are you using? My gut reaction is to ask you to make this a docs bug to ensure it is made clear there. It would be useful to link to the section where it is suggested it is OK to not manually set the node-identifier.

Tom
Comment 2 Takayoshi Kimura 2014-09-08 03:46:56 EDT
Yes we always recommend to set it manually.

The story was, the customer assigned ${jboss.node.name} to the node-identifier, but the node name was like "jboss-node1" and got this WARN message. The customer fixed the name, so it's no longer a problem but wonders about this puzzling WARN message.

The config is pretty much EAP 6 default, uuid process id.
Comment 3 tom.jenkinson 2014-09-08 06:43:17 EDT
Hi Takayoshi,

I think the fix should be to fail to start up if an invalid node name is specified. TBH I thought it did that but it looks like that is just in master. If you are happy I will modify this Jira to address that.

Tom
Comment 4 Takayoshi Kimura 2014-09-08 20:50:55 EDT
Yes I agree with you, we need validation at boot time.

1. Invalid node-identifier specified it should fail at boot.
2. No node-identifier specified, warn it, generate default.

What do we do in #2 scenario? In some cases XA is not used at all so we may allow no node-identifier configuration but it would be a problem when XA is actually used.
Comment 5 tom.jenkinson 2014-09-09 05:19:32 EDT
Hi Takayoshi,

There is a default of "1" allocated in the integration code. The times to override that are if there are multiple servers accessing the same resource manager or they are using JTS. I guess one of these scenarios is true for your customer and hence they have overriden the default.

Thanks,
Tom
Comment 6 Takayoshi Kimura 2014-09-09 20:45:15 EDT
Ok, I'm fine as long as we WARN at boot time if the value remains the default "1".
Comment 7 JBoss JIRA Server 2014-09-16 10:55:10 EDT
Tom Jenkinson <tom.jenkinson@redhat.com> updated the status of jira JBTM-1341 to Closed
Comment 8 tom.jenkinson 2014-09-16 10:57:14 EDT
I have fixed this upstream, the validation about the "-" was not required anyway but I validate the length of the identifier at startup which is required
Comment 9 tom.jenkinson 2014-12-05 08:48:47 EST
Fixed in 4.17.23
Comment 10 Hayk Hovsepyan 2014-12-08 05:10:01 EST
Verified on revision EAP 6.4.0.DR12.

Now when node-identifier is not set, it shows warning message "Node identifier property is set to the default value. Please make sure it is unique." in startup.

It checks the length of node-identifier on startup. "Values must have a maximum length of 23 characters" error is shown.

It is possible to specify node-identifier with dash char '-' in the name.

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