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-82 OpenFlowPlugin <p>When I pushed a particular flow entry using "addMDFlow openflow:1 f4" and request the flow using readConfigurationData() API, readOperationalData() API both the Flows are not matching. The fields like Flow id/values, FlowKey, _barrier=false, _cookieMask=10, _flowName=Bar etc. are either malformed or not present. This bug might affect <a href="https://jira.opendaylight.org/browse/VTN-2" title="Computation of logical ports names fail when adding more than once ODL controller to VTN Coordinator" class="issue-link" data-issue-key="VTN-2"><del>VTN-2</del></a> as suggested by Michal Rehak.</p> <p>I am using latest openflowplugin and OVSK switch with OF1.3</p> <p>getAllControllerFlows: tableId= 4<br/> getAllControllerFlows: flow= Flow [_id=Uri <span class="error">&#91;_value=127&#93;</span>, _key=FlowKey [_id=Uri <span class="error">&#91;_value=127&#93;</span>], _barrier=false, _cookieMask=10, _flags=FlowModFlags <span class="error">&#91;_cHECKOVERLAP=false, _rESETCOUNTS=false, _nOPKTCOUNTS=false, _nOBYTCOUNTS=false, _sENDFLOWREM=true&#93;</span>, _flowName=Bar, _installHw=false, _instructions=Instructions [_instruction=[Instruction [_key=InstructionKey [], _instruction=ApplyActionsCase [_applyActions=ApplyActions [_action=[Action [_key=ActionKey [], _action=DropActionCase [_dropAction=DropAction [augmentation=[]], augmentation=[]], augmentation=[]]], augmentation=[]], augmentation=[]], augmentation=[]]], augmentation=[]], _match=Match [_ethernetMatch=EthernetMatch [_ethernetDestination=EthernetDestination [_address=MacAddress <span class="error">&#91;_value=ff:ff:ff:ff:ff:ff&#93;</span>, _mask=MacAddress <span class="error">&#91;_value=ff:ff:ff:00:00:00&#93;</span>, augmentation=[]], _ethernetSource=EthernetSource [_address=MacAddress <span class="error">&#91;_value=00:00:00:00:23:ae&#93;</span>, _mask=MacAddress <span class="error">&#91;_value=ff:ff:00:00:00:00&#93;</span>, augmentation=[]], _ethernetType=EthernetType [_type=EtherType <span class="error">&#91;_value=2048&#93;</span>, augmentation=[]], augmentation=[]], augmentation=[]], _strict=false, _cookie=10, _hardTimeout=0, _idleTimeout=0, _priority=2, _tableId=4, augmentation=[]]</p> <p>getAllSwitchFlows: tableId= 4<br/> getAllSwitchFlows: flow= Flow [_id=Uri <a href="#UF$TABLE*4*2" target="_blank" rel="noopener">_value=#UF$TABLE*4*2</a>, _key=FlowKey <a href="#UF$TABLE*4*2]" target="_blank" rel="noopener">_id=Uri [_value=#UF$TABLE*4*2]</a>, augmentation=[FlowStatisticsData [_flowStatistics=FlowStatistics [_byteCount=Counter64 <span class="error">&#91;_value=0&#93;</span>, _packetCount=Counter64 <span class="error">&#91;_value=0&#93;</span>, _duration=Duration [_nanosecond=Counter32 <span class="error">&#91;_value=707000000&#93;</span>, _second=Counter32 <span class="error">&#91;_value=722&#93;</span>, augmentation=[]], _flags=FlowModFlags <span class="error">&#91;_cHECKOVERLAP=false, _rESETCOUNTS=false, _nOPKTCOUNTS=false, _nOBYTCOUNTS=false, _sENDFLOWREM=true&#93;</span>, _instructions=Instructions [_instruction=[], augmentation=[]], _match=Match [_ethernetMatch=EthernetMatch [_ethernetDestination=EthernetDestination [_address=MacAddress <span class="error">&#91;_value=FF:FF:FF:00:00:00&#93;</span>, augmentation=[]], _ethernetSource=EthernetSource [_address=MacAddress <span class="error">&#91;_value=00:00:00:00:00:00&#93;</span>, augmentation=[]], _ethernetType=EthernetType [_type=EtherType <span class="error">&#91;_value=2048&#93;</span>, augmentation=[]], augmentation=[]], augmentation=[]], _cookie=10, _hardTimeout=0, _idleTimeout=0, _priority=2, _tableId=4, augmentation=[]]]]]</p> <p>Operating System: Linux<br/> Platform: PC</p> OPNFLWPLUG-82

Flow Entries from Switch and Controller are not same. Might affect Bug375

Improvement Resolved Done Anil Vishnoi Praveen Darshanam Wed, 12 Mar 2014 15:49:51 +0000 Mon, 27 Sep 2021 09:01:06 +0000 Tue, 16 Aug 2016 12:41:56 +0000 General 0 4 <p>Sorry, it is <a href="https://jira.opendaylight.org/browse/OPNFLWPLUG-56" title="Existing flows not cleared when Controller reconnects to the switch" class="issue-link" data-issue-key="OPNFLWPLUG-56"><del>OPNFLWPLUG-56</del></a> not 385</p> <p>Investigating</p> <p>For better flow identification we should use the cookie field.</p> <p>Hi Praveen,<br/> Contents of the flow stored in the operational data store seems fine to me. Here is what actually happening.<br/> You are installing flow with <br/> source _mac = ff:ff:ff:ff:ff:ff src_mac_mask = ff:ff:ff:00:00:00<br/> dest_mac = 00:00:00:00:23:ae dest_mac_mask = ff:ff:00:00:00:00</p> <p>When you install flow with above mac address and their respective mask values, switch installs flow with following src and dst mac.</p> <p>src_mac = (ff:ff:ff:ff:ff:ff &amp; ff:ff:ff:00:00:00 = ff:ff:ff:00:00:00)<br/> dst_mac = (00:00:00:00:23:ae &amp; ff:ff:00:00:00:00 = 00:00:00:00:00:00)</p> <p>So in actual flow rule gets install to the switch has src_mac = ff:ff:ff:00:00:00 &amp; dst_mac = 00:00:00:00:00:00 because of the mask, and this is what operational data store flow reports. When controller fetch flow details from switch, switch don't send cookie mask value with the details, and that why its missing.</p> <p>FlowName is something thats being used for controller bookkeeping purpose and never goes down to the switch, so its also not there in the details we fetch from switch. </p> <p>When we fetch flow statistics from switch, we compare each flow retrieved from the switch with the flows stored in config data store to find the matching flow, and if we find the matching flow, we augment the statistics respectively in operational data store under the matching flow key. If we don't find any matching flow in config data store, we consider those flow as unaccounted flows and we augment details of those flows to operational data store using specific key value. The key (_value=#UF$TABLE*4*2) you see in the operational data store flow is a key of an unaccounted flow. You can decode this key in following way<br/> #UF - Unaccounted flow<br/> $TABLE*4 &#8211; Table = 4<br/> *2, this is second unaccounted flow in this table on switch.</p> <p>Matching between the flow fetched from switch and flows stored in config data store is done through custom match functions ( comparison based on each element of the flow construct ), because flow is the only construct in openflow spec that don't have unique ID associated with it. If we can find some alternate mechanism of providing unique id to the flow, we can avoid this custom matching and directly augment the flow stats to the flow associated with that unique key in the operational data store.</p> <p>Now the question is why controller is considering this flow as unaccounted flow given that its respective flow is already there in the config data store?<br/> I feel this is a bug in the custom matching functions where we compare ethernet address for match, we probably are ignoring the mask based comparison. The reason this custom comparison gets more tricky is because when we fetch flow stats from switch, it doesn't provide the mask values, it just provide the final executed details of flow, rather then what user sent to it.</p> <p>For short term i will try to fix it in the existing implementation and will push patch sometime next week. For the long term solution we are looking at alternate mechanism of assigning unique identifier to the flow using "cookies". </p> <p>Thanks<br/> Anil</p> <p>Closing bug since the main issue here was the alien flow id which was fixed in <a href="https://jira.opendaylight.org/browse/OPNFLWPLUG-722" title="Some Flows not found in DeviceFlowRegistry" class="issue-link" data-issue-key="OPNFLWPLUG-722"><del>OPNFLWPLUG-722</del></a> and <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>.</p> Development External issue ID 521 External issue URL Rank 0|i02yyf: