[OPNFLWPLUG-432] [Lithium redesign] some TTL flows (Dec_TTL, Set_MPLS_TTL, Dec_MPLS_TTL) causing NPE and not sent to switch Created: 05/May/15 Updated: 27/Sep/21 Resolved: 26/May/15 |
|
| Status: | Resolved |
| Project: | OpenFlowPlugin |
| Component/s: | General |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Jamo Luhrsen | Assignee: | Jamo Luhrsen |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| Issue Links: |
|
||||||||
| External issue ID: | 3129 | ||||||||
| Description |
|
flow services CI tests have at least three failures because of this: these tests pass in the helium based tests: The short of it is that pushing a flow for TTL decrement, MPLS TTL decrement, or Set MPLS TTL will result in a similar 2015-05-05 16:03:17,474 | ERROR | lt-dispatcher-25 | DataTreeChangeListenerActor | 179 - org.opendaylight.controller.sal-distributed-datastore - 1.2.0.SNAPSHOT | Error notifying listener org.opendaylight.controller.md.sal.binding.impl.BindingDOMDataTreeChangeListenerAdapter@1045caf5 The flow is also not sent to the switch (as I guess could be expected) The exceptions seem similar, but a separate comment for each flow mod XML and exception trace is added. |
| Comments |
| Comment by Jamo Luhrsen [ 05/May/15 ] |
|
This is the dec-nw-ttl XML used to push the flow to config: <flow xmlns="urn:opendaylight:flow:inventory"><instructions><instruction><order>0</order><apply-actions><action><order>0</order><dec-nw-ttl/></action></apply-actions></instruction></instructions><match><ethernet-match><ethernet-type><type>0x800</type></ethernet-type><ethernet-source><address>00:00:00:01:23:ae</address></ethernet-source><ethernet-destination><address>ff:ff:ff:ff:ff:ff</address></ethernet-destination></ethernet-match><ipv4-source>10.1.2.0/24</ipv4-source><ipv4-destination>40.4.0.0/16</ipv4-destination></match><table_id>3</table_id><id>305</id></flow> This is the resulting Exception in karaf.log 2015-05-05 15:47:34,350 | ERROR | lt-dispatcher-18 | DataTreeChangeListenerActor | 179 - org.opendaylight.controller.sal-distributed-datastore - 1.2.0.SNAPSHOT | Error notifying listener org.opendaylight.controller.md.sal.binding.impl.BindingDOMDataTreeChangeListenerAdapter@1045caf5 |
| Comment by Jamo Luhrsen [ 05/May/15 ] |
|
The flow xml for set-mpls-ttl-action: <flow xmlns="urn:opendaylight:flow:inventory"><instructions><instruction><order>0</order><apply-actions><action><order>0</order><set-mpls-ttl-action><mpls-ttl>1</mpls-ttl></set-mpls-ttl-action></action></apply-actions></instruction></instructions><match><ethernet-match><ethernet-type><type>0x800</type></ethernet-type><ethernet-source><address>00:00:00:01:23:ae</address></ethernet-source><ethernet-destination><address>ff:ff:ff:ff:ff:ff</address></ethernet-destination></ethernet-match><ipv4-source>10.1.2.0/24</ipv4-source><ipv4-destination>40.4.0.0/16</ipv4-destination></match><table_id>4</table_id><id>505</id></flow> The resulting Exception: 2015-05-05 15:52:54,253 | ERROR | entLoopGroup-6-6 | ExecutionList | 61 - com.google.guava - 18.0.0 | RuntimeException while executing runnable com.google.common.util.concurrent.Futures$6@2c31a8f8 with executor INSTANCE |
| Comment by Jamo Luhrsen [ 05/May/15 ] |
|
The flow xml for dec-mpls-ttl: <flow xmlns="urn:opendaylight:flow:inventory"><instructions><instruction><order>0</order><apply-actions><action><order>0</order><dec-mpls-ttl /></action></apply-actions></instruction></instructions><match><ethernet-match><ethernet-type><type>0x800</type></ethernet-type><ethernet-source><address>00:00:00:01:23:ae</address></ethernet-source><ethernet-destination><address>ff:ff:ff:ff:ff:ff</address></ethernet-destination></ethernet-match><ipv4-source>10.1.2.0/24</ipv4-source><ipv4-destination>40.4.0.0/16</ipv4-destination></match><table_id>2</table_id><id>1001</id></flow> The resulting Exception: 2015-05-05 15:57:25,482 | ERROR | entLoopGroup-6-6 | ExecutionList | 61 - com.google.guava - 18.0.0 | RuntimeException while executing runnable com.google.common.util.concurrent.Futures$6@300206ec with executor INSTANCE |
| Comment by Michal Rehak [ 06/May/15 ] |
|
Hi Jamo, |
| Comment by Michal Rehak [ 06/May/15 ] |
| Comment by Michal Rehak [ 06/May/15 ] |
|
merged, please retest |
| Comment by Anil Vishnoi [ 06/May/15 ] |
|
I also hit the same issue, and i believe it's happening if user don't set the cookie value and cookie value is optional for flow. |
| Comment by Martin Bobak [ 26/May/15 ] |
|
This bug is related to something that doesn't exist in code base anymore. Please retest and close this bug. |
| Comment by Jamo Luhrsen [ 26/May/15 ] |
|
looks like the general behavior is now different as well. before, we did not accept a flow mod without a priority. Now, we will take it and assign a default priority of 32768. as long as this is by design, no problem with this bug marked as FIXED. I have an email to openflowplugin-dev asking about this. |