Bug 1399195

Summary: [GSS](6.4.z) RequestCorrelator "channel is not connected" error during shutdown
Product: [JBoss] JBoss Enterprise Application Platform 6 Reporter: Shay Matasaro <smatasar>
Component: ClusteringAssignee: Peter Palaga <ppalaga>
Status: CLOSED CURRENTRELEASE QA Contact: Jiří Bílek <jbilek>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 6.4.5CC: bmaxwell, paul.ferraro, ppalaga, rnetuka, rstancel, smatasar
Target Milestone: CR1   
Target Release: EAP 6.4.16   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-06-22 09:19:10 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: 1387585, 1434495    

Description Shay Matasaro 2016-11-28 14:24:39 UTC
https://issues.jboss.org/browse/JGRP-1958

Comment 1 Peter Palaga 2017-02-09 15:08:41 UTC
PR sent https://github.com/belaban/JGroups/pull/343

Comment 2 Peter Palaga 2017-02-09 15:38:07 UTC
Note that there is no upstream required, because the backported commit [1] https://github.com/belaban/JGroups/commit/03d7c93145f6a9314c2380f5ad66bdc8ec4209f0 is a part of both JGroups 3.6.10 (in EAP 7.0) and 3.6.12 (in EAP 7.1)

Comment 5 Jiří Bílek 2017-06-08 13:34:28 UTC
Verified with EAP 6.4.16.CP.CR1

Comment 7 Shay Matasaro 2017-06-14 13:16:00 UTC
i also thought that it would be WARN or even lower since as far as i understand it has no adverse effect , and  its just a result of the shutdown  process.

Is the correct?

Comment 8 Jiří Bílek 2017-06-14 16:33:02 UTC
I think so.
In that case I have to reopen this BZ. Because the fix is just particular.
The stack trace is not shown. Message is SEVERE but should be WARN or lower.

Comment 9 Peter Palaga 2017-06-19 16:08:25 UTC
I interpreted the sparse description if this bug as "backport the fix for JGRP-1958" which I did. If you are not content with the original fix https://github.com/belaban/JGroups/commit/03d7c93145f6a9314c2380f5ad66bdc8ec4209f0 then apparently, my interpretation was wrong :)

This being a backport of an upstream fix that was delivered ~18 month ago, I do not think reopening this bug can bring anything useful. I simply cannot put a new behavior to a maintenance branch that is not present in master. That would require a new JGroups Jira.

I propose that (1) Jirka accepts this as a correct backport of an imperfect upstream fix and (2) Shay files a new BZ with proper acceptance criteria that will result in creating a new upstream Jira that will request the change in the log level in master and all relevant maintenance branches. WDYT?

Comment 11 Jiří Bílek 2017-06-20 15:29:43 UTC
Verified because backport was successfuly done.

Comment 12 Shay Matasaro 2017-06-20 18:03:31 UTC
ok we can close this , I opened :
https://bugzilla.redhat.com/show_bug.cgi?id=1463387

Comment 13 Petr Penicka 2017-06-22 09:19:10 UTC
Released on June 20 2017 as part of the EAP 6.4.16 maintenance release.