R4.1: Segment health check: HC packets should not be sent with vrrp mac as dst mac

Bug #1724131 reported by Shashikiran H
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R4.1
Fix Committed
Critical
Hari Prasad Killi
Trunk
Fix Committed
Critical
Hari Prasad Killi

Bug Description

For segment health check for transparent SI, the HC packets are dropped by the VMs intermittently.

Naveen debugged it to find that the packet's dst mac is set to vrrp mac, and as its already learnt, packets are dropped on the left interface. And after arp timeout, maybe packets go through.

Shashikiran H (skiranh)
tags: added: blocker
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.1

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

Ashok Singh (ashoksr)
Changed in juniperopenstack:
assignee: Ashok Singh (ashoksr) → Hari Prasad Killi (haripk)
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/36722
Committed: http://github.com/Juniper/contrail-controller/commit/b31a3f2326c9569b3921453789c1bcccfab64d7f
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit b31a3f2326c9569b3921453789c1bcccfab64d7f
Author: Hari Prasad Killi <email address hidden>
Date: Mon Oct 23 15:53:34 2017 +0530

Do not use vrouter mac in the segment health check packet

Hard coding to use the same MAC that we use in other traffic to transparent
service instance (02:00:00:00:00:01 while sending on left and 02:00:00:00:00:02
while sending on right interfaces).

Change-Id: If01d527be0cecca428749b845805cde5e11b08e3
closes-bug: #1724131

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

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

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

Review in progress for https://review.opencontrail.org/36968
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/36834
Committed: http://github.com/Juniper/contrail-controller/commit/1b0360bea91fcf87da19b4a122e4493578219339
Submitter: Zuul (<email address hidden>)
Branch: master

commit 1b0360bea91fcf87da19b4a122e4493578219339
Author: Hari Prasad Killi <email address hidden>
Date: Mon Oct 23 15:53:34 2017 +0530

Do not use vrouter mac in the segment health check packet

Hard coding to use the same MAC that we use in other traffic to transparent
service instance (02:00:00:00:00:01 while sending on left and 02:00:00:00:00:02
while sending on right interfaces).

Change-Id: If01d527be0cecca428749b845805cde5e11b08e3
closes-bug: #1724131
(cherry picked from commit b31a3f2326c9569b3921453789c1bcccfab64d7f)

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

Review in progress for https://review.opencontrail.org/36968
Submitter: Hari Prasad Killi (<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/37009
Submitter: Hari Prasad Killi (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/36968
Submitter: Hari Prasad Killi (<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/37009
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/36968
Committed: http://github.com/Juniper/contrail-controller/commit/eec960d4ba8d0c64b0348d1ed8c1bf31e6c899ef
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit eec960d4ba8d0c64b0348d1ed8c1bf31e6c899ef
Author: Hari Prasad Killi <email address hidden>
Date: Sun Oct 29 19:50:51 2017 +0530

Change the MAC used for segment health check packets

Using vrouter mac as destination MAC could result in packets getting
dropped in the transaparent SI, as it is learnt on different interfaces
while receiving ARP packets. Changing to use new source and destination
MACs for these packets and add bridge entries for these.

Change-Id: Ice1ff28eb347528aa004c5d2c19f95147e52725a
closes-bug: #1724131

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

Reviewed: https://review.opencontrail.org/37009
Committed: http://github.com/Juniper/contrail-controller/commit/67c008af6c6c4a50b5d450a4aa8c6125744def8e
Submitter: Zuul (<email address hidden>)
Branch: master

commit 67c008af6c6c4a50b5d450a4aa8c6125744def8e
Author: Hari Prasad Killi <email address hidden>
Date: Sun Oct 29 19:50:51 2017 +0530

Change the MAC used for segment health check packets

Using vrouter mac as destination MAC could result in packets getting
dropped in the transaparent SI, as it is learnt on different interfaces
while receiving ARP packets. Changing to use new source and destination
MACs for these packets and add bridge entries for these.

Change-Id: Ice1ff28eb347528aa004c5d2c19f95147e52725a
closes-bug: #1724131

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.