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-679 OpenFlowPlugin <p>When working on the distribution produced after running the following builds in order </p> <p>Distribution repo with <a href="https://git.opendaylight.org/gerrit/#/c/38047/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/38047/</a></p> <p>L2Switch repo with <a href="https://git.opendaylight.org/gerrit/#/c/33303/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/33303/</a></p> <p>Openflowplugin repo with <a href="https://git.opendaylight.org/gerrit/#/c/35892" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/35892</a></p> <p>or </p> <p><a href="https://jenkins.opendaylight.org/releng/view/integration/job/integration-multipatch-test-boron/4/org.opendaylight.integration$distribution-karaf/artifact/org.opendaylight.integration/distribution-karaf/0.5.0-SNAPSHOT/distribution-karaf-0.5.0-SNAPSHOT.zip" class="external-link" target="_blank" rel="nofollow noopener">https://jenkins.opendaylight.org/releng/view/integration/job/integration-multipatch-test-boron/4/org.opendaylight.integration$distribution-karaf/artifact/org.opendaylight.integration/distribution-karaf/0.5.0-SNAPSHOT/distribution-karaf-0.5.0-SNAPSHOT.zip</a></p> <p>Start controller --&gt; Start mininet with topo=2, LLDP flows or/and other flows are not seen in stats of controller (check below links). When flows are checked on the switches they are present though.</p> <p>http://&lt;Controller-Ip&gt;:8181/restconf/operational/opendaylight-inventory:nodes/node/openflow:1/table/0/ </p> <p>http://&lt;Controller-Ip&gt;:8181/restconf/operational/opendaylight-inventory:nodes/node/openflow:2/table/0/ </p> <p>http://&lt;Controller-Ip&gt;:8181/restconf/operational/opendaylight-inventory:nodes/node/openflow:3/table/0/</p> <p>Operating System: All<br/> Platform: All</p> OPNFLWPLUG-679

Li Migration: l2switch - LLDP flows or/and other flows not seen in controller stats

Bug Resolved Done Shuva Jyoti Kar Sai MarapaReddy Fri, 29 Apr 2016 21:09:21 +0000 Mon, 27 Sep 2021 09:01:48 +0000 Tue, 16 Aug 2016 05:35:20 +0000 General 0 5 <p>Made blocker as per: <a href="https://lists.opendaylight.org/pipermail/openflowplugin-dev/2016-May/005036.html" class="external-link" target="_blank" rel="nofollow noopener">https://lists.opendaylight.org/pipermail/openflowplugin-dev/2016-May/005036.html</a></p> <p>Make it a blocker for the release - but not for M3 (for Li migration default)</p> <p>Sai, apart from LLDP flows , what are the other flows that are not seen in operational DS? since if i add a flow through restconf , i am able to see it in operDS</p> <p>LLDP flows consistently not seen, also the default flows installed by controller (not the flows user installs using restconf) are not seen very often.</p> <p>Hi Shuva any update on this. <br/> Can we please make it as a blocker ?</p> <p>Regarding other flows (not LLDP) you can refer to <a href="https://jira.opendaylight.org/browse/OPNFLWPLUG-736" title="Flow doesn&#39;t show up in operational data with correct flow id" class="issue-link" data-issue-key="OPNFLWPLUG-736"><del>OPNFLWPLUG-736</del></a>. But these two are independent bugs, which work in stable/ber.</p> <p>with a lot many fixes that have gone in can you please test it out on the current plugin build(as of today) and let us know</p> <p>Sai,</p> <p>Please refer to this patch</p> <p><a href="https://git.opendaylight.org/gerrit/42851" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/42851</a></p> <p>(In reply to Shuva Jyoti Kar from comment #7)<br/> &gt; Sai,<br/> &gt; <br/> &gt; Please refer to this patch<br/> &gt; <br/> &gt; <a href="https://git.opendaylight.org/gerrit/42851" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/42851</a></p> <p>working on the fix, this is primarily happening since the flow-key is not unique.Still working on the actual/proper fix</p> <p>the fix of appending the priority to the flowid to guarantee its uniqueness is not a good fix.<br/> As we see, the flowid of both the flows are same, hence this issue occurs. The flowRegistryKey generation and comparison happens correctly, Is it possible to handle it at the application level itself?</p> <p>Flows that are not seen in particular are LLDP and flow with action "dropall".</p> <p>Flows on mininet:- <br/> cookie=0x2b0000000000000b, duration=90.844s, table=0, n_packets=91, n_bytes=13134, priority=131 actions=CONTROLLER:65535<br/> cookie=0x2b00000000000003, duration=5987.185s, table=0, n_packets=38, n_bytes=6573, priority=0 actions=drop</p> <p>Shuva has a patch for this - <a href="https://git.opendaylight.org/gerrit/#/c/42851/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/42851/</a>. Needs to be reviewed by Anil &amp; tested for performance by Shuva.</p> <p>So, I was testing it, and when someone will add flow with for example ID 22, it is correctly added to config, operational and, deviceflowregistry and device.</p> <p>Then, when someone add same flow, but just changes the id to for example 23, it will be replaced in config, replaced in deviceflowregistry, but removed from operational and device, because of FRM (becouse first, flow add is called, but FRM will see, that no flow with ID 22 is in config, so afterwards, it deletes that flow. But, because flow is unique by match, priority and table id, it removes that old/new flow with ID 23).</p> <p>TL:DR<br/> Adding 2 exactly same flows with different flow IDs is bad idea, because flow is not unique by flow id, and also, Shuvas patch what creates another alien and uses BiMap will not solve the problem, because it is not problem with DeviceFlowRegistry.</p> <p>(In reply to Tomas Slusny from comment #12)<br/> &gt; So, I was testing it, and when someone will add flow with for example ID 22,<br/> &gt; it is correctly added to config, operational and, deviceflowregistry and<br/> &gt; device.<br/> &gt; <br/> &gt; Then, when someone add same flow, but just changes the id to for example 23,<br/> &gt; it will be replaced in config, replaced in deviceflowregistry, but removed<br/> &gt; from operational and device, because of FRM (becouse first, flow add is<br/> &gt; called, but FRM will see, that no flow with ID 22 is in config, so<br/> &gt; afterwards, it deletes that flow. But, because flow is unique by match,<br/> &gt; priority and table id, it removes that old/new flow with ID 23).<br/> &gt; <br/> &gt; TL:DR<br/> &gt; Adding 2 exactly same flows with different flow IDs is bad idea, because<br/> &gt; flow is not unique by flow id, and also, Shuvas patch what creates another<br/> &gt; alien and uses BiMap will not solve the problem, because it is not problem<br/> &gt; with DeviceFlowRegistry.</p> <p>Tomas, <br/> currently the problem is not with adding two flows with same </p> {match, priority,table-id and cookie} <p> and different flowids, but with adding 2 different flows with the same flow id. This is because the flowregistry stores a unique(k,v) pair but fails to guarantee unique values.<br/> so 2 flows with different match, priority,table-id and cookie} and same flow ids will be re-written in the oper-Ds (since flowKey contains the Flow id). Hence in the oper-Ds it appears as one.<br/> With my patch, it will guarantee an uniqueness of(k,v) pairs. I have tested it with l2switch and i do see 2 distinct flws in the oper-Ds</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/42851/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/42851/</a></p> <p>Sai, </p> <p>Can you please test with the latest patchset of <br/> <a href="https://git.opendaylight.org/gerrit/#/c/42851/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/42851/</a><br/> and let us know if that works for you?</p> <p>Thanks Shuva.</p> <p>I can see flows in operational. </p> <p>pastebin link:- <a href="http://pastebin.com/tD75LBKA" class="external-link" target="_blank" rel="nofollow noopener">http://pastebin.com/tD75LBKA</a></p> <p>One concern:- The drop all flow "action" is not seen in operational, any thoughts ?</p> <p>Hi Sai, <br/> afaik this is wanted behavior for now.</p> <p>Since the flows are seen in the operational and the behaviour is similar to the earlier model of the plugin,marking the bug as reslved .</p> Development External issue ID 5822 External issue URL ODL SR Target Milestone Priority Rank 0|i032n3: