Fat flow support

Bug #1518234 reported by Hari Prasad Killi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Fix Committed
High
Naveen N
Trunk
Fix Committed
High
Naveen N

Bug Description

For configured protocols, flows created in the vrouter should ignore the port numbers and create a single forward and reverse flow pair for all traffic from a VMI for that protocol.

This could be configured on VM interfaces where corresponding servers are running.

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/15302
Submitter: Hari Prasad Killi (<email address hidden>)

Changed in juniperopenstack:
assignee: Manish Singh (manishs) → Naveen N (naveenn)
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/15302
Committed: http://github.org/Juniper/contrail-controller/commit/6cdb6f15977e5768eb31d52dae8b68ef88e0c8b7
Submitter: Zuul
Branch: R2.20

commit 6cdb6f15977e5768eb31d52dae8b68ef88e0c8b7
Author: Hari <email address hidden>
Date: Fri Nov 20 14:42:44 2015 +0530

Schema update to configure protocols on VMI for which fat flows are desired.

Change-Id: I21399300a5cafb09402d7d0e710315a2bd48df30
related-bug: 1518234

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

Review in progress for https://review.opencontrail.org/15374
Submitter: Hari Prasad Killi (<email address hidden>)

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

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

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

Reviewed: https://review.opencontrail.org/15374
Committed: http://github.org/Juniper/contrail-controller/commit/53a2ccf6f439797a399143666272315f2014b3de
Submitter: Zuul
Branch: R2.20

commit 53a2ccf6f439797a399143666272315f2014b3de
Author: Hari <email address hidden>
Date: Tue Nov 24 14:40:34 2015 +0530

Change fat-flow schema to take port and protocol.

Change-Id: I122e2cf7af39d996f967adf319cfb1e8686b9050
related-bug: 1518234

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

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

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

Review in progress for https://review.opencontrail.org/15414
Submitter: Hari Prasad Killi (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/15456
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/15414
Committed: http://github.org/Juniper/contrail-controller/commit/6722f562d375771debe8b4705b247b38155494b9
Submitter: Zuul
Branch: R2.20

commit 6722f562d375771debe8b4705b247b38155494b9
Author: Naveen N <email address hidden>
Date: Wed Nov 25 12:16:05 2015 +0530

* Agent changes to support fat flow
* Maintain a list of protocol and port specified as fat flow
Ignore flow matching protocol + port combination.
Closes-bug:#1518234

Conflicts:
 src/vnsw/agent/oper/test/test_intf.cc
 src/vnsw/agent/oper/vm_interface.cc

Change-Id: Iaa9b14f74276ca011906bff89b03841049acd4f4

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

Reviewed: https://review.opencontrail.org/15456
Committed: http://github.org/Juniper/contrail-vrouter/commit/1cc6cc900c6e01f9aa742da7a14d65d12649a803
Submitter: Zuul
Branch: R2.20

commit 1cc6cc900c6e01f9aa742da7a14d65d12649a803
Author: Anand H. Krishnan <email address hidden>
Date: Fri Nov 27 16:43:11 2015 +0530

Combine streams of configured L4 protocol, port into a single stream

Change-Id: Iaf6b5ae27fec9d58e60852aee267aacb875a773f
Partial-BUG: 1518234

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

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

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

Reviewed: https://review.opencontrail.org/15466
Committed: http://github.org/Juniper/contrail-controller/commit/c9821d496ca0de4f054a022bb606f6134da2fb6b
Submitter: Zuul
Branch: R2.20

commit c9821d496ca0de4f054a022bb606f6134da2fb6b
Author: Naveen N <email address hidden>
Date: Sat Nov 28 22:25:02 2015 +0530

Encode fat flow config in vrouter ksync.

Also, add introspect for fat flow config.

Change-Id: I9261c39a034b884b273619ee3b9bd55a2d5001b3
closes-bug: 1518234

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

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

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

Review in progress for https://review.opencontrail.org/15521
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/15469
Committed: http://github.org/Juniper/contrail-vrouter/commit/9b547da068bdefacaafa8b310b895d295f736912
Submitter: Zuul
Branch: R2.20

commit 9b547da068bdefacaafa8b310b895d295f736912
Author: Anand H. Krishnan <email address hidden>
Date: Mon Nov 30 10:29:30 2015 +0530

Handle FatFlow configuration absence in requests

If FatFlow configuration is present in vif, but no configuration is
present in the interface request, all the existing configuration in
the interface should be removed.

Change-Id: Ieb74ed5a3955773fd39d70727cd5f095ad266c0a
Partial-BUG: 1518234

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

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

commit a67adbda1ab6329828657f6fa457b3fdce276d37
Author: Anand H. Krishnan <email address hidden>
Date: Tue Dec 1 16:40:10 2015 +0530

FatFlow: wrong port calculation

The macro to extract port from proto_port combination uses logical
and instead of binary and, resulting in wrong port calulcation.

Change-Id: Icc4c7e6038a6e15ad21f17fa03bb377b724372ac
Partial-BUG: 1518234

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

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

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

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

commit d38f87d1a1977424ea28703893f321ca956adf3b
Author: Anand H. Krishnan <email address hidden>
Date: Fri Dec 4 11:45:48 2015 +0530

FatFlow - BUG fixes

A vif get/dump overwrites the same request memory for every protocol
that is configured for FatFlow, resulting in some configured values
not showing up in the output.

TCP state machine shouldn't be applied for FatFlows since multiple
flows can map to the same entry

Change-Id: Ie67d7b049bd2a4de15041380c9f757dc4627f250
Partial-BUG: 1518234

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

Review in progress for https://review.opencontrail.org/15671
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/15671
Committed: http://github.org/Juniper/contrail-vrouter/commit/6d01d7381c4b753c454e88abdd7f44639f970770
Submitter: Zuul
Branch: R2.20

commit 6d01d7381c4b753c454e88abdd7f44639f970770
Author: Anand H. Krishnan <email address hidden>
Date: Tue Dec 8 12:27:59 2015 +0530

For non tcp/udp/sctp FatFlows, print proto:*

'vif' utility displays protocol:port for configured FatFlows. For
not TCP/UDP/SCTP protocols, vif will print protocol:*, indicating
that all ports corresponding to the protocol will undergo FatFlow.

Change-Id: I74161e905cf010dd68f049cf28a63c2996aac589
Partial-BUG: 1518234

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

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

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

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

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

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

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

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

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

Review in progress for https://review.opencontrail.org/15826
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/15820
Committed: http://github.org/Juniper/contrail-vrouter/commit/542d299282c4caf1c4bfe553819cf4f71eb3cb39
Submitter: Zuul
Branch: master

commit 542d299282c4caf1c4bfe553819cf4f71eb3cb39
Author: Anand H. Krishnan <email address hidden>
Date: Fri Nov 27 16:43:11 2015 +0530

Combine streams of configured L4 protocol, port into a single stream

Change-Id: Iaf6b5ae27fec9d58e60852aee267aacb875a773f
Partial-BUG: 1518234

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

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

commit c8bffc8f294bf5ccba74475f94c63d2646ef4c2e
Author: Anand H. Krishnan <email address hidden>
Date: Mon Nov 30 10:29:30 2015 +0530

Handle FatFlow configuration absence in requests

If FatFlow configuration is present in vif, but no configuration is
present in the interface request, all the existing configuration in
the interface should be removed.

Change-Id: Ieb74ed5a3955773fd39d70727cd5f095ad266c0a
Partial-BUG: 1518234

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

Reviewed: https://review.opencontrail.org/15823
Committed: http://github.org/Juniper/contrail-vrouter/commit/2b59f288c324c2ed132604bb5e21ad0afe45b4d3
Submitter: Zuul
Branch: master

commit 2b59f288c324c2ed132604bb5e21ad0afe45b4d3
Author: Anand H. Krishnan <email address hidden>
Date: Tue Dec 1 16:40:10 2015 +0530

FatFlow: wrong port calculation

The macro to extract port from proto_port combination uses logical
and instead of binary and, resulting in wrong port calulcation.

Change-Id: Icc4c7e6038a6e15ad21f17fa03bb377b724372ac
Partial-BUG: 1518234

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

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

commit a87fe2ce5f01870a5e1a6e0c7aa03654c3b3d0a6
Author: Anand H. Krishnan <email address hidden>
Date: Fri Dec 4 11:45:48 2015 +0530

FatFlow - BUG fixes

A vif get/dump overwrites the same request memory for every protocol
that is configured for FatFlow, resulting in some configured values
not showing up in the output.

TCP state machine shouldn't be applied for FatFlows since multiple
flows can map to the same entry

Change-Id: Ie67d7b049bd2a4de15041380c9f757dc4627f250
Partial-BUG: 1518234

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

Reviewed: https://review.opencontrail.org/15826
Committed: http://github.org/Juniper/contrail-vrouter/commit/3b543a83253258ba85b65995ed78e958a4c3b085
Submitter: Zuul
Branch: master

commit 3b543a83253258ba85b65995ed78e958a4c3b085
Author: Anand H. Krishnan <email address hidden>
Date: Tue Dec 8 12:27:59 2015 +0530

For non tcp/udp/sctp FatFlows, print proto:*

'vif' utility displays protocol:port for configured FatFlows. For
not TCP/UDP/SCTP protocols, vif will print protocol:*, indicating
that all ports corresponding to the protocol will undergo FatFlow.

Change-Id: I74161e905cf010dd68f049cf28a63c2996aac589
Partial-BUG: 1518234

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

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

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

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

commit 9c10d20b1d55687cec007128831d60b9c3ab929d
Author: Naveen N <email address hidden>
Date: Mon Dec 28 08:56:55 2015 +0530

Add fat flow support
* Maintain a list of protocol and port specified as fat flow
Ignore flow matching protocol + port combination.
Fixes-Bug: #1518234,#1522285

Change-Id: I6e98b68467f36f1a91227a811f7860c71aaff411

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

Review in progress for https://review.opencontrail.org/16814
Submitter: Hari Prasad Killi (<email address hidden>)

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

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

commit caa89c1d595d34a6e9e4228b640707d72d1b8e78
Author: Naveen N <email address hidden>
Date: Sat Nov 28 22:25:02 2015 +0530

Encode fat flow config in vrouter ksync.

Also, add introspect for fat flow config.

(cherry picked from commit c9821d496ca0de4f054a022bb606f6134da2fb6b)

Conflicts:
 src/vnsw/agent/oper/agent.sandesh

Change-Id: I9261c39a034b884b273619ee3b9bd55a2d5001b3
closes-bug: 1518234

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.