Bug 908720 - Enabling cluster connections attribute "Direct connections" should be allowed only for connections with static-connectors (or raise some more specific error message)
Summary: Enabling cluster connections attribute "Direct connections" should be allowed...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: Web Console
Version: 6.1.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ER2
: EAP 6.1.1
Assignee: baranowb
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 975403
TreeView+ depends on / blocked
 
Reported: 2013-02-07 11:34 UTC by Jakub Cechacek
Modified: 2015-02-01 22:59 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-09-16 20:28:45 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker WFLY-256 0 Minor Resolved Remove direct connections parameter 2015-11-04 12:47:31 UTC

Description Jakub Cechacek 2013-02-07 11:34:08 UTC
Currently there is no support for creating Cluster connections and defining static connections (only discovery group) in admin console. However "Direct connections" attribute can be set only for clusters with defined static connections, thus it cannot be set for clusters with defined Discovery group. With no support for static connections I think the option for setting this attribute should be removed.

Comment 1 baranowb 2013-04-02 13:12:06 UTC
I claim thee.

Comment 2 Jakub Cechacek 2013-04-10 15:32:33 UTC
After discussion with bbaranow I have realized that it is not a good idea to actually remove this property because there potentially could be a connection which was previously properly created or configured (including needed static connectors) via Cli. However part of this issue is "Unknown error" which is being thrown whenever you try to enable this property on connection without static connector. I believe that some more specific error, which will describe what actually happened, should be raised instead.  

I have also rephrased the title to reflect the issue more precisely

Comment 3 baranowb 2013-04-17 10:28:03 UTC
This is wicked, console output:
Message
	
	
 
	
 
	
Cancel
 Wed Apr 17 10:37:53 CEST 2013
Unknown error

Unexpected HTTP response: 500

Request
{
    "operation" => "composite",
    "address" => [],
    "steps" => [{
        "address" => [
            ("subsystem" => "messaging"),
            ("hornetq-server" => "default"),
            ("cluster-connection" => "CC")
        ],
        "operation" => "write-attribute",
        "name" => "allow-direct-connections-only",
        "value" => true,
        "child-type" => undefined
    }]
}



What AS shows in debug:
DEBUG [org.jboss.as.controller.management-operation] (XNIO-1 task-5) JBAS014616: Operation ("write-attribute") failed - address: ([
    ("subsystem" => "messaging"),
    ("hornetq-server" => "default"),
    ("cluster-connection" => "CC")
]) - failure description: "JBAS011630: Alternative attribute of (allow-direct-connections-only) is already defined."

Comment 4 baranowb 2013-04-18 09:14:16 UTC
There is no way to determine if static connectors have been defined, hence it makes little sense to have this property. Its like having a button in house, which will turn wipers in car you might not even have.

Ive talked with <hpehl>. There will be separate issue, which will introduce static-connectors.

Comment 6 JBoss JIRA Server 2013-04-23 03:02:06 UTC
Brian Stansberry <brian.stansberry> made a comment on jira AS7-6939

This is merged in HAL; AS7 issue can be resolved when a version of the console with the fix in it is merged.

Comment 7 Jakub Cechacek 2013-07-03 11:00:31 UTC
This seems to be OK. Status will be appropriately changed once verified by console test-suite (run currently in progress).

Comment 8 Jakub Cechacek 2013-07-04 10:54:07 UTC
Verified 6.1.1.ER2

Comment 9 Scott Mumford 2013-08-12 01:18:19 UTC
Sounds like this issue was transitioned to another ticket. Switching 'requires doc text' flag to '-' to remove from Release Notes list.

Please comment and reflag if I've misunderstood.


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