Hi Vishal, This improvement will make sure to push the group first and followed by flow (only if both the flow and group present the config ☺).
When the application programs both the flow and group at the same instance there is a chance that flow can reach the switch before group and now dependency will be enforced. Regards, Arun From: openflowplugin-dev-boun...@lists.opendaylight.org [mailto:openflowplugin-dev-boun...@lists.opendaylight.org] On Behalf Of Vishal Thapar Sent: Tuesday, July 17, 2018 8:50 AM To: Luis Gomez <ece...@gmail.com>; Aswin Suryanarayanan <asury...@redhat.com> Cc: openflowplugin-dev <openflowplugin-dev@lists.opendaylight.org> Subject: Re: [openflowplugin-dev] Flow addition failed - missing group Thanks Luis. I recalled discussions on this wasn't sure if it was already fixed or a WiP. Will add this as dependency to the Netvirt bug. On Mon, Jul 16, 2018 at 11:14 PM, Luis Gomez <ece...@gmail.com<mailto:ece...@gmail.com>> wrote: This is a known issue of current FRM, see ticket below: https://jira.opendaylight.org/browse/OPNFLWPLUG-991 AFAIK work is in progress but no merge yet. BR/Luis On Jul 16, 2018, at 9:43 AM, Vishal Thapar <vtha...@redhat.com<mailto:vtha...@redhat.com>> wrote: Greetings OFP Devs, We're seeing an issue in Netvirt CSIT. The failure has been root caused to a missing flow and from OVS logs [1] we can see the reason being flow is added before the group. 2018-07-15T03:42:58.872Z|09648|vconn|DBG|tcp:10.30.170.132:6653<http://10.30.170.132:6653/>: received: OFPT_FLOW_MOD (OF1.3) (xid=0x1142): ADD table:26 priority=5,ip,metadata=0x30d78/0xfffffe cookie:0x8000006 actions=set_field:0x186bc->tun_id,group:225006 2018-07-15T03:42:58.872Z|09649|connmgr|INFO|br-int<->tcp:10.30.170.132:6653<http://10.30.170.132:6653/>: sending OFPBAC_BAD_OUT_GROUP error reply to OFPT_FLOW_MOD message 2018-07-15T03:42:58.872Z|09650|vconn|DBG|tcp:10.30.170.132:6653<http://10.30.170.132:6653/>: sent (Success): OFPT_ERROR (OF1.3) (xid=0x1142): OFPBAC_BAD_OUT_GROUP OFPT_FLOW_MOD (OF1.3) (xid=0x1142): (***truncated to 64 bytes from 112***) 00000000 04 0e 00 70 00 00 11 42-00 00 00 00 08 00 00 06 |...p...B........| 00000010 00 00 00 00 00 00 00 00-1a 00 00 00 00 00 00 05 |................| 00000020 ff ff ff ff ff ff ff ff-ff ff ff ff 00 00 00 00 |................| 00000030 00 01 00 1e 80 00 05 10-00 00 00 00 00 03 0d 78 |...............x| 2018-07-15T03:42:58.872Z|09651|vconn|DBG|tcp:10.30.170.132:6653<http://10.30.170.132:6653/>: received: OFPT_GROUP_MOD (OF1.3) (xid=0x1143): ADD group_id=225006,type=all,bucket=actions=load:0x300->NXM_NX_REG6[],resubmit(,220) Wasn't there some feature/fix done in OFP to ensure groups were always added before a flow? Or was that only for resync use case? Am I misremembering a feature that was never implemented or is this a bug? Regards, Vishal. [1] https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-queens-upstream-stateful-snat-conntrack-oxygen/329/compute_2/ovs-vswitchd.log.gz _______________________________________________ openflowplugin-dev mailing list openflowplugin-dev@lists.opendaylight.org<mailto:openflowplugin-dev@lists.opendaylight.org> https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
_______________________________________________ openflowplugin-dev mailing list openflowplugin-dev@lists.opendaylight.org https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
