<!-- 
RSS generated by JIRA (8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d) at Wed Feb 07 20:33:24 UTC 2024

It is possible to restrict the fields that are returned in this document by specifying the 'field' parameter in your request.
For example, to request only the issue key and summary append 'field=key&field=summary' to the URL of your request.
-->
<rss version="0.92" >
<channel>
    <title>OpenDaylight JIRA</title>
    <link>https://jira.opendaylight.org</link>
    <description>This file is an XML representation of an issue</description>
    <language>en-us</language>    <build-info>
        <version>8.20.10</version>
        <build-number>820010</build-number>
        <build-date>22-06-2022</build-date>
    </build-info>


<item>
            <title>[OPNFLWPLUG-796] Flow matching function (operational flow reconciliation) is not stable</title>
                <link>https://jira.opendaylight.org/browse/OPNFLWPLUG-796</link>
                <project id="10155" key="OPNFLWPLUG">OpenFlowPlugin</project>
                    <description>&lt;p&gt;To reproduce:&lt;/p&gt;

&lt;p&gt;1) start mininet 1 switch: sudo mn --controller=remote,ip=127.0.0.1 --topo tree,1 --switch ovsk,protocols=OpenFlow13&lt;/p&gt;

&lt;p&gt;2) Push this flow:&lt;br/&gt;
PUT restconf/config/opendaylight-inventory:nodes/node/openflow:1/table/2/flow/144&lt;br/&gt;
&amp;lt;flow xmlns=&quot;urn:opendaylight:flow:inventory&quot;&amp;gt;&lt;br/&gt;
    &amp;lt;strict&amp;gt;false&amp;lt;/strict&amp;gt;&lt;br/&gt;
    &amp;lt;flow-name&amp;gt;FooXf21&amp;lt;/flow-name&amp;gt;&lt;br/&gt;
    &amp;lt;id&amp;gt;144&amp;lt;/id&amp;gt;&lt;br/&gt;
    &amp;lt;cookie_mask&amp;gt;255&amp;lt;/cookie_mask&amp;gt;&lt;br/&gt;
    &amp;lt;cookie&amp;gt;21&amp;lt;/cookie&amp;gt;&lt;br/&gt;
    &amp;lt;table_id&amp;gt;2&amp;lt;/table_id&amp;gt;&lt;br/&gt;
    &amp;lt;priority&amp;gt;21&amp;lt;/priority&amp;gt;&lt;br/&gt;
    &amp;lt;installHw&amp;gt;false&amp;lt;/installHw&amp;gt;&lt;br/&gt;
    &amp;lt;instructions&amp;gt;&lt;br/&gt;
        &amp;lt;instruction&amp;gt;&lt;br/&gt;
            &amp;lt;order&amp;gt;0&amp;lt;/order&amp;gt;&lt;br/&gt;
            &amp;lt;apply-actions&amp;gt;&lt;br/&gt;
                &amp;lt;action&amp;gt;&lt;br/&gt;
                    &amp;lt;order&amp;gt;0&amp;lt;/order&amp;gt;&lt;br/&gt;
                    &amp;lt;dec-nw-ttl/&amp;gt;&lt;br/&gt;
                &amp;lt;/action&amp;gt;&lt;br/&gt;
            &amp;lt;/apply-actions&amp;gt;&lt;br/&gt;
        &amp;lt;/instruction&amp;gt;&lt;br/&gt;
    &amp;lt;/instructions&amp;gt;&lt;br/&gt;
    &amp;lt;match&amp;gt;&lt;br/&gt;
        &amp;lt;ethernet-match&amp;gt;&lt;br/&gt;
            &amp;lt;ethernet-type&amp;gt;&lt;br/&gt;
                &amp;lt;type&amp;gt;34525&amp;lt;/type&amp;gt;&lt;br/&gt;
            &amp;lt;/ethernet-type&amp;gt;&lt;br/&gt;
        &amp;lt;/ethernet-match&amp;gt;&lt;br/&gt;
        &amp;lt;ipv6-source&amp;gt;1234:5678:9ABC:DEF0:FDCD:A987:6543:210F/76&amp;lt;/ipv6-source&amp;gt;&lt;br/&gt;
        &amp;lt;ipv6-destination&amp;gt;fe80::2acf:e9ff:fe21:6431/128&amp;lt;/ipv6-destination&amp;gt;&lt;br/&gt;
        &amp;lt;metadata&amp;gt;&lt;br/&gt;
            &amp;lt;metadata&amp;gt;12345&amp;lt;/metadata&amp;gt;&lt;br/&gt;
        &amp;lt;/metadata&amp;gt;&lt;br/&gt;
        &amp;lt;ip-match&amp;gt;&lt;br/&gt;
            &amp;lt;ip-protocol&amp;gt;17&amp;lt;/ip-protocol&amp;gt;&lt;br/&gt;
            &amp;lt;ip-dscp&amp;gt;8&amp;lt;/ip-dscp&amp;gt;&lt;br/&gt;
            &amp;lt;ip-ecn&amp;gt;3&amp;lt;/ip-ecn&amp;gt;&lt;br/&gt;
        &amp;lt;/ip-match&amp;gt;&lt;br/&gt;
        &amp;lt;udp-source-port&amp;gt;25364&amp;lt;/udp-source-port&amp;gt;&lt;br/&gt;
        &amp;lt;udp-destination-port&amp;gt;8080&amp;lt;/udp-destination-port&amp;gt;&lt;br/&gt;
    &amp;lt;/match&amp;gt;&lt;br/&gt;
&amp;lt;/flow&amp;gt;&lt;/p&gt;

&lt;p&gt;3) Check flow is reconciled in operational:&lt;/p&gt;

&lt;p&gt;GET restconf/operational/opendaylight-inventory:nodes/node/openflow:1/table/2&lt;br/&gt;
{&lt;br/&gt;
  &quot;flow-node-inventory:table&quot;: [&lt;br/&gt;
    {&lt;br/&gt;
      &quot;id&quot;: 2,&lt;br/&gt;
      &quot;flow&quot;: [&lt;br/&gt;
        {&lt;br/&gt;
          &quot;id&quot;: &quot;144&quot;,&lt;br/&gt;
          &quot;opendaylight-flow-statistics:flow-statistics&quot;: {&lt;br/&gt;
            &quot;packet-count&quot;: 0,&lt;br/&gt;
            &quot;duration&quot;: &lt;/p&gt;
{
              &quot;nanosecond&quot;: 1000000,
              &quot;second&quot;: 3
            }
&lt;p&gt;,&lt;br/&gt;
            &quot;byte-count&quot;: 0&lt;br/&gt;
          },&lt;br/&gt;
          &quot;priority&quot;: 21,&lt;br/&gt;
          &quot;table_id&quot;: 2,&lt;br/&gt;
          &quot;hard-timeout&quot;: 0,&lt;br/&gt;
          &quot;match&quot;: {&lt;br/&gt;
            &quot;ethernet-match&quot;: {&lt;br/&gt;
              &quot;ethernet-type&quot;: &lt;/p&gt;
{
                &quot;type&quot;: 34525
              }
&lt;p&gt;            },&lt;br/&gt;
            &quot;metadata&quot;: &lt;/p&gt;
{
              &quot;metadata&quot;: 12345
            }
&lt;p&gt;,&lt;br/&gt;
            &quot;ip-match&quot;: &lt;/p&gt;
{
              &quot;ip-dscp&quot;: 8,
              &quot;ip-protocol&quot;: 17,
              &quot;ip-ecn&quot;: 3
            }
&lt;p&gt;,&lt;br/&gt;
            &quot;ipv6-destination&quot;: &quot;fe80::2acf:e9ff:fe21:6431/128&quot;,&lt;br/&gt;
            &quot;ipv6-source&quot;: &quot;1234:5678:9abc:def0:fdc0::/76&quot;,&lt;br/&gt;
            &quot;udp-source-port&quot;: 25364,&lt;br/&gt;
            &quot;udp-destination-port&quot;: 8080&lt;br/&gt;
          },&lt;br/&gt;
          &quot;cookie&quot;: 21,&lt;br/&gt;
          &quot;flags&quot;: &quot;&quot;,&lt;br/&gt;
          &quot;instructions&quot;: {&lt;br/&gt;
            &quot;instruction&quot;: [&lt;br/&gt;
              {&lt;br/&gt;
                &quot;order&quot;: 0,&lt;br/&gt;
                &quot;apply-actions&quot;: {&lt;br/&gt;
                  &quot;action&quot;: [&lt;br/&gt;
                    {&lt;br/&gt;
                      &quot;order&quot;: 0,&lt;br/&gt;
                      &quot;dec-nw-ttl&quot;: {}&lt;br/&gt;
                    }&lt;br/&gt;
                  ]&lt;br/&gt;
                }&lt;br/&gt;
              }&lt;br/&gt;
            ]&lt;br/&gt;
          },&lt;br/&gt;
          &quot;idle-timeout&quot;: 0&lt;br/&gt;
        }&lt;br/&gt;
      ],&lt;br/&gt;
      &quot;opendaylight-flow-table-statistics:flow-table-statistics&quot;: &lt;/p&gt;
{
        &quot;packets-matched&quot;: 0,
        &quot;active-flows&quot;: 0,
        &quot;packets-looked-up&quot;: 0
      }
&lt;p&gt;    }&lt;br/&gt;
  ]&lt;br/&gt;
}&lt;/p&gt;

&lt;p&gt;4) Delete flow:&lt;br/&gt;
DELETE restconf/config/opendaylight-inventory:nodes/node/openflow:1/table/2/flow/144&lt;/p&gt;

&lt;p&gt;5) repeat 2-4 until you see alien ID is step 3) (normally 4-5 times)&lt;br/&gt;
GET restconf/operational/opendaylight-inventory:nodes/node/openflow:1/table/2&lt;br/&gt;
{&lt;br/&gt;
  &quot;flow-node-inventory:table&quot;: [&lt;br/&gt;
    {&lt;br/&gt;
      &quot;id&quot;: 2,&lt;br/&gt;
      &quot;flow&quot;: [&lt;br/&gt;
        {&lt;br/&gt;
          &quot;id&quot;: &quot;#UF$TABLE*2-13&quot;,&lt;br/&gt;
          &quot;opendaylight-flow-statistics:flow-statistics&quot;: {&lt;br/&gt;
            &quot;packet-count&quot;: 0,&lt;br/&gt;
            &quot;duration&quot;: &lt;/p&gt;
{
              &quot;nanosecond&quot;: 609000000,
              &quot;second&quot;: 21
            }
&lt;p&gt;,&lt;br/&gt;
            &quot;byte-count&quot;: 0&lt;br/&gt;
          },&lt;br/&gt;
          &quot;priority&quot;: 21,&lt;br/&gt;
          &quot;table_id&quot;: 2,&lt;br/&gt;
          &quot;hard-timeout&quot;: 0,&lt;br/&gt;
          &quot;match&quot;: {&lt;br/&gt;
            &quot;ethernet-match&quot;: {&lt;br/&gt;
              &quot;ethernet-type&quot;: &lt;/p&gt;
{
                &quot;type&quot;: 34525
              }
&lt;p&gt;            },&lt;br/&gt;
            &quot;metadata&quot;: &lt;/p&gt;
{
              &quot;metadata&quot;: 12345
            }
&lt;p&gt;,&lt;br/&gt;
            &quot;ip-match&quot;: &lt;/p&gt;
{
              &quot;ip-dscp&quot;: 8,
              &quot;ip-protocol&quot;: 17,
              &quot;ip-ecn&quot;: 3
            }
&lt;p&gt;,&lt;br/&gt;
            &quot;ipv6-destination&quot;: &quot;fe80::2acf:e9ff:fe21:6431/128&quot;,&lt;br/&gt;
            &quot;ipv6-source&quot;: &quot;1234:5678:9abc:def0:fdc0::/76&quot;,&lt;br/&gt;
            &quot;udp-source-port&quot;: 25364,&lt;br/&gt;
            &quot;udp-destination-port&quot;: 8080&lt;br/&gt;
          },&lt;br/&gt;
          &quot;cookie&quot;: 21,&lt;br/&gt;
          &quot;flags&quot;: &quot;&quot;,&lt;br/&gt;
          &quot;instructions&quot;: {&lt;br/&gt;
            &quot;instruction&quot;: [&lt;br/&gt;
              {&lt;br/&gt;
                &quot;order&quot;: 0,&lt;br/&gt;
                &quot;apply-actions&quot;: {&lt;br/&gt;
                  &quot;action&quot;: [&lt;br/&gt;
                    {&lt;br/&gt;
                      &quot;order&quot;: 0,&lt;br/&gt;
                      &quot;dec-nw-ttl&quot;: {}&lt;br/&gt;
                    }&lt;br/&gt;
                  ]&lt;br/&gt;
                }&lt;br/&gt;
              }&lt;br/&gt;
            ]&lt;br/&gt;
          },&lt;br/&gt;
          &quot;idle-timeout&quot;: 0&lt;br/&gt;
        }&lt;br/&gt;
      ],&lt;br/&gt;
      &quot;opendaylight-flow-table-statistics:flow-table-statistics&quot;: &lt;/p&gt;
{
        &quot;packets-matched&quot;: 0,
        &quot;active-flows&quot;: 1,
        &quot;packets-looked-up&quot;: 0
      }
&lt;p&gt;    }&lt;br/&gt;
  ]&lt;br/&gt;
}&lt;/p&gt;</description>
                <environment>&lt;p&gt;Operating System: All&lt;br/&gt;
