vRouter: Make provision for mirror source ip to come from the original packet

Bug #1545590 reported by Anand H. Krishnan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Fix Committed
Undecided
Anand H. Krishnan
R2.21.x
Fix Committed
Undecided
Anand H. Krishnan
R2.22.x
Fix Committed
Undecided
Anand H. Krishnan
R3.0
Fix Committed
Undecided
Anand H. Krishnan
Trunk
Fix Committed
Undecided
Anand H. Krishnan

Bug Description

vRouter has to make provision to allow source ip of the mirror tunnel to come from the original packet. Till now, we were using the vhost ip address, which may not make sense in the vrf to which the mirror vm belongs.

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

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

commit d390035d5aebad06fc41b5365853e1a039237c67
Author: Anand H. Krishnan <email address hidden>
Date: Mon Feb 15 11:09:41 2016 +0530

Provision to take mirror source ip from the packet

vRouter has to make provision to allow source ip of the mirror tunnel
to come from the original packet. Till now, we were using the vhost ip
address, which may not make sense in the vrf to which the mirror vm
belongs.

Change-Id: I3147d57aa16ad808327036db557387fb2ef3782c
Closes-BUG: #1545590

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

Review in progress for https://review.opencontrail.org/17417
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/17433
Submitter: Ashok Singh (<email address hidden>)

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

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

commit 0db92dec3ebe882b120727c35989de89471689c3
Author: Ashok Singh <email address hidden>
Date: Fri Feb 19 03:25:35 2016 -0800

Program source IP copy flag for Mirror NH

Set the copy flag when VRF is not fabric-VRF for Mirror NH.
Also update the KState NH code to display this new flag.

Change-Id: I4f78ca3bcc5c2e71def730a72d8ef7ab80800b73
Closes-Bug: #1545590

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

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

commit 5668e4c8d5b9777c60f4c57ab70680c2c5add787
Author: Ashok Singh <email address hidden>
Date: Fri Feb 19 01:33:36 2016 -0800

Program source IP copy flag for Mirror NH

Set the copy flag when VRF is not fabric-VRF for Mirror NH.
Also update the KState NH code to display this new flag.

Change-Id: I377dbe5c2d46e998b17ddf2772617d0a30cfc725
Closes-Bug: #1545590

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

Review in progress for https://review.opencontrail.org/18427
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/18427
Committed: http://github.org/Juniper/contrail-vrouter/commit/fd0566f80587de2691d6a1b7093da37266f804af
Submitter: Zuul
Branch: R2.21.x

commit fd0566f80587de2691d6a1b7093da37266f804af
Author: Anand H. Krishnan <email address hidden>
Date: Mon Feb 15 11:09:41 2016 +0530

Provision to take mirror source ip from the packet

vRouter has to make provision to allow source ip of the mirror tunnel
to come from the original packet. Till now, we were using the vhost ip
address, which may not make sense in the vrf to which the mirror vm
belongs.

Change-Id: I3147d57aa16ad808327036db557387fb2ef3782c
Closes-BUG: #1545590

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

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

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

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

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

Review in progress for https://review.opencontrail.org/18478
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/18467
Committed: http://github.org/Juniper/contrail-vrouter/commit/fde2003c6cceb601eb85ddd8521c2a01baaae9a2
Submitter: Zuul
Branch: R2.22.x

commit fde2003c6cceb601eb85ddd8521c2a01baaae9a2
Author: Anand H. Krishnan <email address hidden>
Date: Mon Feb 15 11:09:41 2016 +0530

Provision to take mirror source ip from the packet

vRouter has to make provision to allow source ip of the mirror tunnel
to come from the original packet. Till now, we were using the vhost ip
address, which may not make sense in the vrf to which the mirror vm
belongs.

Change-Id: I3147d57aa16ad808327036db557387fb2ef3782c
Closes-BUG: #1545590

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

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

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

Review in progress for https://review.opencontrail.org/18478
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/18478
Committed: http://github.org/Juniper/contrail-controller/commit/82d367f27e96821a59a7b392cfa6a2f504acc009
Submitter: Zuul
Branch: R2.21.x

commit 82d367f27e96821a59a7b392cfa6a2f504acc009
Author: Ashok Singh <email address hidden>
Date: Fri Feb 19 03:25:35 2016 -0800

Program source IP copy flag for Mirror NH

Set the copy flag when VRF is not fabric-VRF for Mirror NH.
Also update the KState NH code to display this new flag.

Closes-Bug: #1545590
(cherry picked from commit 0db92dec3ebe882b120727c35989de89471689c3)

Change-Id: If409a8ac159ea8b756c692c6e90012159099b075

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

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

commit c4cb8724834b33df95df6200f66f5cc406655e80
Author: Anand H. Krishnan <email address hidden>
Date: Mon Feb 15 11:09:41 2016 +0530

Provision to take mirror source ip from the packet

vRouter has to make provision to allow source ip of the mirror tunnel
to come from the original packet. Till now, we were using the vhost ip
address, which may not make sense in the vrf to which the mirror vm
belongs.

Change-Id: I3147d57aa16ad808327036db557387fb2ef3782c
Closes-BUG: #1545590

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

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

commit 853267fdc2a51f98ddbbd12a99b4e0f6e2d792c8
Author: Ashok Singh <email address hidden>
Date: Thu Mar 17 07:46:01 2016 -0700

Program source IP copy flag for Mirror NH

Set the copy flag when VRF is not fabric-VRF for Mirror NH.
Also update the KState NH code to display this new flag.

Change-Id: I9ef45ce30236f5e1ce20dbf8ff7ab44989034df3
Closes-Bug: #1545590

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

Reviewed: https://review.opencontrail.org/18484
Committed: http://github.org/Juniper/contrail-controller/commit/be3749dbf5e0aba3e582b6825cf3f4cce4128d9e
Submitter: Zuul
Branch: R2.22.x

commit be3749dbf5e0aba3e582b6825cf3f4cce4128d9e
Author: Ashok Singh <email address hidden>
Date: Wed Mar 16 08:18:38 2016 -0700

Program source IP copy flag for Mirror NH

Set the copy flag when VRF is not fabric-VRF for Mirror NH.
Also update the KState NH code to display this new flag.

Change-Id: Ie7513ac412f0f35837e1a994a7aadc7551c36b51
Closes-Bug: #1545590

information type: Proprietary → Public
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.