[BGPCEP-693] extension start doesnt force restart of existing sessions Created: 10/Oct/17 Updated: 16/Nov/21 Resolved: 16/Nov/21 |
|
| Status: | Resolved |
| Project: | bgpcep |
| Component/s: | BGP |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Claudio David Gasparini | Assignee: | Unassigned |
| Resolution: | Duplicate | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 9260 |
| Description |
|
Hi, some notes Actual 2017-09-09 00:11:06,177 | INFO | pool-2-thread-1 | xtensionProviderContextActivator | 100 - org.opendaylight.bgpcep.bgp-rib-spi - 0.9.0.SNAPSHOT | Starting 0 RIBExtensionProviderActivator instances 2017-09-09 00:11:49,148 | INFO | pool-2-thread-1 | BGPActivator | 222 - org.opendaylight.bgpcep.bgp-inet - 0.9.0.SNAPSHOT | Starting Inet BGP Activator 2017-09-09 00:11:49,610 | INFO | pool-2-thread-1 | BgpDeployerImpl | 228 - org.opendaylight.bgpcep.bgp-rib-impl - 0.9.0.SNAPSHOT | BGP Deployer global-bgp started. steps feature:install odl-bgpcep-bgp-rib-api ===================================================== Expected 2017-09-09 00:30:55,229 | INFO | pool-2-thread-1 | BGPActivator | 98 - org.opendaylight.bgpcep.bgp-inet - 0.9.0.SNAPSHOT | Starting Inet BGP Activator 2017-09-09 00:30:55,247 | INFO | rint Extender: 2 | xtensionProviderContextActivator | 105 - org.opendaylight.bgpcep.bgp-rib-spi - 0.9.0.SNAPSHOT | Starting 1 RIBExtensionProviderActivator instances 2017-09-09 00:31:38,113 | INFO | pool-2-thread-1 | BgpDeployerImpl | 228 - org.opendaylight.bgpcep.bgp-rib-impl - 0.9.0.SNAPSHOT | BGP Deployer global-bgp started. feature:install odl-bgpcep-bgp-inet Conclusion: Issue here is that SimpleRIBExtensionProviderContext under rib-spi is created and expose RIBExtensionProviderContext service before that any extension is loaded. Fix should imply that registry is restarted each time a new extension starts, forcing all sessions to restart and support this new extensions Read https://bugs.opendaylight.org/show_bug.cgi?id=8929 |
| Comments |
| Comment by Robert Varga [ 16/Nov/21 ] |
|
Addressed as part of BGPCEP-881. |