Platform: All&lt;/p&gt;</environment>
        <key id="28064">OPNFLWPLUG-796</key>
            <summary>Flow matching function (operational flow reconciliation) is not stable</summary>
                <type id="10104" iconUrl="https://jira.opendaylight.org/secure/viewavatar?size=xsmall&amp;avatarId=10303&amp;avatarType=issuetype">Bug</type>
                                                <status id="5" iconUrl="https://jira.opendaylight.org/images/icons/statuses/resolved.png" description="A resolution has been taken, and it is awaiting verification by reporter. From here issues are either reopened, or are closed.">Resolved</status>
                    <statusCategory id="3" key="done" colorName="green"/>
                                    <resolution id="10000">Done</resolution>
                                        <assignee username="ecelgp">Luis Gomez</assignee>
                                    <reporter username="ecelgp">Luis Gomez</reporter>
                        <labels>
                    </labels>
                <created>Tue, 11 Oct 2016 03:51:42 +0000</created>
                <updated>Mon, 27 Sep 2021 09:01:56 +0000</updated>
                            <resolved>Thu, 13 Apr 2017 07:30:50 +0000</resolved>
                                                                    <component>General</component>
                        <due></due>
                            <votes>0</votes>
                                    <watches>6</watches>
                                                                                                                <comments>
                            <comment id="58380" author="ecelgp" created="Tue, 11 Oct 2016 04:13:23 +0000"  >&lt;p&gt;BTW this is reprodcible with latest Boron.&lt;/p&gt;</comment>
                            <comment id="58381" author="shuva.jyoti.kar.87@gmail.com" created="Tue, 11 Oct 2016 05:56:13 +0000"  >&lt;p&gt;One question that i have is does the flow with alien id in the Oper-DS remain forever?&lt;/p&gt;</comment>
                            <comment id="58382" author="ecelgp" created="Tue, 11 Oct 2016 14:35:03 +0000"  >&lt;p&gt;Once you get an alien ID further flow delete/push attempts also produce alien ID, if this is your question.&lt;/p&gt;</comment>
                            <comment id="58383" author="ecelgp" created="Tue, 11 Oct 2016 15:23:57 +0000"  >&lt;p&gt;BTW this issue could be the same affecting the FRS test (random alien ID).&lt;/p&gt;</comment>
                            <comment id="58384" author="shuva.jyoti.kar.87@gmail.com" created="Tue, 11 Oct 2016 18:04:15 +0000"  >&lt;p&gt;(In reply to Luis Gomez from comment #3)&lt;br/&gt;
&amp;gt; Once you get an alien ID further flow delete/push attempts also produce&lt;br/&gt;
&amp;gt; alien ID, if this is your question.&lt;/p&gt;

&lt;p&gt;There is a window in which the stats polling fetches the flow from the switch faster than the rpc add-flow callback succeeds resulting in the alien id. &lt;/p&gt;

&lt;p&gt;If we increase the stats polling interval, the window reduces. Manually its really difficult to hit that , its quite surprising to see that you are able to reproduce it deterministically.&lt;/p&gt;</comment>
                            <comment id="58385" author="ecelgp" created="Tue, 11 Oct 2016 18:16:33 +0000"  >&lt;p&gt;I think there is something else:&lt;/p&gt;

&lt;ul class=&quot;alternate&quot; type=&quot;square&quot;&gt;
	&lt;li&gt;After connecting switch it normally works.&lt;/li&gt;
	&lt;li&gt;After retrying a few times delete/add flow there is a moment after which it consistently fails to match.&lt;/li&gt;
	&lt;li&gt;The only way to get rid of the failure is to restart the switch connection.&lt;/li&gt;
&lt;/ul&gt;
</comment>
                            <comment id="58386" author="ecelgp" created="Wed, 12 Oct 2016 02:39:28 +0000"  >&lt;p&gt;Another observation is that I have to give 5 secs in automation between delete and create the flow to consitently reproduce.&lt;/p&gt;</comment>
                            <comment id="58387" author="shuva.jyoti.kar.87@gmail.com" created="Wed, 12 Oct 2016 15:28:51 +0000"  >&lt;p&gt;I tried it around 10/11 times manually today and could reproduce it only once.&lt;/p&gt;</comment>
                            <comment id="58388" author="ecelgp" created="Wed, 12 Oct 2016 16:00:41 +0000"  >&lt;p&gt;FYI I wrote a test that consistently reproduces the issue:&lt;/p&gt;

&lt;p&gt;&lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/46797/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/46797/&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;Hopefully this gets merged soon.&lt;/p&gt;</comment>
                            <comment id="58389" author="anipbu" created="Thu, 13 Oct 2016 15:50:04 +0000"  >&lt;p&gt;Based on comment #1, I believe this bug is targeted for Boron.&lt;/p&gt;</comment>
                            <comment id="58390" author="ecelgp" created="Fri, 14 Oct 2016 04:53:38 +0000"  >&lt;p&gt;Shuva, is there a way to disable the stats polling? if this is really interfering we can run the test without stats collection and see what happens.&lt;/p&gt;</comment>
                            <comment id="58391" author="shuva.jyoti.kar.87@gmail.com" created="Fri, 14 Oct 2016 05:11:31 +0000"  >&lt;p&gt;We can use PUT @ &lt;a href=&quot;http://localhost:8181/restconf/config/openflow-provider-config:openflow-provider-config/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://localhost:8181/restconf/config/openflow-provider-config:openflow-provider-config/&lt;/a&gt; with the following configuration &lt;/p&gt;

&lt;p&gt;{&lt;br/&gt;
                &quot;openflow-provider-config&quot;: &lt;/p&gt;
{
                                &quot;is-statistics-polling-off&quot;:  true
                }
&lt;p&gt;}&lt;/p&gt;

&lt;p&gt;To turn stats polling off. Please do it before connecting switches since the last time i checked it would restart the modules and with switches connected, it can result in huge errors.&lt;/p&gt;

&lt;p&gt;FYI &lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/46718/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/46718/&lt;/a&gt;&lt;br/&gt;
patch it can be configured in a cfg file that will persist reboots&lt;/p&gt;</comment>
                            <comment id="58392" author="ecelgp" created="Fri, 14 Oct 2016 15:24:41 +0000"  >&lt;p&gt;Hi Shuva,&lt;/p&gt;

&lt;p&gt;When I disable stats polling I do not get anything in inventory or topology when switch connects, I think something is broken when doing that because I also see this exception in karaf console:&lt;/p&gt;

&lt;p&gt;opendaylight-user@root&amp;gt;Exception in thread &quot;Thread-297&quot; io.netty.channel.unix.Errors$NativeIoException: bind() failed: Address already in use&lt;br/&gt;
	at io.netty.channel.unix.Errors.newIOException(Errors.java:115)&lt;br/&gt;
	at io.netty.channel.unix.Socket.bind(Socket.java:204)&lt;br/&gt;
	at io.netty.channel.epoll.EpollServerSocketChannel.doBind(EpollServerSocketChannel.java:91)&lt;br/&gt;
	at io.netty.channel.AbstractChannel$AbstractUnsafe.bind(AbstractChannel.java:490)&lt;br/&gt;
	at io.netty.channel.DefaultChannelPipeline$HeadContext.bind(DefaultChannelPipeline.java:1198)&lt;br/&gt;
	at io.netty.channel.AbstractChannelHandlerContext.invokeBind(AbstractChannelHandlerContext.java:481)&lt;br/&gt;
	at io.netty.channel.AbstractChannelHandlerContext.bind(AbstractChannelHandlerContext.java:466)&lt;br/&gt;
	at io.netty.channel.ChannelDuplexHandler.bind(ChannelDuplexHandler.java:38)&lt;br/&gt;
	at io.netty.handler.logging.LoggingHandler.bind(LoggingHandler.java:197)&lt;br/&gt;
	at io.netty.channel.AbstractChannelHandlerContext.invokeBind(AbstractChannelHandlerContext.java:481)&lt;br/&gt;
	at io.netty.channel.AbstractChannelHandlerContext.bind(AbstractChannelHandlerContext.java:466)&lt;br/&gt;
	at io.netty.channel.DefaultChannelPipeline.bind(DefaultChannelPipeline.java:944)&lt;br/&gt;
	at io.netty.channel.AbstractChannel.bind(AbstractChannel.java:203)&lt;br/&gt;
	at io.netty.bootstrap.AbstractBootstrap$2.run(AbstractBootstrap.java:350)&lt;br/&gt;
	at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:358)&lt;br/&gt;
	at io.netty.channel.epoll.EpollEventLoop.run(EpollEventLoop.java:307)&lt;br/&gt;
	at io.netty.util.concurrent.SingleThreadEventExecutor$2.run(SingleThreadEventExecutor.java:112)&lt;br/&gt;
	at io.netty.util.concurrent.DefaultThreadFactory$DefaultRunnableDecorator.run(DefaultThreadFactory.java:145)&lt;br/&gt;
	at java.lang.Thread.run(Thread.java:745)&lt;/p&gt;</comment>
                            <comment id="58393" author="shuva.jyoti.kar.87@gmail.com" created="Fri, 14 Oct 2016 15:27:53 +0000"  >&lt;p&gt;did you change it before connecting the switch ?&lt;/p&gt;</comment>
                            <comment id="58394" author="ecelgp" created="Fri, 14 Oct 2016 15:29:57 +0000"  >&lt;p&gt;Yes, also in a second attempt I do not get the exception but still the switch does not show in topology or inventory. Have you tried this lately? I am using latest Boron.&lt;/p&gt;</comment>
                            <comment id="58395" author="shuva.jyoti.kar.87@gmail.com" created="Fri, 14 Oct 2016 15:32:56 +0000"  >&lt;p&gt;had tried it about a month back, will try and keep you posted&lt;/p&gt;</comment>
                            <comment id="58396" author="shuva.jyoti.kar.87@gmail.com" created="Fri, 14 Oct 2016 15:34:21 +0000"  >&lt;p&gt;(In reply to Shuva Jyoti Kar from comment #16)&lt;br/&gt;
&amp;gt; had tried it about a month back, will try and keep you posted&lt;/p&gt;

&lt;p&gt;Also try provisioning a flow to see if that triggers an update ?&lt;/p&gt;</comment>
                            <comment id="58397" author="ecelgp" created="Fri, 14 Oct 2016 15:59:50 +0000"  >&lt;p&gt;Just tried with boron release and same issue so I opened this:&lt;/p&gt;

&lt;p&gt;&lt;a href=&quot;https://bugs.opendaylight.org/show_bug.cgi?id=6941&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://bugs.opendaylight.org/show_bug.cgi?id=6941&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;BR/Luis&lt;/p&gt;</comment>
                            <comment id="58398" author="ecelgp" created="Fri, 14 Oct 2016 16:15:09 +0000"  >&lt;p&gt;For this specific issue I have 2 more findings:&lt;/p&gt;

&lt;ul class=&quot;alternate&quot; type=&quot;square&quot;&gt;
	&lt;li&gt;The isse is reproducible with any flow body.&lt;/li&gt;
	&lt;li&gt;The time between the flow is deleted from oper DS and we repush the flow has to be 5s, with more or less time, it is more difficult to reproduce.&lt;/li&gt;
&lt;/ul&gt;


&lt;p&gt;In partcular this is what the automation does (consistently fails):&lt;/p&gt;

&lt;p&gt;Loop on:&lt;/p&gt;

&lt;p&gt;    Sleep    5&lt;br/&gt;
    FlowLib.Create Flow Variables For Suite From XML File    ${XmlsDir}/${flowfile}&lt;br/&gt;
    FlowLib.Add Flow Via Restconf    ${switch_idx}    ${table_id}    ${data}&lt;br/&gt;
    BuiltIn.Wait Until Keyword Succeeds    10s    1s    FlowLib.Check Operational Flow    ${True}    ${data}&lt;br/&gt;
    FlowLib.Check Datastore Presence    ${flowfile}    ${True}    ${True}    ${False}    ${True}&lt;br/&gt;
    FlowLib.Delete Flow Via Restconf    ${switch_idx}    ${table_id}    ${flow_id}&lt;br/&gt;
    BuiltIn.Wait Until Keyword Succeeds    10s    1s    FlowLib.Check Operational Flow    ${False}    ${data}&lt;br/&gt;
    FlowLib.Check Datastore Presence    ${flowfile}    ${False}    ${False}    ${True}&lt;/p&gt;</comment>
                            <comment id="58399" author="shuva.jyoti.kar.87@gmail.com" created="Sat, 15 Oct 2016 06:26:02 +0000"  >&lt;p&gt;I tried the 5 sec delay between deletion and repush , and i could see the alien flow id once, but the next sec i retried it was showing the actual one. I tried about 23 times today, no luck.&lt;/p&gt;</comment>
                            <comment id="58400" author="shuva.jyoti.kar.87@gmail.com" created="Sat, 15 Oct 2016 06:27:21 +0000"  >&lt;p&gt;I tried with this flow&lt;/p&gt;

&lt;p&gt;&amp;lt;?xml version=&quot;1.0&quot; encoding=&quot;UTF-8&quot; standalone=&quot;no&quot;?&amp;gt;&lt;br/&gt;
&amp;lt;flow xmlns=&quot;urn:opendaylight:flow:inventory&quot;&amp;gt;&lt;br/&gt;
    &amp;lt;hard-timeout&amp;gt;0&amp;lt;/hard-timeout&amp;gt;&lt;br/&gt;
    &amp;lt;idle-timeout&amp;gt;0&amp;lt;/idle-timeout&amp;gt;&lt;br/&gt;
    &amp;lt;priority&amp;gt;2&amp;lt;/priority&amp;gt;&lt;br/&gt;
    &amp;lt;flow-name&amp;gt;flow2&amp;lt;/flow-name&amp;gt;&lt;br/&gt;
    &amp;lt;match&amp;gt;&lt;br/&gt;
        &amp;lt;ethernet-match&amp;gt;&lt;br/&gt;
            &amp;lt;ethernet-type&amp;gt;&lt;br/&gt;
                &amp;lt;type&amp;gt;2048&amp;lt;/type&amp;gt;&lt;br/&gt;
            &amp;lt;/ethernet-type&amp;gt;&lt;br/&gt;
        &amp;lt;/ethernet-match&amp;gt;&lt;br/&gt;
        &amp;lt;ipv4-destination&amp;gt;10.0.17.2/32&amp;lt;/ipv4-destination&amp;gt;&lt;br/&gt;
    &amp;lt;/match&amp;gt;&lt;br/&gt;
    &amp;lt;id&amp;gt;2&amp;lt;/id&amp;gt;&lt;br/&gt;
    &amp;lt;table_id&amp;gt;0&amp;lt;/table_id&amp;gt;&lt;br/&gt;
    &amp;lt;instructions&amp;gt;&lt;br/&gt;
        &amp;lt;instruction&amp;gt;&lt;br/&gt;
            &amp;lt;order&amp;gt;0&amp;lt;/order&amp;gt;&lt;br/&gt;
            &amp;lt;apply-actions&amp;gt;&lt;br/&gt;
                &amp;lt;action&amp;gt;&lt;br/&gt;
                    &amp;lt;output-action&amp;gt;&lt;br/&gt;
                        &amp;lt;output-node-connector&amp;gt;1&amp;lt;/output-node-connector&amp;gt;&lt;br/&gt;
                    &amp;lt;/output-action&amp;gt;&lt;br/&gt;
                    &amp;lt;order&amp;gt;0&amp;lt;/order&amp;gt;&lt;br/&gt;
                &amp;lt;/action&amp;gt;&lt;br/&gt;
            &amp;lt;/apply-actions&amp;gt;&lt;br/&gt;
        &amp;lt;/instruction&amp;gt;&lt;br/&gt;
    &amp;lt;/instructions&amp;gt;&lt;br/&gt;
&amp;lt;/flow&amp;gt;&lt;/p&gt;</comment>
                            <comment id="58401" author="ecelgp" created="Sat, 15 Oct 2016 16:48:46 +0000"  >&lt;p&gt;Shuva I was wrong, the issue shows normally with big flows (lot of matches like the one I posted in the bug) and I was also able to reproduce it by just adding (no deleting) different big flows every 8 secs. I will update the CI test to show this too.&lt;/p&gt;</comment>
                            <comment id="58402" author="shuva.jyoti.kar.87@gmail.com" created="Sat, 15 Oct 2016 18:17:24 +0000"  >&lt;p&gt;No issues Luis. I will try it with the flow that you had reported initially with and then come back tomorrow. thanks&lt;/p&gt;</comment>
                            <comment id="58403" author="ecelgp" created="Sat, 15 Oct 2016 19:50:44 +0000"  >&lt;p&gt;Here it is:&lt;/p&gt;

&lt;p&gt;&lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/46984&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/46984&lt;/a&gt;&lt;/p&gt;</comment>
                            <comment id="58404" author="shuva.jyoti.kar.87@gmail.com" created="Mon, 17 Oct 2016 08:47:55 +0000"  >&lt;p&gt;(In reply to Luis Gomez from comment #24)&lt;br/&gt;
&amp;gt; Here it is:&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; &lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/46984&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/46984&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;I am trying with ovs2.4 and getting OFP_BMC_BAD_WILDCARDS errors. Any other example flow that will work with ovs2.4 ?&lt;/p&gt;</comment>
                            <comment id="58405" author="miroslav.macko" created="Mon, 17 Oct 2016 14:34:03 +0000"  >&lt;p&gt;(In reply to Shuva Jyoti Kar from comment #25)&lt;br/&gt;
&amp;gt; (In reply to Luis Gomez from comment #24)&lt;br/&gt;
&amp;gt; &amp;gt; Here it is:&lt;br/&gt;
&amp;gt; &amp;gt; &lt;br/&gt;
&amp;gt; &amp;gt; &lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/46984&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/46984&lt;/a&gt;&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; I am trying with ovs2.4 and getting OFP_BMC_BAD_WILDCARDS errors. Any other&lt;br/&gt;
&amp;gt; example flow that will work with ovs2.4 ?&lt;/p&gt;

&lt;p&gt;Hello Shuva,&lt;/p&gt;

&lt;p&gt;You can use flow from LUIS on the very top.&lt;/p&gt;

&lt;p&gt;You just need to change&lt;/p&gt;

&lt;p&gt;&amp;lt;ipv6-source&amp;gt;1234:5678:9ABC:DEF0:FDCD:A987:6543:210F/76&amp;lt;/ipv6-source&amp;gt;&lt;/p&gt;

&lt;p&gt;to &lt;/p&gt;

&lt;p&gt;&amp;lt;ipv6-source&amp;gt;1234:5678:9ABC:DEF0:FDC0::/76&amp;lt;/ipv6-source&amp;gt;&lt;/p&gt;

&lt;p&gt;Miro&lt;/p&gt;

&lt;p&gt;&amp;lt;flow xmlns=&quot;urn:opendaylight:flow:inventory&quot;&amp;gt;&lt;br/&gt;
    &amp;lt;strict&amp;gt;false&amp;lt;/strict&amp;gt;&lt;br/&gt;
    &amp;lt;flow-name&amp;gt;FooXf21&amp;lt;/flow-name&amp;gt;&lt;br/&gt;
    &amp;lt;id&amp;gt;144&amp;lt;/id&amp;gt;&lt;br/&gt;
    &amp;lt;cookie_mask&amp;gt;255&amp;lt;/cookie_mask&amp;gt;&lt;br/&gt;
    &amp;lt;cookie&amp;gt;21&amp;lt;/cookie&amp;gt;&lt;br/&gt;
    &amp;lt;table_id&amp;gt;2&amp;lt;/table_id&amp;gt;&lt;br/&gt;
    &amp;lt;priority&amp;gt;21&amp;lt;/priority&amp;gt;&lt;br/&gt;
    &amp;lt;installHw&amp;gt;false&amp;lt;/installHw&amp;gt;&lt;br/&gt;
    &amp;lt;instructions&amp;gt;&lt;br/&gt;
        &amp;lt;instruction&amp;gt;&lt;br/&gt;
            &amp;lt;order&amp;gt;0&amp;lt;/order&amp;gt;&lt;br/&gt;
            &amp;lt;apply-actions&amp;gt;&lt;br/&gt;
                &amp;lt;action&amp;gt;&lt;br/&gt;
                    &amp;lt;order&amp;gt;0&amp;lt;/order&amp;gt;&lt;br/&gt;
                    &amp;lt;dec-nw-ttl/&amp;gt;&lt;br/&gt;
                &amp;lt;/action&amp;gt;&lt;br/&gt;
            &amp;lt;/apply-actions&amp;gt;&lt;br/&gt;
        &amp;lt;/instruction&amp;gt;&lt;br/&gt;
    &amp;lt;/instructions&amp;gt;&lt;br/&gt;
    &amp;lt;match&amp;gt;&lt;br/&gt;
        &amp;lt;ethernet-match&amp;gt;&lt;br/&gt;
            &amp;lt;ethernet-type&amp;gt;&lt;br/&gt;
                &amp;lt;type&amp;gt;34525&amp;lt;/type&amp;gt;&lt;br/&gt;
            &amp;lt;/ethernet-type&amp;gt;&lt;br/&gt;
        &amp;lt;/ethernet-match&amp;gt;&lt;br/&gt;
        &amp;lt;ipv6-source&amp;gt;1234:5678:9abc:def0:fdc0::/76&amp;lt;/ipv6-source&amp;gt;&lt;br/&gt;
        &amp;lt;ipv6-destination&amp;gt;fe80::2acf:e9ff:fe21:6431/128&amp;lt;/ipv6-destination&amp;gt;&lt;br/&gt;
        &amp;lt;metadata&amp;gt;&lt;br/&gt;
            &amp;lt;metadata&amp;gt;12345&amp;lt;/metadata&amp;gt;&lt;br/&gt;
        &amp;lt;/metadata&amp;gt;&lt;br/&gt;
        &amp;lt;ip-match&amp;gt;&lt;br/&gt;
            &amp;lt;ip-protocol&amp;gt;17&amp;lt;/ip-protocol&amp;gt;&lt;br/&gt;
            &amp;lt;ip-dscp&amp;gt;8&amp;lt;/ip-dscp&amp;gt;&lt;br/&gt;
            &amp;lt;ip-ecn&amp;gt;3&amp;lt;/ip-ecn&amp;gt;&lt;br/&gt;
        &amp;lt;/ip-match&amp;gt;&lt;br/&gt;
        &amp;lt;udp-source-port&amp;gt;25364&amp;lt;/udp-source-port&amp;gt;&lt;br/&gt;
        &amp;lt;udp-destination-port&amp;gt;8080&amp;lt;/udp-destination-port&amp;gt;&lt;br/&gt;
    &amp;lt;/match&amp;gt;&lt;br/&gt;
&amp;lt;/flow&amp;gt;&lt;/p&gt;</comment>
                            <comment id="58406" author="shuva.jyoti.kar.87@gmail.com" created="Mon, 17 Oct 2016 16:20:27 +0000"  >&lt;p&gt;Strangely enough 13/13 times i tried this but donot see this in my set-up, with 8 secs delay between additions, and a total 10-15 flows .&lt;/p&gt;

&lt;p&gt;@Miroslav any points you have&lt;/p&gt;</comment>
                            <comment id="58407" author="miroslav.macko" created="Mon, 17 Oct 2016 16:35:04 +0000"  >&lt;p&gt;Hello Shuva,&lt;/p&gt;

&lt;p&gt;I think this happen when flow is removed from config, and statistics(in aprox. 3 seconds interval) bring the flow. &lt;/p&gt;

&lt;p&gt;It is not present in DeviceFlowRegistry, so it will create flow with alien id.&lt;/p&gt;

&lt;p&gt;I was able to reproduce. I will attach the karaf log. &lt;/p&gt;

&lt;p&gt;We have an idea, what could fix it. I will try.&lt;/p&gt;

&lt;p&gt;If you have some idea, please feel free to do it.&lt;/p&gt;

&lt;p&gt;Thanks,&lt;br/&gt;
Miro&lt;/p&gt;</comment>
                            <comment id="58445" author="miroslav.macko" created="Mon, 17 Oct 2016 16:36:04 +0000"  >&lt;p&gt;Attachment device-flow-registry-karaf.log has been added with description: karaf log&lt;/p&gt;</comment>
                            <comment id="58408" author="shuva.jyoti.kar.87@gmail.com" created="Mon, 17 Oct 2016 17:51:21 +0000"  >&lt;p&gt;(In reply to Miroslav Macko from comment #28)&lt;br/&gt;
&amp;gt; Hello Shuva,&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; I think this happen when flow is removed from config, and statistics(in&lt;br/&gt;
&amp;gt; aprox. 3 seconds interval) bring the flow. &lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; It is not present in DeviceFlowRegistry, so it will create flow with alien&lt;br/&gt;
&amp;gt; id.&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; I was able to reproduce. I will attach the karaf log. &lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; We have an idea, what could fix it. I will try.&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; If you have some idea, please feel free to do it.&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; Thanks,&lt;br/&gt;
&amp;gt; Miro&lt;/p&gt;

&lt;p&gt;Yes Miroslav. i too guessed the same. Please try your fix, i am yet to arrive at a solution to address it permanently. However i feel with stats turned off, we might not see this problem&lt;/p&gt;</comment>
                            <comment id="58409" author="ecelgp" created="Mon, 17 Oct 2016 19:21:49 +0000"  >&lt;p&gt;Shuva, please save the flow miroslav sent as f21.xml and run this script in same folder:&lt;/p&gt;

&lt;p&gt;#!/bin/bash&lt;br/&gt;
for i in `seq 1 5`&lt;br/&gt;
do&lt;br/&gt;
    echo &quot;Add flow ${i} time&quot;&lt;br/&gt;
    curl -u admin:admin -H &quot;Expect:&quot; -H &quot;Content-Type:application/xml&quot; -H &quot;Accept:application/xml&quot; -X PUT &lt;a href=&quot;http://localhost:8181/restconf/config/opendaylight-inventory:nodes/node/openflow:1/table/2/flow/144&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://localhost:8181/restconf/config/opendaylight-inventory:nodes/node/openflow:1/table/2/flow/144&lt;/a&gt; -d @f21.xml&lt;br/&gt;
    sleep 5&lt;br/&gt;
    echo &quot;Get Flow in Oper&quot;&lt;br/&gt;
    curl -u admin:admin -H &quot;Expect:&quot; -H &quot;Content-Type:application/xml&quot; -H &quot;Accept:application/xml&quot; -X GET &lt;a href=&quot;http://localhost:8181/restconf/operational/opendaylight-inventory:nodes/node/openflow:1/table/2&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://localhost:8181/restconf/operational/opendaylight-inventory:nodes/node/openflow:1/table/2&lt;/a&gt;&lt;br/&gt;
    echo -e &quot;\nDelete flow&quot;&lt;br/&gt;
    curl -u admin:admin -H &quot;Expect:&quot; -H &quot;Content-Type:application/xml&quot; -H &quot;Accept:application/xml&quot; -X DELETE &lt;a href=&quot;http://localhost:8181/restconf/config/opendaylight-inventory:nodes/node/openflow:1/table/2/flow/144&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://localhost:8181/restconf/config/opendaylight-inventory:nodes/node/openflow:1/table/2/flow/144&lt;/a&gt;&lt;br/&gt;
    sleep 6&lt;br/&gt;
done&lt;/p&gt;

&lt;p&gt;At least in my laptop I am getting alien ID very consistent before script reaches 5 attempts.&lt;/p&gt;</comment>
                            <comment id="58410" author="ecelgp" created="Mon, 17 Oct 2016 19:26:39 +0000"  >&lt;p&gt;I think the failure is even more reproducible when we add some fix sleep after stats has been updated in controller, like we do in the robot test.&lt;/p&gt;</comment>
                            <comment id="58411" author="ecelgp" created="Mon, 17 Oct 2016 20:09:28 +0000"  >&lt;p&gt;Definitely it has to be with stats collection, if I use this script sleeping 6 secs after stats are received for deleted flow, I hit the issue much more often:&lt;/p&gt;

&lt;p&gt;#!/bin/bash&lt;br/&gt;
for i in `seq 1 5`&lt;br/&gt;
do&lt;br/&gt;
    echo &quot;Add flow ${i} time&quot;&lt;br/&gt;
    curl -u admin:admin -H &quot;Expect:&quot; -H &quot;Content-Type:application/xml&quot; -H &quot;Accept:application/xml&quot; -X PUT &lt;a href=&quot;http://localhost:8181/restconf/config/opendaylight-inventory:nodes/node/openflow:1/table/2/flow/144&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://localhost:8181/restconf/config/opendaylight-inventory:nodes/node/openflow:1/table/2/flow/144&lt;/a&gt; -d @f21.xml&lt;br/&gt;
    while true&lt;br/&gt;
    do&lt;br/&gt;
        STRING=`curl -u admin:admin -H &quot;Expect:&quot; -H &quot;Content-Type:application/xml&quot; -H &quot;Accept:application/xml&quot; -X GET &lt;a href=&quot;http://localhost:8181/restconf/operational/opendaylight-inventory:nodes/node/openflow:1/table/2&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://localhost:8181/restconf/operational/opendaylight-inventory:nodes/node/openflow:1/table/2&lt;/a&gt; 2&amp;gt;/dev/null`&lt;br/&gt;
        if [[ ${STRING} == &lt;b&gt;&quot;&amp;lt;priority&amp;gt;21&amp;lt;/priority&amp;gt;&quot;&lt;/b&gt; ]]; then&lt;br/&gt;
            break&lt;br/&gt;
        fi&lt;br/&gt;
    done&lt;br/&gt;
    echo &quot;Get Flow in Oper&quot;&lt;br/&gt;
    curl -u admin:admin -H &quot;Expect:&quot; -H &quot;Content-Type:application/xml&quot; -H &quot;Accept:application/xml&quot; -X GET &lt;a href=&quot;http://localhost:8181/restconf/operational/opendaylight-inventory:nodes/node/openflow:1/table/2&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://localhost:8181/restconf/operational/opendaylight-inventory:nodes/node/openflow:1/table/2&lt;/a&gt;&lt;br/&gt;
    echo -e &quot;\nDelete flow&quot;&lt;br/&gt;
    curl -u admin:admin -H &quot;Expect:&quot; -H &quot;Content-Type:application/xml&quot; -H &quot;Accept:application/xml&quot; -X DELETE &lt;a href=&quot;http://localhost:8181/restconf/config/opendaylight-inventory:nodes/node/openflow:1/table/2/flow/144&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://localhost:8181/restconf/config/opendaylight-inventory:nodes/node/openflow:1/table/2/flow/144&lt;/a&gt;&lt;br/&gt;
    while true&lt;br/&gt;
    do&lt;br/&gt;
        STRING=`curl -u admin:admin -H &quot;Expect:&quot; -H &quot;Content-Type:application/xml&quot; -H &quot;Accept:application/xml&quot; -X GET &lt;a href=&quot;http://localhost:8181/restconf/operational/opendaylight-inventory:nodes/node/openflow:1/table/2&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;http://localhost:8181/restconf/operational/opendaylight-inventory:nodes/node/openflow:1/table/2&lt;/a&gt; 2&amp;gt;/dev/null`&lt;br/&gt;
        if [[ ${STRING} != &lt;b&gt;&quot;&amp;lt;priority&amp;gt;21&amp;lt;/priority&amp;gt;&quot;&lt;/b&gt; ]]; then&lt;br/&gt;
            break&lt;br/&gt;
        fi&lt;br/&gt;
    done&lt;br/&gt;
    sleep 6&lt;br/&gt;
done&lt;/p&gt;</comment>
                            <comment id="58412" author="shuva.jyoti.kar.87@gmail.com" created="Tue, 18 Oct 2016 10:13:05 +0000"  >&lt;p&gt;Luis,&lt;/p&gt;

&lt;p&gt;With &lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/47004/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/47004/&lt;/a&gt;, turning off stats would be easy. And with stats off we might not see this issue&lt;/p&gt;</comment>
                            <comment id="58413" author="miroslav.macko" created="Tue, 18 Oct 2016 17:21:54 +0000"  >&lt;p&gt;Hi guys,&lt;/p&gt;

&lt;p&gt;@Shuva if stats are turned off, flow will get to the operational only with reconciliation. Right? Is it ok?&lt;/p&gt;

&lt;p&gt;@Luis I have pushed the patch&lt;/p&gt;

&lt;p&gt;&lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/47085/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/47085/&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;Could you please test it?&lt;/p&gt;

&lt;p&gt;Thank you,&lt;br/&gt;
Miro&lt;/p&gt;</comment>
                            <comment id="58414" author="shuva.jyoti.kar.87@gmail.com" created="Tue, 18 Oct 2016 17:49:19 +0000"  >&lt;p&gt;(In reply to Miroslav Macko from comment #35)&lt;br/&gt;
&amp;gt; Hi guys,&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; @Shuva if stats are turned off, flow will get to the operational only with&lt;br/&gt;
&amp;gt; reconciliation. Right? Is it ok?&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; @Luis I have pushed the patch&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; &lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/47085/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/47085/&lt;/a&gt;&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; Could you please test it?&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; Thank you,&lt;br/&gt;
&amp;gt; Miro&lt;/p&gt;


&lt;p&gt;If stats are turned off, then by current design they will appear in the Oper-Ds on rpcs success. It would be better not to change that behaviour since it gives a fair guarantee to applications whether the flow succeeded or not.&lt;/p&gt;

&lt;p&gt;Could not access your patch, could you please add me to it.&lt;/p&gt;</comment>
                            <comment id="58415" author="miroslav.macko" created="Tue, 18 Oct 2016 21:16:22 +0000"  >&lt;p&gt;Hello Shuva,&lt;/p&gt;

&lt;p&gt;But this is not added by RPCs. It is put to config and it is added to operational by FRM or FRS using statistics. Isn&apos;t it?&lt;/p&gt;

&lt;p&gt;Thanks,&lt;br/&gt;
Miro&lt;/p&gt;</comment>
                            <comment id="58416" author="ecelgp" created="Tue, 18 Oct 2016 21:44:58 +0000"  >&lt;p&gt;Hi Miroslav, this is the result of your patch:&lt;/p&gt;

&lt;p&gt;&lt;a href=&quot;https://logs.opendaylight.org/releng/jenkins092/openflowplugin-csit-1node-flow-services-only-carbon/172/archives/log.html.gz&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://logs.opendaylight.org/releng/jenkins092/openflowplugin-csit-1node-flow-services-only-carbon/172/archives/log.html.gz&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;It does fix the alien ID issue but for some reason it breaks the RPC update operation.&lt;/p&gt;</comment>
                            <comment id="58417" author="miroslav.macko" created="Wed, 19 Oct 2016 07:38:00 +0000"  >&lt;p&gt;Hello Luis,&lt;/p&gt;

&lt;p&gt;I see. I will check it. Thanks a lot.&lt;/p&gt;

&lt;p&gt;Miro&lt;/p&gt;</comment>
                            <comment id="58418" author="shuva.jyoti.kar.87@gmail.com" created="Wed, 19 Oct 2016 09:05:14 +0000"  >&lt;p&gt;(In reply to Miroslav Macko from comment #37)&lt;br/&gt;
&amp;gt; Hello Shuva,&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; But this is not added by RPCs. It is put to config and it is added to&lt;br/&gt;
&amp;gt; operational by FRM or FRS using statistics. Isn&apos;t it?&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; Thanks,&lt;br/&gt;
&amp;gt; Miro&lt;/p&gt;

&lt;p&gt;You are correct Miroslav!&lt;br/&gt;
I only wanted to point out that without statistics, on the addFlow rpc success we write the flow to the operational&lt;/p&gt;</comment>
                            <comment id="58419" author="shuva.jyoti.kar.87@gmail.com" created="Wed, 19 Oct 2016 15:56:27 +0000"  >&lt;p&gt;(In reply to Luis Gomez from comment #38)&lt;br/&gt;
&amp;gt; Hi Miroslav, this is the result of your patch:&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; &lt;a href=&quot;https://logs.opendaylight.org/releng/jenkins092/openflowplugin-csit-1node-&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://logs.opendaylight.org/releng/jenkins092/openflowplugin-csit-1node-&lt;/a&gt;&lt;br/&gt;
&amp;gt; flow-services-only-carbon/172/archives/log.html.gz&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; It does fix the alien ID issue but for some reason it breaks the RPC update&lt;br/&gt;
&amp;gt; operation.&lt;/p&gt;

&lt;p&gt;Luis, in the CSIT TCs, do we do some kind of a retry to check if the flow is in the operational DS? Or is it a one time check?&lt;/p&gt;</comment>
                            <comment id="58420" author="miroslav.macko" created="Wed, 19 Oct 2016 17:05:51 +0000"  >&lt;p&gt;Hi guys,&lt;/p&gt;

&lt;p&gt;I have pushed the patch:&lt;/p&gt;

&lt;p&gt;&lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/47138/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/47138/&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;It should work already. &lt;/p&gt;

&lt;p&gt;But I need to check behavior with statistics turned off yet. &lt;/p&gt;

&lt;p&gt;I will do it tomorrow.&lt;/p&gt;

&lt;p&gt;Thanks.&lt;br/&gt;
Miro&lt;/p&gt;</comment>
                            <comment id="58421" author="ecelgp" created="Wed, 19 Oct 2016 17:22:15 +0000"  >&lt;p&gt;In the failing test the flows seem to be updated but no flow shows up in operational after 3 sec. There are multiple retries after during 5 secs (check updated flow) but in the last retry I still see no flow. I can retrigger the test to see if this consisten or just a transient failure.&lt;/p&gt;</comment>
                            <comment id="58422" author="ecelgp" created="Wed, 19 Oct 2016 18:03:58 +0000"  >&lt;p&gt;transient discarded, something in Miroslav&apos;s patch impacts stats collection after RPC update operation.&lt;/p&gt;

&lt;p&gt;&lt;a href=&quot;https://jenkins.opendaylight.org/releng/view/openflowplugin/job/openflowplugin-csit-1node-flow-services-only-carbon/175/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://jenkins.opendaylight.org/releng/view/openflowplugin/job/openflowplugin-csit-1node-flow-services-only-carbon/175/&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;BR/Luis&lt;/p&gt;</comment>
                            <comment id="58423" author="miroslav.macko" created="Wed, 19 Oct 2016 21:24:09 +0000"  >&lt;p&gt;(In reply to Luis Gomez from comment #44)&lt;br/&gt;
&amp;gt; transient discarded, something in Miroslav&apos;s patch impacts stats collection&lt;br/&gt;
&amp;gt; after RPC update operation.&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; &lt;a href=&quot;https://jenkins.opendaylight.org/releng/view/openflowplugin/job/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://jenkins.opendaylight.org/releng/view/openflowplugin/job/&lt;/a&gt;&lt;br/&gt;
&amp;gt; openflowplugin-csit-1node-flow-services-only-carbon/175/&lt;br/&gt;
&amp;gt; &lt;br/&gt;
&amp;gt; BR/Luis&lt;/p&gt;

&lt;p&gt;Hello Luis,&lt;/p&gt;

&lt;p&gt;This test(175) ran on this patch?&lt;/p&gt;

&lt;p&gt;&lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/47138/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/47138/&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;Thanks,&lt;br/&gt;
Miro&lt;/p&gt;</comment>
                            <comment id="58424" author="ecelgp" created="Wed, 19 Oct 2016 21:47:09 +0000"  >&lt;p&gt;No, still the old patch, you want to run the test with the new patch? I can do that too.&lt;/p&gt;</comment>
                            <comment id="58425" author="miroslav.macko" created="Thu, 20 Oct 2016 08:00:37 +0000"  >&lt;p&gt;Hello Luis,&lt;/p&gt;

&lt;p&gt;It would be fine, if you will test it. It should fix RPC updates in test:&lt;/p&gt;

&lt;p&gt;/test/csit/suites/openflowplugin/Flows_Additional_TCs/Stat_Manager_extended/020_SM_sal_add_upd_del_flows.robot&lt;/p&gt;

&lt;p&gt;But I need to test use case when statistics are off.&lt;/p&gt;

&lt;p&gt;I will let you know, when it is done.&lt;/p&gt;

&lt;p&gt;Thanks a lot,&lt;br/&gt;
Miro&lt;/p&gt;</comment>
                            <comment id="58426" author="miroslav.macko" created="Thu, 20 Oct 2016 18:36:37 +0000"  >&lt;p&gt;Hello Luis,&lt;/p&gt;

&lt;p&gt;I have pushed the patch:&lt;/p&gt;

&lt;p&gt;&lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/47138/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/47138/&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;Could you please test it?&lt;/p&gt;

&lt;p&gt;I am not seeing the problem with &quot;add port&quot; locally. &lt;/p&gt;

&lt;p&gt;Please let me know, it the problem persists.&lt;/p&gt;

&lt;p&gt;Thank you,&lt;br/&gt;
Miro&lt;/p&gt;</comment>
                            <comment id="58427" author="shuva.jyoti.kar.87@gmail.com" created="Sun, 23 Oct 2016 05:37:20 +0000"  >&lt;p&gt;Pushed on master:&lt;br/&gt;
&lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/47138/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/47138/&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;and stable/bo&lt;br/&gt;
&lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/47320&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/47320&lt;/a&gt;&lt;/p&gt;</comment>
                            <comment id="58428" author="ecelgp" created="Mon, 24 Oct 2016 16:24:06 +0000"  >&lt;p&gt;Yes, this is fixed now.&lt;/p&gt;</comment>
                            <comment id="58429" author="ecelgp" created="Thu, 8 Dec 2016 19:14:27 +0000"  >&lt;p&gt;We have to reopen this bug for stable boron as current fix has been reverted because it  changes plugin behavior.&lt;/p&gt;</comment>
                            <comment id="58430" author="anipbu" created="Tue, 13 Dec 2016 01:20:19 +0000"  >&lt;p&gt;Are there plans to fix it now, or can we retarget for Boron-SR3 for more time?&lt;/p&gt;</comment>
                            <comment id="58431" author="ecelgp" created="Tue, 13 Dec 2016 03:08:16 +0000"  >&lt;p&gt;Lower down priority as this does not impact forwarding, just operational info + some conditions are required in order to reproduce (e.g. big flows pushed at very specific interval)&lt;/p&gt;</comment>
                            <comment id="58432" author="tomas.slusny@pantheon.tech" created="Fri, 24 Feb 2017 08:47:23 +0000"  >&lt;p&gt;Pushed correct fix to Gerrit: &lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/52237/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/52237/&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;Tested it with Luis reproduction steps, and without this patch, the bug was still present, but after my changes, I was not able to reproduce it anymore. Also, tested the case, that previous patch was not checking for, and that is manually adding flow via OVS CLI, and it got properly propagated to operational.&lt;/p&gt;

&lt;p&gt;Luis, can you check this and verify that it resolves this issue?&lt;/p&gt;</comment>
                            <comment id="58433" author="ecelgp" created="Fri, 24 Feb 2017 16:41:04 +0000"  >&lt;p&gt;Hi Tomas, to verify you just need to run the test-openflowplugin-core, and check this job: &lt;a href=&quot;https://logs.opendaylight.org/releng/jenkins092/openflowplugin-csit-1node-flow-services-only-carbon/532/archives/log.html.gz&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://logs.opendaylight.org/releng/jenkins092/openflowplugin-csit-1node-flow-services-only-carbon/532/archives/log.html.gz&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;It seems the issue is still there (see the &lt;a href=&quot;https://jira.opendaylight.org/browse/OPNFLWPLUG-796&quot; title=&quot;Flow matching function (operational flow reconciliation) is not stable&quot; class=&quot;issue-link&quot; data-issue-key=&quot;OPNFLWPLUG-796&quot;&gt;&lt;del&gt;OPNFLWPLUG-796&lt;/del&gt;&lt;/a&gt; at the end).&lt;/p&gt;</comment>
                            <comment id="58434" author="ecelgp" created="Fri, 24 Feb 2017 16:44:28 +0000"  >&lt;p&gt;Hi Tomas, to verify you just need to run the test-openflowplugin-core, and check this job: &lt;a href=&quot;https://logs.opendaylight.org/releng/jenkins092/openflowplugin-csit-1node-flow-services-only-carbon/532/archives/log.html.gz&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://logs.opendaylight.org/releng/jenkins092/openflowplugin-csit-1node-flow-services-only-carbon/532/archives/log.html.gz&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;It seems the issue is still there (see the &lt;a href=&quot;https://jira.opendaylight.org/browse/OPNFLWPLUG-796&quot; title=&quot;Flow matching function (operational flow reconciliation) is not stable&quot; class=&quot;issue-link&quot; data-issue-key=&quot;OPNFLWPLUG-796&quot;&gt;&lt;del&gt;OPNFLWPLUG-796&lt;/del&gt;&lt;/a&gt; at the end).&lt;/p&gt;</comment>
                            <comment id="58435" author="ecelgp" created="Mon, 13 Mar 2017 19:28:57 +0000"  >&lt;p&gt;I think current patch is good.&lt;/p&gt;</comment>
                            <comment id="58436" author="tomas.slusny@pantheon.tech" created="Thu, 16 Mar 2017 14:06:15 +0000"  >&lt;p&gt;Patch was merged, so I think that this one can be closed, right?&lt;/p&gt;</comment>
                            <comment id="58437" author="ecelgp" created="Sun, 19 Mar 2017 20:35:33 +0000"  >&lt;p&gt;Tomas, can you cherry pick for stable/boron?&lt;/p&gt;</comment>
                            <comment id="58438" author="ecelgp" created="Sun, 19 Mar 2017 20:37:18 +0000"  >&lt;p&gt;Reopen so we do not forget to cherry-pick.&lt;/p&gt;</comment>
                            <comment id="58439" author="tomas.slusny@pantheon.tech" created="Mon, 20 Mar 2017 09:31:09 +0000"  >&lt;p&gt;Created cherry-pick for stable/boron here: &lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/53545/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/53545/&lt;/a&gt;&lt;/p&gt;</comment>
                            <comment id="58440" author="abhijit2511" created="Thu, 23 Mar 2017 15:40:57 +0000"  >&lt;p&gt;Raise to blocker - so that the patch can be added in SR 3.&lt;/p&gt;</comment>
                            <comment id="58441" author="ecelgp" created="Tue, 28 Mar 2017 19:19:28 +0000"  >&lt;p&gt;Changing milestone to SR4.&lt;/p&gt;</comment>
                            <comment id="58442" author="tomas.slusny@pantheon.tech" created="Mon, 3 Apr 2017 11:15:13 +0000"  >&lt;p&gt;Patch that fixes performance regression: &lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/53972/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/53972/&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;Can be cherry-picked to boron (requires also re-revert of original patch).&lt;/p&gt;</comment>
                            <comment id="58443" author="ecelgp" created="Mon, 3 Apr 2017 18:42:53 +0000"  >&lt;p&gt;Sounds good, we can merge all these changes after boron is unlocked.&lt;/p&gt;</comment>
                            <comment id="58444" author="jozef.bacigal@pantheon.tech" created="Thu, 13 Apr 2017 07:30:50 +0000"  >&lt;p&gt;Merged into Boron. Closing bug.&lt;/p&gt;</comment>
                    </comments>
                    <attachments>
                            <attachment id="14088" name="device-flow-registry-karaf.log" size="922053" author="miroslav.macko@pantheon.tech" created="Mon, 17 Oct 2016 16:36:04 +0000"/>
                    </attachments>
                <subtasks>
                    </subtasks>
                <customfields>
                                                                            <customfield id="customfield_11400" key="com.atlassian.jira.plugins.jira-development-integration-plugin:devsummary">
                        <customfieldname>Development</customfieldname>
                        <customfieldvalues>
                            
                        </customfieldvalues>
                    </customfield>
                                                                                                                        <customfield id="customfield_10208" key="com.atlassian.jira.plugin.system.customfieldtypes:textfield">
                        <customfieldname>External issue ID</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>6917</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_10201" key="com.atlassian.jira.plugin.system.customfieldtypes:url">
                        <customfieldname>External issue URL</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue><![CDATA[https://bugs.opendaylight.org/show_bug.cgi?id=6917]]></customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                <customfield id="customfield_10204" key="com.atlassian.jira.plugin.system.customfieldtypes:select">
                        <customfieldname>ODL SR Target Milestone</customfieldname>
                        <customfieldvalues>
                                <customfieldvalue key="10322"><![CDATA[Boron-4]]></customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                                                    <customfield id="customfield_10000" key="com.pyxis.greenhopper.jira:gh-lexo-rank">
                        <customfieldname>Rank</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>0|i033d3:</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                </customfields>
    </item>
</channel>
</rss>