OpenDaylight JIRA

OpenDaylight JIRA https://jira.opendaylight.org This file is an XML representation of an issue en-us

8.20.10

820010

22-06-2022

[OPNFLWPLUG-716] Miising flows in operational DS (and on switches as well) https://jira.opendaylight.org/browse/OPNFLWPLUG-716 OpenFlowPlugin <p>Certain flows fail to be configured on OVS.</p> <p>Please see the attachment.</p> <p>Operating System: All<br/> Platform: All</p> OPNFLWPLUG-716

Miising flows in operational DS (and on switches as well)

Bug Resolved Done Unassigned Tomas Cechvala Fri, 24 Jun 2016 09:53:43 +0000 Mon, 27 Sep 2021 09:01:50 +0000 Wed, 20 Jul 2016 07:33:19 +0000 General 0 2 <p>These flows are configured successfully, but fail to be written on OVS.</p> <p>Attachment flows_config_ds_table6.txt has been added with description: Flows configured in table6</p> <p>To filter out redundand logging</p> <p>log:set ERROR org.opendaylight.groupbasedpolicy</p> <p>Attachment karaflog.txt has been added with description: karaf.log</p> <p>Missing flows can be seen directly in CSIT job:</p> <p>ODL data (including inventory/nodes config/operational) and flows on OVS switches can be seen directly in groupbasedpolicy CSIT job:<br/> <a href="https://jenkins.opendaylight.org/releng/view/groupbasedpolicy/job/groupbasedpolicy-csit-1node-3-node-only-boron/694/robot/report/log.html#s1-s2-s6" class="external-link" target="_blank" rel="nofollow noopener">https://jenkins.opendaylight.org/releng/view/groupbasedpolicy/job/groupbasedpolicy-csit-1node-3-node-only-boron/694/robot/report/log.html#s1-s2-s6</a></p> <p>Please notice that flows for table6 are missing.<br/> <a href="https://jenkins.opendaylight.org/releng/view/groupbasedpolicy/job/groupbasedpolicy-csit-1node-3-node-only-boron/694/robot/report/log.html#s1-s2-s6-t1-k3-k2" class="external-link" target="_blank" rel="nofollow noopener">https://jenkins.opendaylight.org/releng/view/groupbasedpolicy/job/groupbasedpolicy-csit-1node-3-node-only-boron/694/robot/report/log.html#s1-s2-s6-t1-k3-k2</a></p> <p>Hi Tomas,</p> <p>I guess you are seeing the flows in the inventory config DS , but not in the switch . Am i correct ?</p> <p>if yes:</p> <p>given this in the log :</p> <p>2016-06-24 12:01:56,979 | ERROR | on-dispatcher-77 | DataTreeChangeListenerActor | 185 - org.opendaylight.controller.sal-distributed-datastore - 1.4.0.SNAPSHOT | Error notifying listener org.opendaylight.controller.md.sal.binding.impl.BindingClusteredDOMDataTreeChangeListenerAdapter@38b0d026<br/> java.lang.NullPointerException: Match value must not be null<br/> at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:226)<span class="error">[65:com.google.guava:18.0.0]</span><br/> at org.opendaylight.openflowplugin.impl.registry.flow.FlowRegistryKeyFactory$FlowRegistryKeyDto.<init>(FlowRegistryKeyFactory.java:43)<span class="error">[205:org.opendaylight.openflowplugin.impl:0.3.0.SNAPSHOT]</span><br/> at org.opendaylight.openflowplugin.impl.registry.flow.FlowRegistryKeyFactory.create(FlowRegistryKeyFactory.java:29)<span class="error">[205:org.opendaylight.openflowplugin.impl:0.3.0.SNAPSHOT]</span><br/> at org.opendaylight.openflowplugin.impl.services.SalFlowServiceImpl.addFlow(SalFlowServiceImpl.java:86)<span class="error">[205:org.opendaylight.openflowplugin.impl:0.3.0.SNAPSHOT]</span></p> <p>It looks like having a non-null match s what is mandated by the inventory.</p> <p>Can you please try with a non-null match and let us know if that works ?</p> <p>Hi Shuva, so i tried to add empty match instead of null one and it helped.</p> <p>Thanks for pointing that out.</p> <p>Fixed in GBP<br/> <a href="https://git.opendaylight.org/gerrit/#/c/40812/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/40812/</a></p> <p>cool ....</p> <p>Reviewing the openflow specifications and post discussions, i feel having no match in the request is entirely fine, because data store doesn't put any restriction, and hence we should handle it internally.</p> <p>The gerrit patch for the same is<br/> <a href="https://git.opendaylight.org/gerrit/#/c/41605/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/41605/</a> .</p> <p>closing bug since the patch was merged already in boron and beryllium</p> Development External issue ID 6109 External issue URL Priority Rank 0|i032vb: