Juniper Openstack

vrouter: 'hold' as action in flow table does not work

Bug #1425992 reported by Anand H. Krishnan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R1.1
Fix Committed
Medium
Anand H. Krishnan
R2.0
Fix Committed
Medium
Anand H. Krishnan
Juniper Openstack r2.01
R2.1
Fix Committed
Medium
Anand H. Krishnan
Juniper Openstack r2.1-fcs
Trunk
Fix Committed
Medium
Anand H. Krishnan

Bug Description

HOLD is an action that is allowed to be set in a flow entry. Agent can set HOLD as action and expect the first packet to be trapped. To hold packets in an entry there needs to be hold list (an array, rather). While this hold list is allocated when a new flow is created by kernel, the hold list is freed once all the cached packets are flushed. A subsequent 'HOLD' set needs to have the hold list allocated so that packets can be cached.

Tags: vrouter
Anand H. Krishnan (anandhk)
Changed in juniperopenstack:
milestone: r2.1-fcs → none
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/7876
Committed: http://github.org/Juniper/contrail-vrouter/commit/2c3ba0ade72d4b1fa359b24ebb86d9d1cfec8a76
Submitter: Zuul
Branch: R2.1

commit 2c3ba0ade72d4b1fa359b24ebb86d9d1cfec8a76
Author: Anand H. Krishnan <email address hidden>
Date: Thu Feb 26 21:11:16 2015 +0530

Allocate hold array when the action is set as 'hold' in the flow table entry

HOLD is an action that is allowed to be set in a flow entry. Agent can set HOLD
as action and expect the first packet to be trapped. To hold packets in an
entry there needs to be hold list (an array, rather). While this hold list is
allocated when a new flow is created by kernel, the hold list is freed once all
the cached packets are flushed. A subsequent 'HOLD' set needs to have the hold
list allocated so that packets can be cached.

Closes BUG: #1425992

Change-Id: Ic32a03f402278a351c72cb6a4f72bafdaad2149c

Hari Prasad Killi (haripk)
information type: Proprietary → Public
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : R2.0

Review in progress for https://review.opencontrail.org/8139
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/8139
Committed: http://github.org/Juniper/contrail-vrouter/commit/251eb2b6ad31fd739d8503573a5dbccfc7727cdf
Submitter: Zuul
Branch: R2.0

commit 251eb2b6ad31fd739d8503573a5dbccfc7727cdf
Author: Anand H. Krishnan <email address hidden>
Date: Thu Feb 26 21:11:16 2015 +0530

Allocate hold array when the action is set as 'hold' in the flow table entry

HOLD is an action that is allowed to be set in a flow entry. Agent can set HOLD
as action and expect the first packet to be trapped. To hold packets in an
entry there needs to be hold list (an array, rather). While this hold list is
allocated when a new flow is created by kernel, the hold list is freed once all
the cached packets are flushed. A subsequent 'HOLD' set needs to have the hold
list allocated so that packets can be cached.

Closes BUG: #1425992

Change-Id: Ic32a03f402278a351c72cb6a4f72bafdaad2149c

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

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

commit 451029813e533f1dced87825b102beaedab5d06a
Author: Anand H. Krishnan <email address hidden>
Date: Thu Feb 26 21:11:16 2015 +0530

Allocate hold array when the action is set as 'hold' in the flow table entry

HOLD is an action that is allowed to be set in a flow entry. Agent can set HOLD
as action and expect the first packet to be trapped. To hold packets in an
entry there needs to be hold list (an array, rather). While this hold list is
allocated when a new flow is created by kernel, the hold list is freed once all
the cached packets are flushed. A subsequent 'HOLD' set needs to have the hold
list allocated so that packets can be cached.

Closes BUG: #1425992

Change-Id: Ic32a03f402278a351c72cb6a4f72bafdaad2149c

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

Review in progress for https://review.opencontrail.org/8835
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/8835
Committed: http://github.org/Juniper/contrail-vrouter/commit/e5b7c919ee7f0db3e4ddb59dd12ab40eaab3e953
Submitter: Zuul
Branch: R1.10

commit e5b7c919ee7f0db3e4ddb59dd12ab40eaab3e953
Author: Anand H. Krishnan <email address hidden>
Date: Thu Feb 26 21:11:16 2015 +0530

Allocate hold array when the action is set as 'hold' in the flow table entry

HOLD is an action that is allowed to be set in a flow entry. Agent can set HOLD
as action and expect the first packet to be trapped. To hold packets in an
entry there needs to be hold list (an array, rather). While this hold list is
allocated when a new flow is created by kernel, the hold list is freed once all
the cached packets are flushed. A subsequent 'HOLD' set needs to have the hold
list allocated so that packets can be cached.

Closes BUG: #1425992

Change-Id: Ic32a03f402278a351c72cb6a4f72bafdaad2149c

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

Other bug subscribers

Remote bug watches

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