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-751 OpenFlowPlugin <p>When trying to remove a group it uses the group type that the group currently have.<br/> If the group was of type "indirect" it will create an error and will not remove the group (OVS - will give back a GROUPMODFAILED error because of INVALIDGROUP.</p> <p>According to OpenFlow 1.3 spec when removing a group it should not use any type.</p> <p>Operating System: All<br/> Platform: All</p> OPNFLWPLUG-751

Removal of group table with type indirect

Bug Resolved Won't Do Unassigned Yair Zinger Mon, 22 Aug 2016 09:29:49 +0000 Mon, 27 Sep 2021 09:01:53 +0000 Tue, 12 Dec 2017 00:34:05 +0000 General 0 4 <p>Which version of OVS are you testing against ?</p> <p>(In reply to Shuva Jyoti Kar from comment #1)<br/> &gt; Which version of OVS are you testing against ?</p> <p>2.5.1</p> <p>(In reply to Yair Zinger from comment #2)<br/> &gt; (In reply to Shuva Jyoti Kar from comment #1)<br/> &gt; &gt; Which version of OVS are you testing against ?<br/> &gt; <br/> &gt; 2.5.1</p> <p>could you please attach the Wireshark capture of the same to the bug</p> <p>See packets number 216 and 218</p> <p>Attachment indirect group removal error.pcap has been added with description: indirect group removal error pcap</p> <p>(In reply to Shuva Jyoti Kar from comment #3)<br/> &gt; (In reply to Yair Zinger from comment #2)<br/> &gt; &gt; (In reply to Shuva Jyoti Kar from comment #1)<br/> &gt; &gt; &gt; Which version of OVS are you testing against ?<br/> &gt; &gt; <br/> &gt; &gt; 2.5.1<br/> &gt; <br/> &gt; could you please attach the Wireshark capture of the same to the bug</p> <p>Yes, I've attached it.</p> <p>Perhaps this could help</p> <p><a href="http://openvswitch.org/pipermail/dev/2016-March/067852.html" class="external-link" target="_blank" rel="nofollow noopener">http://openvswitch.org/pipermail/dev/2016-March/067852.html</a></p> <p>I remember that the patch was merged sometime in April , but couldnot find it out.</p> <p>(In reply to Shuva Jyoti Kar from comment #6)<br/> &gt; Perhaps this could help<br/> &gt; <br/> &gt; <a href="http://openvswitch.org/pipermail/dev/2016-March/067852.html" class="external-link" target="_blank" rel="nofollow noopener">http://openvswitch.org/pipermail/dev/2016-March/067852.html</a><br/> &gt; <br/> &gt; I remember that the patch was merged sometime in April , but couldnot find<br/> &gt; it out.</p> <p>It's good that this was reported.<br/> But it we still have a problem in the 2.5 official release and 2.4 OVS versions and the spec says to ignore the type, maybe we should always use "all" type for groups removal?</p> <p>FYI we decided to workaround <a href="https://jira.opendaylight.org/browse/NETVIRT-91" title="External network group tables removal is not working - indirect type" class="issue-link" data-issue-key="NETVIRT-91"><del>NETVIRT-91</del></a> by using "all" group with a single bucket instead of "indirect" in netvirt, so no need for this in Boron from our side.</p> <p>We should probably do something with this in openflowplugin for Carbon (use all when deleting indirect? verify fix of delete with indirect in OVS2.6?)</p> <p><br/> Is this a bug in ovs?</p> <p>(In reply to Andrej Leitner from comment #9)<br/> &gt; Is this a bug in ovs?</p> <p>According to the openflow spec the switch should ignore it.<br/> However, this is not fixed in OVS 2.4 and 2.5 official releases.</p> <p>I think it will be best if we support the use of OVS 2.4 and 2.5.</p> <p>In addition, there's a bug in ODL: <br/> When trying to remove a group and specifically use a type for the removal it ignore it and use the type that was used for the group installation.</p> <p>We should definitely fix ODL/OFP bugs, but in general, I don't think it is good idea to adjust our code to work with different bugs in other related systems.</p> <p>(In reply to Andrej Leitner from comment #9)<br/> &gt; Is this a bug in ovs?</p> <p>Yes that's fixed as a part of <br/> <a href="https://github.com/openvswitch/ovs/commit/beaa2b8702a20fa1f55e3b4c072dfe6fa394d6d1" class="external-link" target="_blank" rel="nofollow noopener">https://github.com/openvswitch/ovs/commit/beaa2b8702a20fa1f55e3b4c072dfe6fa394d6d1</a></p> <p>(In reply to Yair Zinger from comment #10)<br/> &gt; (In reply to Andrej Leitner from comment #9)<br/> &gt; &gt; Is this a bug in ovs?<br/> &gt; <br/> &gt; According to the openflow spec the switch should ignore it.<br/> &gt; However, this is not fixed in OVS 2.4 and 2.5 official releases.<br/> &gt; <br/> &gt; I think it will be best if we support the use of OVS 2.4 and 2.5.<br/> &gt; <br/> &gt; In addition, there's a bug in ODL: <br/> &gt; When trying to remove a group and specifically use a type for the removal it<br/> &gt; ignore it and use the type that was used for the group installation.</p> <p>Could you please rephrase the question?</p> <p>It was reaction to:</p> <p>(In reply to Yair Zinger from comment #7)<br/> &gt; It's good that this was reported.<br/> &gt; But it we still have a problem in the 2.5 official release and 2.4 OVS<br/> &gt; versions and the spec says to ignore the type, maybe we should always use<br/> &gt; "all" type for groups removal?</p> <p>No activity on this bug from last 1 year, closing the bug. Please re-open the bug if you see this issue with carbon/nitrogen/oxygen branch.</p> Blocks NETVIRT-91 Development External issue ID 6492 External issue URL Rank 0|i03333: