Flow records : enhancements to UVE

Bug #1493206 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
Medium
Ashok Singh
Trunk
Fix Committed
Medium
Ashok Singh

Bug Description

Add the following to UVE:
- Drop reason
- TCP bits
- agent count of flows

Also, when action == log, always send the flow records (irrespective of flow sampling) and send them with highest priority.

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/14318
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/14363
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/14318
Committed: http://github.org/Juniper/contrail-controller/commit/0477a93ae933302bac90d58d298e84c5c7ab40b5
Submitter: Zuul
Branch: R2.20

commit 0477a93ae933302bac90d58d298e84c5c7ab40b5
Author: Ashok Singh <email address hidden>
Date: Thu Oct 8 07:17:01 2015 -0700

Send TCP flags in Flow messages exported to collector.

Add UT and enable the previous disabled UT.

Change-Id: I91b8274d24cf8d04f974b087377d7a1176d5cec3
Closes-Bug: #1493206

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

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

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

Review in progress for https://review.opencontrail.org/14543
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/14543
Committed: http://github.org/Juniper/contrail-controller/commit/82f174e5f9d3f6eb808da3528401c3bf4ed3562f
Submitter: Zuul
Branch: master

commit 82f174e5f9d3f6eb808da3528401c3bf4ed3562f
Author: Ashok Singh <email address hidden>
Date: Tue Oct 20 02:52:36 2015 -0700

Send TCP flags in Flow messages exported to collector.

Add UT and enable the previous disabled UT.

Change-Id: I5b40b0543a3740e5a9dc7c3eefba4cf84eba6090
Closes-Bug: #1493206

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

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

commit 84c77e73772ea4540141bda14510eafd011189d5
Author: Ashok Singh <email address hidden>
Date: Sun Oct 25 06:36:47 2015 -0700

Define "log" and "alert" as a separate actions in schema for policy rules.

This is required so that log/alert can be specified in addition to other
actions like pass and deny. Also remove the currently defined log/alert/drop/reject
as possible action strings under simple-action.

Change-Id: Ic0dfe0057be69a9af24788bfa54b4d93fd3f3a15
Partial-Bug: #1493206

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

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

commit 3ddb6b492065876af8ed3d75ef2514656f03ebbf
Author: Ashok Singh <email address hidden>
Date: Tue Oct 27 02:54:19 2015 -0700

Agent changes to handle "log" as a possible action in ACL rules.

Agent config handling for ACL rules should handle "log" and "alert" as possible actions and store them in AclEntrySpec and AclEntry objects. These new action values will be used during flow policy handling and can be possible actions for flow.
Removed "DROP" and "REJECT" action definitions. Also when a flow was made as short flow, we were marking the action as DROP. Changed this to use "DENY"
Also added UT to verify log action configured in ACL rule gets reflected in flow Action.

Change-Id: I1cae90a14e96921830c0fdb2c43b22fbd2fc9320
Closes-Bug: #1493206

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

Review in progress for https://review.opencontrail.org/14693
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/14693
Committed: http://github.org/Juniper/contrail-controller/commit/49e13e70ddbb246ec390d8fecb7f1e32e050d9af
Submitter: Zuul
Branch: master

commit 49e13e70ddbb246ec390d8fecb7f1e32e050d9af
Author: Ashok Singh <email address hidden>
Date: Sun Oct 25 06:36:47 2015 -0700

Define "log" and "alert" as a separate actions in schema for policy rules.

This is required so that log/alert can be specified in addition to other
actions like pass and deny. Also remove the currently defined log/alert/drop/reject
as possible action strings under simple-action.

Partial-Bug: #1493206
(cherry picked from commit 84c77e73772ea4540141bda14510eafd011189d5)

Change-Id: I9f7ed7a15c033b9b70a0c99f94805a6cef4a824b

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

Review in progress for https://review.opencontrail.org/14719
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/14719
Committed: http://github.org/Juniper/contrail-controller/commit/c9fb0454d612b7db2a1bc6c75f92ec6c80d76ce2
Submitter: Zuul
Branch: master

commit c9fb0454d612b7db2a1bc6c75f92ec6c80d76ce2
Author: Ashok Singh <email address hidden>
Date: Thu Oct 29 20:56:55 2015 -0700

Agent changes to handle "log" as a possible action in ACL rules.

Agent config handling for ACL rules should handle "log" and "alert" as possible actions and store them in AclEntrySpec
and AclEntry objects. These new action values will be used during flow policy handling and can be possible actions for flow.
Removed "DROP" and "REJECT" action definitions. Also when a flow was made as short flow, we were marking the action as DROP.
Changed this to use "DENY"
Also added UT to verify log action configured in ACL rule gets reflected in flow Action.

Change-Id: I8b95f324e9dbc2e6b33c5b1a228b07496f908cf3
Closes-Bug: #1493206

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.