https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/BGPCEP-680 bgpcep <p>2017-08-04 17:06:30,736 | ERROR | ult-dispatcher-3 | ClusterSingletonServiceGroupImpl | 183 - org.opendaylight.mdsal.singleton-dom-impl - 2.3.0.Nitrogen | Unexpected exception state for service Provider example-bgp-rib-service-group in TakeLeadership<br/> java.lang.NullPointerException<br/> at org.opendaylight.protocol.bgp.rib.impl.RIBImpl.startLocRib(RIBImpl.java:181)<span class="error">&#91;225:org.opendaylight.bgpcep.bgp-rib-impl:0.8.0.Nitrogen&#93;</span><br/> at java.lang.Iterable.forEach(Iterable.java:75)<span class="error">&#91;:1.8.0_141&#93;</span><br/> at org.opendaylight.protocol.bgp.rib.impl.RIBImpl.instantiateServiceInstance(RIBImpl.java:353)<span class="error">&#91;225:org.opendaylight.bgpcep.bgp-rib-impl:0.8.0.Nitrogen&#93;</span><br/> at org.opendaylight.mdsal.singleton.dom.impl.ClusterSingletonServiceRegistrationDelegator.instantiateServiceInstance(ClusterSingletonServiceRegistrationDelegator.java:47)<span class="error">&#91;183:org.opendaylight.mdsal.singleton-dom-impl:2.3.0.Nitrogen&#93;</span><br/> at org.opendaylight.mdsal.singleton.dom.impl.ClusterSingletonServiceGroupImpl.takeOwnership(ClusterSingletonServiceGroupImpl.java:305)<span class="error">&#91;183:org.opendaylight.mdsal.singleton-dom-impl:2.3.0.Nitrogen&#93;</span><br/> at org.opendaylight.mdsal.singleton.dom.impl.ClusterSingletonServiceGroupImpl.ownershipChanged(ClusterSingletonServiceGroupImpl.java:249)<span class="error">&#91;183:org.opendaylight.mdsal.singleton-dom-impl:2.3.0.Nitrogen&#93;</span><br/> at org.opendaylight.mdsal.singleton.dom.impl.AbstractClusterSingletonServiceProviderImpl.ownershipChanged(AbstractClusterSingletonServiceProviderImpl.java:146)<span class="error">&#91;183:org.opendaylight.mdsal.singleton-dom-impl:2.3.0.Nitrogen&#93;</span><br/> at org.opendaylight.mdsal.singleton.dom.impl.DOMClusterSingletonServiceProviderImpl.ownershipChanged(DOMClusterSingletonServiceProviderImpl.java:23)<span class="error">&#91;183:org.opendaylight.mdsal.singleton-dom-impl:2.3.0.Nitrogen&#93;</span><br/> at org.opendaylight.controller.cluster.datastore.entityownership.EntityOwnershipListenerActor.onEntityOwnershipChanged(EntityOwnershipListenerActor.java:44)<span class="error">&#91;133:org.opendaylight.controller.sal-distributed-datastore:1.6.0.Nitrogen&#93;</span><br/> at org.opendaylight.controller.cluster.datastore.entityownership.EntityOwnershipListenerActor.handleReceive(EntityOwnershipListenerActor.java:33)<span class="error">&#91;133:org.opendaylight.controller.sal-distributed-datastore:1.6.0.Nitrogen&#93;</span><br/> at org.opendaylight.controller.cluster.common.actor.AbstractUntypedActor.onReceive(AbstractUntypedActor.java:28)<span class="error">&#91;126:org.opendaylight.controller.sal-clustering-commons:1.6.0.Nitrogen&#93;</span><br/> at akka.actor.UntypedActor$$anonfun$receive$1.applyOrElse(UntypedActor.scala:165)<span class="error">&#91;58:com.typesafe.akka.actor:2.4.18&#93;</span><br/> at akka.actor.Actor$class.aroundReceive(Actor.scala:502)<span class="error">&#91;58:com.typesafe.akka.actor:2.4.18&#93;</span><br/> at akka.actor.UntypedActor.aroundReceive(UntypedActor.scala:95)<span class="error">&#91;58:com.typesafe.akka.actor:2.4.18&#93;</span><br/> at akka.actor.ActorCell.receiveMessage(ActorCell.scala:526)<span class="error">&#91;58:com.typesafe.akka.actor:2.4.18&#93;</span><br/> at akka.actor.ActorCell.invoke(ActorCell.scala:495)<span class="error">&#91;58:com.typesafe.akka.actor:2.4.18&#93;</span><br/> at akka.dispatch.Mailbox.processMailbox(Mailbox.scala:257)<span class="error">&#91;58:com.typesafe.akka.actor:2.4.18&#93;</span><br/> at akka.dispatch.Mailbox.run(Mailbox.scala:224)<span class="error">&#91;58:com.typesafe.akka.actor:2.4.18&#93;</span><br/> at akka.dispatch.Mailbox.exec(Mailbox.scala:234)<span class="error">&#91;58:com.typesafe.akka.actor:2.4.18&#93;</span><br/> at scala.concurrent.forkjoin.ForkJoinTask.doExec(ForkJoinTask.java:260)<span class="error">&#91;204:org.scala-lang.scala-library:2.11.11.v20170413-090219-8a413ba7cc&#93;</span><br/> at scala.concurrent.forkjoin.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1339)<span class="error">&#91;204:org.scala-lang.scala-library:2.11.11.v20170413-090219-8a413ba7cc&#93;</span><br/> at scala.concurrent.forkjoin.ForkJoinPool.runWorker(ForkJoinPool.java:1979)<span class="error">&#91;204:org.scala-lang.scala-library:2.11.11.v20170413-090219-8a413ba7cc&#93;</span><br/> at scala.concurrent.forkjoin.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:107)<span class="error">&#91;204:org.scala-lang.scala-library:2.11.11.v20170413-090219-8a413ba7cc&#93;</span></p> <p>Reproduced using:</p> <p>feature:install odl-bgpcep-bgp-rib-api<br/> (wait to complete)<br/> feature:install odl-bgpcep-bgp-rib-impl</p> <p>Operating System: All<br/> Platform: All</p> BGPCEP-680

