[CONTROLLER-138] Netconf client stops retrying re-connect after server restarts Created: 26/Jan/14 Updated: 25/Jul/23 Due: 02/May/14 Resolved: 14/May/14 |
|
| Status: | Resolved |
| Project: | controller |
| Component/s: | netconf |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Jan Medved | Assignee: | Maros Marsalek |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: Mac OS |
||
| Issue Links: |
|
||||||||||||||||||||
| External issue ID: | 372 | ||||||||||||||||||||
| Priority: | Normal | ||||||||||||||||||||
| Description |
|
The Netconf client only attempts re-connection to a configured remote server 5x with timeouts of 60s. When the restart of a remote server takes longer than 5 minutes, the netconf client does not reconnect to the server. The netconf client can not be triggered to reconnect manually - it either has to be reconfigured, or the controller must be restarted in order for the client to reconnect. The Netconf client which is configured for a remote server should be attempting to reconnect infinitely, with (for now) a fixed timeout. In a later release, we need to hook this mechanism up to the node discovery. |
| Comments |
| Comment by Robert Varga [ 26/Jan/14 ] |
|
This maps out into requirement for org.opendaylight.controller.netconf.client.NetconfClient() using AbstractDispatcher.createReconnectingClient(). The problem is that that entails a semantic change how Session:Listener is mapped, as having a reconnecting client means that the listener needs to be able to handle mutliple onSessionUp/onSessionDown callbacks. |
| Comment by Robert Varga [ 27/Feb/14 ] |
|
This is being taken care of over at https://git.opendaylight.org/gerrit/#/c/5258/ |
| Comment by Robert Varga [ 27/Feb/14 ] |
|
Also needs BUG-467 so that reconnect strategy can be configured. |
| Comment by Robert Varga [ 08/Apr/14 ] |
|
Waiting for implementatio of BUG-693, which will allow clean implementation of createReconnectingClient(). |
| Comment by Maros Marsalek [ 13/May/14 ] |