Use IANA assigned port number for MPLS in UDP

Bug #1420905 reported by Nischal Sheth
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
Medium
Divakar Dharanalakota

Bug Description

Bug 1420900 tracks change to recognize the IANA assigned destination port
for received packets. After that has been fixed for a few releases, we can
switch to using the IANA assigned port when sending MPLS-in-UDP packets.

Note that we should accept the IANA assigned port as well as port 51234
on the receive side. This ensures interoperability with older releases
which use port 51234 to send packets. We can stop recognizing port 51234
for received packets after a few releases when all supported releases in
the field have switched to using the IANA port for send.

Tags: vrouter
Nischal Sheth (nsheth)
description: updated
Nischal Sheth (nsheth)
information type: Proprietary → Public
Revision history for this message
Nischal Sheth (nsheth) wrote :

Since we fixed bug 1420900 in r2.01 we can fix this in 2.30.
Assumption is that 2.30 will not need to interoperate with
releases older than 2.01.

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

Review in progress for https://review.opencontrail.org/13241
Submitter: Divakar Dharanalakota (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/13241
Committed: http://github.org/Juniper/contrail-vrouter/commit/0aac410719aa68e594c00d21dd916581988fc5de
Submitter: Zuul
Branch: master

commit 0aac410719aa68e594c00d21dd916581988fc5de
Author: Divakar <email address hidden>
Date: Sat Aug 22 08:21:27 2015 +0530

Change MplsOudp destination port to IANA port

Right now the destination port that Vrouter uses in MplsOudp packet is
51234 which is not by IANA. But Vrouter is accepting the packets in the
reception path both old and new port numbers. With this change we will
be sening the destination port also to be IANA assigned port.

Change-Id: Idc6e1b3bf3f58fa15d0ad1184c60ea7e75865336
closes-bug: #1420905

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

Review in progress for https://review.opencontrail.org/13333
Submitter: Naveen N (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/13333
Committed: http://github.org/Juniper/contrail-controller/commit/9113075f06425294fa219984e48fbae7df95ea12
Submitter: Zuul
Branch: master

commit 9113075f06425294fa219984e48fbae7df95ea12
Author: Naveen N <email address hidden>
Date: Wed Aug 26 04:01:24 2015 -0700

Use IANA destination port in agent whike parsing MplsoUdp packet
closes-bug: #1420905

Change-Id: I4717eec538b074977dacf1b443a65b2f711d6ba4

Revision history for this message
Nischal Sheth (nsheth) wrote :

Reopening to fix in a future release since this change
was reverted via the last commit to bug 1420900.

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

Review in progress for https://review.opencontrail.org/21924
Submitter: Divakar Dharanalakota (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/21924
Committed: http://github.org/Juniper/contrail-vrouter/commit/e10f1468308ca32382a9612eb9d8a45f65702aa2
Submitter: Zuul
Branch: master

commit e10f1468308ca32382a9612eb9d8a45f65702aa2
Author: Divakar <email address hidden>
Date: Wed Jul 13 10:09:41 2016 +0530

Change MplsOudp destination port to IANA port

Right now the destination port that Vrouter uses in MplsOudp packet is
51234 which is not by IANA. But Vrouter is accepting the packets in the
reception path both old and new port numbers. With this change we will
be sening the destination port also to be IANA assigned port.

Change-Id: Ida38a910e9d26c2527bd9bb9cb19d462bdd10adc
closes-bug: #1420905

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

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