NPE during singleton startup

Bug Resolved Done Yrineu Felipe Rodrigues Robert Varga Fri, 4 Aug 2017 15:09:01 +0000 Sun, 3 Mar 2019 11:50:04 +0000 Wed, 11 Oct 2017 00:17:55 +0000 Bugzilla Migration Bugzilla Migration BGP 0 2 <p>Attachment karaf.log has been added with description: karaf.log</p> <p>This is something in:</p> <p> final ChoiceNode routes = this.ribContextRegistry.getRIBSupportContext(key).getRibSupport().emptyRoutes();</p> <p>Hi Robert Varga,</p> <p>I believe that the root cause of that issue is concurrency. I did checked the parameters and there's no 'Null' parameters. To solve that problem maybe should be interesting to evaluate a little refactoring in order to remove some dependencies from the constructor and avoid some static methods. I'm trying to provide it, but if do you have some other tip, please let me know.</p> <p>regards,</p> <p>Hi Robert Varga,</p> <p>This problem is happening because the 'RIBSupportContextRegistryImpl ribContextRegistry' doesn't have 'TablesKey' registered for IPv4 and IPv6 Unicast.</p> <p>org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.types.rev130919.Ipv6AddressFamily, _safi=class org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.types.rev130919.UnicastSubsequentAddressFamily</p> <p>org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.types.rev130919.Ipv4AddressFamily, _safi=class org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.bgp.types.rev130919.UnicastSubsequentAddressFamily</p> <p>Please, let me know if do you have some tips to solve that.</p> <p>I believe that some sub-module have to register those keys at the startup.</p> <p>regards,</p> <p>It follows a patch that solve this issue.</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/62870/2" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/62870/2</a></p> <p>This issue just happens when try to install those features individually instead install 'odl-bgpcep-bgp'.</p> <p>That patch avoid a NPE once IPv4Address and IPv6Address Unicast AFI aren't registered at RIB context.</p> <p>Adding conversation under proposed fix to bug</p> <p>Claudio D. Gasparini<br/> Sep 8 9:44 PM<br/> ↩<br/> Patch Set 2:<br/> But it'll happens just if we install only the 'odl-bgpcep-rib-api'<br/> and odl-bgpcep-rib-impl'. So, the ways to solve that are:<br/> -Install 'odl-bgpcep-bgp' or<br/> -Include 'inet-module' as a dependency of 'odl-bgpcep-rib-impl'What is your suggestion on that?<br/> did you check if odl-bgpcep-rib-impl contains inet?<br/> <a href="https://git.opendaylight.org/gerrit/gitweb?p=bgpcep.git;a=blob;f=features/bgp/odl-bgpcep-bgp-rib-impl/pom.xml;h=24def18d442697188f4467b04ba5af03a8b8ca75;hb=refs/heads/master" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/gitweb?p=bgpcep.git;a=blob;f=features/bgp/odl-bgpcep-bgp-rib-impl/pom.xml;h=24def18d442697188f4467b04ba5af03a8b8ca75;hb=refs/heads/master</a><br/> Yrineu Felipe Rodrigues<br/> Sep 8 9:56 PM<br/> ↩<br/> Patch Set 2:<br/> Yep, it's there, but there's no specification of module order. So, if do you manually install 'bgp-inet' before 'rib-api' and 'rib-impl', there's no NPE.<br/> Avoid install 'rib-api' and 'rib-impl' independently is a valid option? I believe that we have to include both features into 'inet'.</p> <p>Hi, some notes</p> <p>Actual</p> <p>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</p> <p>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<br/> 2017-09-09 00:11:49,149 | INFO | pool-2-thread-1 | RIBActivator | 222 - org.opendaylight.bgpcep.bgp-inet - 0.9.0.SNAPSHOT | Starting Inet RIB Activator</p> <p>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.</p> <p>steps </p> <p>feature:install odl-bgpcep-bgp-rib-api<br/> (wait to complete)<br/> feature:install odl-bgpcep-bgp-rib-impl</p> <p>=====================================================</p> <p>Expected</p> <p>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<br/> 2017-09-09 00:30:55,230 | INFO | pool-2-thread-1 | RIBActivator | 98 - org.opendaylight.bgpcep.bgp-inet - 0.9.0.SNAPSHOT | Starting Inet RIB Activator</p> <p>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</p> <p>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.</p> <p>feature:install odl-bgpcep-bgp-inet<br/> (wait to complete)<br/> feature:install odl-bgpcep-bgp-rib-api<br/> (wait to complete)<br/> feature:install odl-bgpcep-bgp-rib-impl</p> <p>Conclusion:</p> <p>Issue here is that SimpleRIBExtensionProviderContext under rib-spi is created and expose RIBExtensionProviderContext service before that any extension is loaded.</p> <p>Fix should imply that all(not only inet) extension must be loaded before service is exposed. Otherwise any race condition combined with BGP Deployer,<br/> could end in a RIB up which won't have support for a extension that was coming up at the same time(and this can happen without throwing any exception, therefore it will be only caught if the user is really checking it).</p> <p>So we need that this steps are done in order</p> <p>1 - Load all extensions<br/> 2 - Expose RIBExtensionProviderContext Service<br/> 3 - Start BGP Deployer</p> <p>As you can see the fix involves more than fix the NPE, and needs more thought. I hope this can help you.</p> <p>Regards, </p> <p>Claudio</p> <p>‎<span class="error">&#91;‎25/09/2017 18:58:38‏&#93;</span> ‎&lt;‎yrineu_rodrigues‎&gt;‎ let me ask you... there is some way to choose the 'load order' of each module?<br/> ‎<span class="error">&#91;‎25/09/2017 19:33:06‏&#93;</span> ‎&lt;‎rovarga‎&gt;‎ hello Yrineu<br/> ‎<span class="error">&#91;‎25/09/2017 19:33:22‏&#93;</span> ‎&lt;‎rovarga‎&gt;‎ not sure about blueprint<br/> ‎<span class="error">&#91;‎25/09/2017 19:34:04‏&#93;</span> ‎&lt;‎rovarga‎&gt;‎ the problem with karaf/osgi is that we do not really know when all plugins have been registered<br/> ‎<span class="error">&#91;‎25/09/2017 19:34:51‏&#93;</span> ‎&lt;‎rovarga‎&gt;‎ I'll follow up in the issue</p> <p>So the issue with registry-type things is that OSGi and karaf's feature:install approach does not really tell us when loading is complete (there is an API in 4.1.x which will help a bit), but even that is only valid until the user types feature:install.</p> <p>At any case, we should be able to start loc-rib without any extensions &#8211; so that part is a bug.</p> <p>Going forward, I am not sure it was implemented, but the idea was for the registry to signal when a new extension is loaded and the BGP sessions would be flapped (which is not nice, but gets the job done).</p> <p>Another option is to have rib-impl check its configuration and figure out the afis/safis required for startup &#8211; before starting netty.</p> <p>@Claudio and @Robert, makes sense to create a new karaf module in order to install in a single place all contents of:<br/> feature:install odl-bgpcep-bgp-inet<br/> feature:install odl-bgpcep-bgp-rib-api<br/> feature:install odl-bgpcep-bgp-rib-impl</p> <p>something like: 'odl-bgpcep-bgp-extensions' that will involve all dependencies of those features above. Thus we can ensure that all extensions will be loaded and avoid install features with missing extensions.</p> <p>Please, let me know what do you think about that.</p> <p>I think it would be okay if bgp/rib-impl depended on bgp/inet. While inet is independent as an implementation, it is also part of RFC7460, which is what we claim to support by default.</p> <p>Aside from this bgp/inet should not be special, e.g. remain a plugin outside of bgp/parser-impl.</p> <p>Hi @Robert and @Claudio,</p> <p>It follows a patch with a proposed fix:<br/> <a href="https://git.opendaylight.org/gerrit/#/c/63537/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/63537/</a></p> <p>PS: Still waiting for Jenkins</p> <p>Hi Yrineu, </p> <p>proposed patch for fix the NPE<br/> <a href="https://git.opendaylight.org/gerrit/#/q/topic:BUG-8929" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/q/topic:BUG-8929</a><br/> has been merged.</p> <p>regarding you proposal to force odl-bgpcep-bgp-rib-impl to be part of odl-bgpcep-bgp-inet. Their should keep their dependency, and this is not a fixing any issue.</p> <p>The issue to be fixed is that when a new extension is installed. All started session are not restarting and loading this new supported expansion.<br/> Once fixed inet and the rest of extension will be ready to be used, no matter when or in which order they are installed.<br/> Also this give freedom to the user to use the implementation of the extension he wants.</p> <p>I'll open a bug with defined task and requirements for solve the issue previously described.</p> <p>Regarding this bug, I'll close it as soon as I can check that you patch solved the NPE, if you already tested that </p> <p>feature:install odl-bgpcep-bgp-rib-api<br/> (wait to complete)<br/> feature:install odl-bgpcep-bgp-rib-impl</p> <p>doesn't longer cause the NPE, feel free to close it.</p> <p>Regards,</p> <p>Thanks for your update!</p> Development External issue ID 8929 External issue URL Issue Type Rank 0|i02ds7: