fat flow enhancements - define exception for vrouter packets

Bug #1762452 reported by richard roberts
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R5.0
Fix Committed
High
Ashok Singh
Trunk
Fix Committed
High
Ashok Singh

Bug Description

following fat flow enhancements in https://bugs.launchpad.net/juniperopenstack/+bug/1739544, packets to vrouter (typically .1 and .2) may be absorbed via a wrong (fat flow).

Things like bgpaas get broken

We should have exception: there is no real benefit in having fat flows for flows to vrouter, while this is likely to introduce more problem than solve.

Jeba Paulaiyan (jebap)
tags: added: vrouter
information type: Proprietary → Public
Revision history for this message
Raghunandan Srinivasan (raghus) wrote :

will be addressed in R5.1

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

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

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

Reviewed: https://review.opencontrail.org/44695
Committed: http://github.com/Juniper/contrail-vrouter/commit/f397c93dfca211222ff43e06faa130f831d34c90
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit f397c93dfca211222ff43e06faa130f831d34c90
Author: Ashok Singh R <email address hidden>
Date: Tue Jul 17 16:24:33 2018 +0530

Sandesh definition to pass fat-flow exclude list

The exclude-list is passed per VMI and we pass both prefix-len and prefix.
The definition accommodates passing IPv6 addresses as well. IPv4 exclude IPs
are passed as 64 bit numbers with lower 32 bit representing IP and higher 32
bit representing prefix length.

Change-Id: Ia614a688ba3a2badeb28258d93d2c653b2e7e95c
Partial-Bug: #1762452

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

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

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

Reviewed: https://review.opencontrail.org/44768
Committed: http://github.com/Juniper/contrail-vrouter/commit/cbe23d6ea8e19c41ac3f4a2a87a1c10150bd9b66
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit cbe23d6ea8e19c41ac3f4a2a87a1c10150bd9b66
Author: Ashok Singh R <email address hidden>
Date: Tue Jul 17 16:24:33 2018 +0530

Sandesh definition to pass fat-flow exclude list

The exclude-list is passed per VMI and we pass both prefix-len and prefix.
The definition accommodates passing IPv6 addresses as well. IPv4 exclude IPs
are passed as 64 bit numbers with lower 32 bit representing IP and higher 32
bit representing prefix length.

Partial-Bug: #1762452
(cherry picked from commit f397c93dfca211222ff43e06faa130f831d34c90)

Change-Id: I5ea29409eec6fbaaad27b3e5ec257435464bdf97

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

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

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

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

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

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

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

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

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

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

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

Reviewed: https://review.opencontrail.org/45166
Committed: http://github.com/Juniper/contrail-controller/commit/f5cb125052f7e728710099b282a24f6c2e395747
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit f5cb125052f7e728710099b282a24f6c2e395747
Author: Ashok Singh R <email address hidden>
Date: Wed Aug 1 16:48:32 2018 +0530

Define exclude list for fat-flows

Send list of IP addresses to be excluded from fat-flow config.
Also consult exclude list while setting up flows to conclude on
flow keys.

Update introspect to reflect the exclude list being sent.

Change-Id: I4dd9c6ece7894e548900fa2f6b1ad25a692a8b82
Partial-Bug: #1762452

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/45214
Committed: http://github.com/Juniper/contrail-vrouter/commit/4d23b0d7ba2d32f46ddbe9bd9a04b36fe5c2d8bd
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 4d23b0d7ba2d32f46ddbe9bd9a04b36fe5c2d8bd
Author: Anand Narayanan Rao <email address hidden>
Date: Thu Aug 2 21:59:22 2018 +0530

vrouter changes to support fat flow exclude list for ipv4 & ipv6.

Change-Id: I5dde76d178355f87859a961ea7e1a3db228d9668
Partial-Bug: #1762452

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/45295
Committed: http://github.com/Juniper/contrail-vrouter/commit/29bd76d0df166aa054b67419f6cbc5c3cfc1e234
Submitter: Mithun Mistry (<email address hidden>)
Branch: R5.0

commit 29bd76d0df166aa054b67419f6cbc5c3cfc1e234
Author: Anand Narayanan Rao <email address hidden>
Date: Mon Aug 6 01:13:31 2018 +0530

vrouter changes to support fat flow exclude list.

Change-Id: I6b808454ae5fd7692adedbbf4d43cbcbcab37d39
Partial-Bug: #1762452

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

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

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

Reviewed: https://review.opencontrail.org/45202
Committed: http://github.com/Juniper/contrail-controller/commit/e1f91e39b8ae4f2ca99bcee449af3ce4ada55660
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit e1f91e39b8ae4f2ca99bcee449af3ce4ada55660
Author: Ashok Singh R <email address hidden>
Date: Wed Aug 1 16:48:32 2018 +0530

Define exclude list for fat-flows

Send list of IP addresses to be excluded from fat-flow config.
Also consult exclude list while setting up flows to conclude on
flow keys.

Update introspect to reflect the exclude list being sent.

Change-Id: Iba7f62cb18ccb3a4d93215ec68400f93bdb514ed
Partial-Bug: #1762452
(cherry picked from commit f5cb125052f7e728710099b282a24f6c2e395747)

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.