OpenContrail

Reverse flow ecmp nh index updates do not propagate to hardware

Bug #1792378 reported by Frik Botha
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R4.1
Fix Committed
Undecided
Pieter Malan
Juniper Openstack r4.1.2.0
R5.0
Invalid
High
Jeya ganesh babu J
Juniper Openstack r5.0.2
Trunk
Invalid
High
Jeya ganesh babu J
Juniper Openstack r5.1.0
OpenContrail
New
Undecided
Unassigned

Bug Description

The ecmp nh index for reverse flows are updated in vr_rflow_update_ecmp_index when a NH_SOURCE_MISMATCH condition occurs. This changes the reverse flow entry, but the change is never programmed to hardware.

Tags: vrouter
Jeba Paulaiyan (jebap)
tags: added: dpdk vrouter
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.1

Review in progress for https://review.opencontrail.org/46208
Submitter: Pieter Malan (<email address hidden>)

Revision history for this message
Jeya ganesh babu J (jjeya) wrote :

Specific to offload. Not valid for 5.0/Trunk.

tags: removed: dpdk
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/46208
Committed: http://github.com/Juniper/contrail-vrouter/commit/8a24cd6873e62c9f68a19aefa17f1a1ad7a898c4
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 8a24cd6873e62c9f68a19aefa17f1a1ad7a898c4
Author: Pieter Malan <email address hidden>
Date: Fri Sep 14 15:24:56 2018 +0200

Offload ECMP next hop index updates

vr_rflow_update_ecmp_index updates the ECMP next hop index for reverse
flows. Add offload hook such that this change reflects in hardware as
well.

Change-Id: I62a28ae276ef59557dfdf8e8cc884255150723ff
Closes-Bug: #1792378
Signed-off-by: Pieter Malan <email address hidden>

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.