evicted flows in vrouter ideally should get removed on agent restart

Bug #1524622 reported by Vedamurthy Joshi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Fix Committed
Medium
Anand H. Krishnan
Trunk
Fix Committed
Medium
Anand H. Krishnan

Bug Description

R2.20 Build 112 Ubuntu 14.04 Juno multi-node

TCP evicted flows ideally should get flushed on agent restart
Functionally, it does not have any impact, but it is more of a consistency issue

root@nodek1:/var/log/contrail# flow -l --show-evicted | wc -l
18669
root@nodek1:/var/log/contrail# service contrail-vrouter-agent restart
contrail-vrouter-agent: stopped
contrail-vrouter-agent: started
root@nodek1:/var/log/contrail#
root@nodek1:/var/log/contrail#
root@nodek1:/var/log/contrail#
root@nodek1:/var/log/contrail# flow -l --show-evicted | wc -l
18648
root@nodek1:/var/log/contrail#

Tags: vrouter
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/15738
Submitter: Anand H. Krishnan (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/15738
Committed: http://github.org/Juniper/contrail-vrouter/commit/374a0b2a4b01a34db4de073be853895b4e8ab78f
Submitter: Zuul
Branch: R2.20

commit 374a0b2a4b01a34db4de073be853895b4e8ab78f
Author: Anand H. Krishnan <email address hidden>
Date: Thu Dec 10 14:40:44 2015 +0530

Reset evicted flows too at the time of soft-reset

Evicted flows, while active in the flow table, is not active in the
database used by entry allocator. The API that is used to get entries
that need to be reset returns only active entries in the allocator
database, thus excluding the evicted flows from the process of reset.
Introduce a new API that will return all entries that are active
in the flow table and use that api from the reset function.

Change-Id: I41dc242dcc308768049737c644f7257f0b581ad7
Closes-BUG: 1524622

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/16666
Submitter: Divakar Dharanalakota (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/16666
Committed: http://github.org/Juniper/contrail-vrouter/commit/6c0a802bf1b0c6de379abca3b1039fb07ca81dc3
Submitter: Zuul
Branch: master

commit 6c0a802bf1b0c6de379abca3b1039fb07ca81dc3
Author: Anand H. Krishnan <email address hidden>
Date: Thu Dec 10 14:40:44 2015 +0530

Reset evicted flows too at the time of soft-reset

Evicted flows, while active in the flow table, is not active in the
database used by entry allocator. The API that is used to get entries
that need to be reset returns only active entries in the allocator
database, thus excluding the evicted flows from the process of reset.
Introduce a new API that will return all entries that are active
in the flow table and use that api from the reset function.

Change-Id: I41dc242dcc308768049737c644f7257f0b581ad7
Closes-BUG: 1524622

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

Other bug subscribers

Remote bug watches

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