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/OPNFLWPLUG-580 OpenFlowPlugin <p>The current code meant to avoid NPE for the getGeneratedUUIDForNode() call leads to IllegalArgumentException because the UUID class is unhappy with string "invalid-uuid". So, might as well return null instead of doing bad workaround.</p> <p>Following exception was thrown on my screen.</p> <p>Exception in thread "pool-30-thread-1" java.lang.RuntimeException: java.lang.IllegalArgumentException: Invalid UUID string: invalid-uuid<br/> at com.lmax.disruptor.FatalExceptionHandler.handleEventException(FatalExceptionHandler.java:45)<br/> at com.lmax.disruptor.BatchEventProcessor.run(BatchEventProcessor.java:147)<br/> at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)<br/> at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)<br/> at java.lang.Thread.run(Thread.java:745)<br/> Caused by: java.lang.IllegalArgumentException: Invalid UUID string: invalid-uuid<br/> at java.util.UUID.fromString(UUID.java:194)<br/> at org.opendaylight.openflowplugin.applications.statistics.manager.impl.StatisticsManagerImpl.getGeneratedUUIDForNode(StatisticsManagerImpl.java:399)<br/> at org.opendaylight.openflowplugin.applications.statistics.manager.impl.StatListenCommitGroup$2.generatedUUIDForNode(StatListenCommitGroup.java:206)<br/> at org.opendaylight.openflowplugin.applications.statistics.manager.StatisticsManager$StatDataStoreOperation.&lt;init&gt;(StatisticsManager.java:85)<br/> at org.opendaylight.openflowplugin.applications.statistics.manager.impl.StatListenCommitGroup$2.&lt;init&gt;(StatListenCommitGroup.java:167)<br/> at org.opendaylight.openflowplugin.applications.statistics.manager.impl.StatListenCommitGroup.onGroupFeaturesUpdated(StatListenCommitGroup.java:167)<br/> at org.opendaylight.yangtools.yang.binding.util.NotificationListenerInvoker.invokeNotification(NotificationListenerInvoker.java:91)<br/> at org.opendaylight.controller.md.sal.binding.impl.BindingDOMNotificationListenerAdapter.onNotification(BindingDOMNotificationListenerAdapter.java:44)<br/> at org.opendaylight.controller.md.sal.dom.broker.impl.DOMNotificationRouterEvent.deliverNotification(DOMNotificationRouterEvent.java:50)<br/> at org.opendaylight.controller.md.sal.dom.broker.impl.DOMNotificationRouter$1.onEvent(DOMNotificationRouter.java:68)<br/> at org.opendaylight.controller.md.sal.dom.broker.impl.DOMNotificationRouter$1.onEvent(DOMNotificationRouter.java:65)<br/> at com.lmax.disruptor.BatchEventProcessor.run(BatchEventProcessor.java:128)<br/> ... 3 more<br/> Exception in thread "Thread-1216" java.util.concurrent.RejectedExecutionException: Task org.opendaylight.openflowplugin.openflow.md.core.HandshakeStepWrapper@10e0fdcd rejected from org.opendaylight.openflowplugin.openflow.md.core.ThreadPoolLoggingExecutor@14df6b09<span class="error">&#91;Shutting down, pool size = 0, active threads = 0, queued tasks = 1, completed tasks = 0&#93;</span><br/> at java.util.concurrent.ThreadPoolExecutor$AbortPolicy.rejectedExecution(ThreadPoolExecutor.java:2047)<br/> at java.util.concurrent.ThreadPoolExecutor.reject(ThreadPoolExecutor.java:823)<br/> at java.util.concurrent.ThreadPoolExecutor.execute(ThreadPoolExecutor.java:1369)<br/> at org.opendaylight.openflowplugin.openflow.md.core.ConnectionConductorImpl.onConnectionReady(ConnectionConductorImpl.java:456)<br/> at org.opendaylight.openflowjava.protocol.impl.core.connection.ConnectionAdapterImpl$3.run(ConnectionAdapterImpl.java:472)<br/> at java.lang.Thread.run(Thread.java:745)</p> <p>Operating System: All<br/> Platform: All</p> OPNFLWPLUG-580

IllegalArgumentException in statistics manager getGeneratedUUIDForNode() call

Bug Resolved Won't Do Shivani Pathak Srini Seetharaman Tue, 8 Dec 2015 12:13:25 +0000 Mon, 27 Sep 2021 09:01:41 +0000 Mon, 19 Jun 2017 12:48:44 +0000 General 0 1 <p>String passed in the fromString() method of java.util.UUID should be in a particular format to avoid the exception.</p> <p>ex. Hexadecimal representation of the plain string should be formatted into a UUID formatted string (standardized to 8-4-4-4-12 hex digits) and then passed to fromString() method.</p> <p>Instead of invalid-uuid, UUID formatted string should be passed.</p> <p>When the UUID is indeed unavailable, there is no way to use a valid UUID. So, might as well return null.</p> <p>Here is a simple patch: <a href="https://git.opendaylight.org/gerrit/#/c/30975/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/30975/</a></p> <p>(In reply to Srini Seetharaman from comment #2)<br/> &gt; When the UUID is indeed unavailable, there is no way to use a valid UUID.<br/> &gt; So, might as well return null.<br/> &gt; <br/> &gt; Here is a simple patch: <a href="https://git.opendaylight.org/gerrit/#/c/30975/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/30975/</a></p> <p>UUID (java.util.UUID) is a class name so how can it be unavailable ?</p> <p>Submitted a patch <a href="https://git.opendaylight.org/gerrit/32333" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/32333</a></p> <p>He-desing won't be updated anymore. Closing</p> Development External issue ID 4738 External issue URL Rank 0|i03213: