port based mirroring broken in 3.0.3-b4

Bug #1636638 reported by amit surana
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Fix Committed
High
Anand H. Krishnan
R3.0.3.x
Fix Committed
High
Anand H. Krishnan
Trunk
Invalid
High
Anand H. Krishnan

Bug Description

Even though mirroring is configured to mirror both rx/tx packets, only tx packets are being mirrored. Mirror destination is in a different VN from the source/destination VMI. Same is seen for sub-interface mirroring also.

VN level mirroring (done via monitor->debug) is working correctly and mirroring both directions.

Tags: vrouter
amit surana (asurana-t)
description: updated
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0.3.x

Review in progress for https://review.opencontrail.org/25410
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/25410
Committed: http://github.org/Juniper/contrail-vrouter/commit/aba039463912c7d32a4554f418632df4f8bff0a6
Submitter: Zuul
Branch: R3.0.3.x

commit aba039463912c7d32a4554f418632df4f8bff0a6
Author: Anand H. Krishnan <email address hidden>
Date: Thu Oct 27 10:22:35 2016 +0530

For port mirroring, flow index should not be set

During transmit port mirror, we seem to be using the mirror values
from the flow of the original packet. This is not a right behavior,
since port mirroring shouldn't be depending on flow based mirror
values.

Hence reset the flow index for the port mirrored packet

Change-Id: I1aff969b08a2f247ccf1fc636d6689350e819f29
Closes-Bug: #1636638

Changed in juniperopenstack:
assignee: Hari Prasad Killi (haripk) → Anand H. Krishnan (anandhk)
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

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

commit 7f5e8b1a70277df6b6f777a9dbd79034ba0365b5
Author: Anand H. Krishnan <email address hidden>
Date: Thu Oct 27 10:22:35 2016 +0530

For port mirroring, flow index should not be set

During transmit port mirror, we seem to be using the mirror values
from the flow of the original packet. This is not a right behavior,
since port mirroring shouldn't be depending on flow based mirror
values.

Hence reset the flow index for the port mirrored packet

Change-Id: I1aff969b08a2f247ccf1fc636d6689350e819f29
Closes-Bug: #1636638

Revision history for this message
Hari Prasad Killi (haripk) wrote :

Issue is not present in R3.1 and later releases.

Changed in juniperopenstack:
status: New → Invalid
Jeba Paulaiyan (jebap)
no longer affects: juniperopenstack/r3.1
no longer affects: juniperopenstack/r3.2
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.