On a setup with Fat flow config and traffic through SI, fat flows are not aged out

[Vedamurthy Joshi]

R3.0 2710 Ubuntu 14.04 Kilo multi-node setup

vn1 : 10.1.1.0/24
vn2: 20.1.1.0/24

Policy applied between vn1 and vn2 to have all traffic through SIs si1 , si2 (in-network)
Fat flow is applied on tcp port 22 on VMI 20.1.1.3

10.1.1.3(on nodek2) does a ssh to 20.1.1.3(nodek1)
gcore of agent will be on http://10.204.216.50:Docs/bugs/#

On nodek1, these flows are never aged out ...even when no traffic is rx/txed on this session

root@nodek1:~# flow -l
Flow table(size 269484032, entries 2105344)

Entries: Created 249926 Added 249926 Processed 249926 Used Overflow entries 0
(Created Flows/CPU: 18410 22011 16005 19271 10594 15547 3395 13282 28949 1985 6231 2800 17143 10604 10526 11833 976 0 5057 1 2924 8176 3618 6663 0 5200 5535 0 0 399 19 2772)(oflows 0)

Action:F=Forward, D=Drop N=NAT(S=SNAT, D=DNAT, Ps=SPAT, Pd=DPAT, L=Link Local Port)
 Other:K(nh)=Key_Nexthop, S(nh)=RPF_Nexthop
 Flags:E=Evicted, Ec=Evict Candidate, N=New Flow, M=Modified
TCP(r=reverse):S=SYN, F=FIN, R=RST, C=HalfClose, E=Established, D=Dead

    Index Source:Port/Destination:Port Proto(V)
-----------------------------------------------------------------------------------
   174396<=>940756 20.1.1.3:22 6 (3)
                         20.1.1.2:0
    (K(nh):44, Action:F, Flags:, TCP:, S(nh):44, Statistics:5/330 UdpSrcPort 55397

   274096<=>414772 10.1.1.3:0 6 (3->4)
                         20.1.1.3:22
    (K(nh):14, Action:F, Flags:, TCP:, S(nh):14, Statistics:0/0 UdpSrcPort 65382

   414772<=>274096 20.1.1.3:22 6 (3->4)
                         10.1.1.3:0
    (K(nh):44, Action:F, Flags:, TCP:, S(nh):44, Statistics:45/8328 UdpSrcPort 61505

   940756<=>174396 20.1.1.2:0 6 (3)
                         20.1.1.3:22
    (K(nh):44, Action:F, Flags:, TCP:, S(nh):7, Statistics:0/0 UdpSrcPort 61953

root@nodek1:~#

[Naveen N (naveenn)]

When there are multiple flow tables enabled, partition for flow processing would be chosen
based on source port and destination port, during processing of flow agent masks
source port or destination port for fat flow, due to this request to delete the flow from
aging context would be enqueued different partition of flow table, hence the flow never
gets deleted.

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/17406
Submitter: Naveen N (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/17531
Submitter: Naveen N (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/17531
Committed: http://github.org/Juniper/contrail-controller/commit/f568ba9a1c4cb30e7c652840db3e8685a271aaed
Submitter: Zuul
Branch: R3.0

commit f568ba9a1c4cb30e7c652840db3e8685a271aaed
Author: Naveen N <email address hidden>
Date: Fri Feb 19 12:21:01 2016 +0530

* Calculate fat flow protocol in packet processing context
Calculate port in packet processing context such that packet
gets enqueued to right partition
Store flow partition in stats collector context and use that
for enqueueing delete request.
Closes-bug:#1542268

Change-Id: I4cd4a1f250581dcfa09e9b23108e4a9736850488

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/17406
Submitter: Manish Singh (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/17406
Committed: http://github.org/Juniper/contrail-controller/commit/9c16a21701c72c4a7130d1c4b9f1a032f4b6a4b8
Submitter: Zuul
Branch: master

commit 9c16a21701c72c4a7130d1c4b9f1a032f4b6a4b8
Author: Naveen N <email address hidden>
Date: Fri Feb 19 12:21:01 2016 +0530

* Calculate fat flow protocol in packet processing context
Calculate port in packet processing context such that packet
gets enqueued to right partition
Store flow partition in stats collector context and use that
for enqueueing delete request.
Closes-bug:#1542268

Change-Id: I4cd4a1f250581dcfa09e9b23108e4a9736850488

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/18102
Submitter: Praveen K V (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/18102
Committed: http://github.org/Juniper/contrail-controller/commit/cf967a891874a313134fd3cb0c4fe020c9d8e89d
Submitter: Zuul
Branch: R3.0

commit cf967a891874a313134fd3cb0c4fe020c9d8e89d
Author: Praveen K V <email address hidden>
Date: Wed Mar 2 17:06:29 2016 +0530

Fix un-initialized variable in constructor

Change-Id: I0dfb57b9a8a7fd9bcf5ed17dca72ffc6fe8c6a96
Fixes-Bug: #1542268

[alok kumar (kalok)]

covered in test case test_fat_flow_with_service_chain.