Bug 1014103

Summary: Issue warning when user sets non-existent DLQ/expiry address to address settings in CLI
Product: [JBoss] JBoss Enterprise Application Platform 6 Reporter: Martin Svehla <msvehla>
Component: HornetQAssignee: Clebert Suconic <csuconic>
Status: CLOSED EOL QA Contact: Miroslav Novak <mnovak>
Severity: low Docs Contact:
Priority: unspecified    
Version: 6.2.0CC: jmesnil, mnovak, msvehla
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-08-19 12:44:21 UTC Type: Feature Request
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On: 1014095    
Bug Blocks:    

Description Martin Svehla 2013-10-01 12:25:29 UTC
Description of problem:
Issue warning to user when she tries to set address-settings dead letter address and/or expiry address to non-existent destination in CLI client.


Version-Release number of selected component (if applicable):
EAP 6.2.0.ER3, HornetQ 2.3.8.Final

Comment 1 Martin Svehla 2013-10-01 12:27:20 UTC
See #1014095 for related discussion.

Comment 2 Martin Svehla 2013-10-01 12:30:04 UTC
Proper link: bug 1014095

Comment 3 Jeff Mesnil 2014-01-29 15:09:49 UTC
I am not sure how we can fix this in the messaging component. HornetQ does not provide a way to tell if an address exists or not in their management API.

Once they add this, we can provide some checks when the user sets them in the address-settings.

Comment 4 Clebert Suconic 2014-01-29 22:56:35 UTC
I'm working on the implementation and there's a side effect on this.


the DLQ and Expiry are deployed through address settings which are using matching algorithms.


I'm implementing a listener on the address settings to verify it any time the settings have changed.

If you have a wrong address, it will throw the log.warn every time the settings are changed. Say if you deploy a new element all the queues will be validated again.


So, if you deploy a wrong address... say.. IDONTExist.. and later you deploy AD1.. AD2... AD3.. say 5 minutes apart... IDONTEXIST will be logged 3 times.


Anyone sees a problem on this approach?

Comment 5 Clebert Suconic 2014-01-29 23:02:01 UTC
Never mind, I will add a check and only throw the exception once for a given address

Comment 6 Clebert Suconic 2014-01-29 23:02:13 UTC
I meant the log.warn

Comment 7 Martin Svehla 2014-01-30 07:39:36 UTC
Does that mean that if the user changes address-settings in WildFly's CLI client, she will only see the warn message in server log or are we gonna be able to get the warning to CLI client as well?

While I would prefer warning in CLI client too, if that's too much hassle I'm ok with it just being in the server log. In that case I would close this bug, since there won't be any work necessary on integration side, and we'll move this under bug 1014095

Comment 8 Clebert Suconic 2014-01-30 14:32:25 UTC
I'm doing it at the server's log...

I'm adding a method that Jeff can use to make it on the CLI.

Comment 9 Martin Svehla 2014-01-30 15:03:38 UTC
@Clebert, awesome, thanks