Recognize IANA assigned port number for MPLS in UDP

Bug #1420900 reported by Nischal Sheth
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.0
Fix Committed
Medium
Divakar Dharanalakota
R2.1
Fix Committed
Medium
Divakar Dharanalakota
R2.20
Fix Committed
Medium
Divakar Dharanalakota
R2.21.x
Fix Committed
Medium
Divakar Dharanalakota
Trunk
Fix Committed
Medium
Divakar Dharanalakota

Bug Description

IANA has now allocated a port number for MPLS in UDP. We should modify
the vRouter code so that we recognize and accept incoming packets with
the IANA assigned destination port (6635). We should continue to accept
the old destination port (51234) we use today.

We should also keep using 51234 for when sending packets. We can switch
over to using the IANA assigned destination port after a few releases.

http://www.iana.org/assignments/service-names-port-numbers/service-names-port-numbers.xml

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

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

commit b32445cfc121b22da1854fd8452c4dadbcef89b8
Author: Divakar <email address hidden>
Date: Sun Mar 29 22:53:24 2015 -0700

closes-bug:#1420900 Start accepting the new MplsoverUdp destination port
allotted by IANA. We continue to accept the old and new destination
ports and continue to use range of source ports

Change-Id: I9bff6fdafbe7e242e0a7aef582a856777879cc17

Nischal Sheth (nsheth)
information type: Proprietary → Public
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : R2.1

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

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

Review in progress for https://review.opencontrail.org/9914
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/9912
Committed: http://github.org/Juniper/contrail-vrouter/commit/cd8648e2364281d7641dfd1fcb98fa3098f9b19d
Submitter: Zuul
Branch: R2.1

commit cd8648e2364281d7641dfd1fcb98fa3098f9b19d
Author: Divakar <email address hidden>
Date: Sun Mar 29 22:53:24 2015 -0700

New MplsoverUdp destination port

Start accepting the new mplsover udp destination ports
allotted by IANA. We continue to accept the old and new
destination ports and continue to use range of source ports

closes-bug: #1420900

Change-Id: I9bff6fdafbe7e242e0a7aef582a856777879cc17

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

Reviewed: https://review.opencontrail.org/9914
Committed: http://github.org/Juniper/contrail-vrouter/commit/3464080f986dc32b048fb98642157edde6a57819
Submitter: Zuul
Branch: R2.0

commit 3464080f986dc32b048fb98642157edde6a57819
Author: Divakar <email address hidden>
Date: Sun Mar 29 22:53:24 2015 -0700

New MplsoverUdp destination port

Start accepting the new mplsover udp destination ports
allotted by IANA. We continue to accept the old and new
destination ports and continue to use range of source ports

closes-bug: #1420900

Change-Id: I9bff6fdafbe7e242e0a7aef582a856777879cc17

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

Review in progress for https://review.opencontrail.org/15578
Submitter: Hari Prasad Killi (<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/15579
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/15605
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/15578
Committed: http://github.org/Juniper/contrail-controller/commit/a7a4669a5a74608edbb72e3da610c011bfd1d469
Submitter: Zuul
Branch: R2.20

commit a7a4669a5a74608edbb72e3da610c011bfd1d469
Author: Hari <email address hidden>
Date: Thu Dec 3 18:50:21 2015 +0530

Accept both destination ports for MPLSoverUDP encapsulation.

The old destination port (51234) and the new IANA assigned destination
port (6635) should be accepted by the agent.

Change-Id: If6a640bddfdb89ca05930b7e3af196d5060a39b1
related-bug: 1420900

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

Reviewed: https://review.opencontrail.org/15579
Committed: http://github.org/Juniper/contrail-controller/commit/820adf41da5d4e3d1a9717abd46b28b9b618ef23
Submitter: Zuul
Branch: R2.21.x

commit 820adf41da5d4e3d1a9717abd46b28b9b618ef23
Author: Hari <email address hidden>
Date: Thu Dec 3 18:53:45 2015 +0530

Accept both destination ports for MPLSoverUDP encapsulation.

The old destination port (51234) and the new IANA assigned destination
port (6635) should be accepted by the agent.

Change-Id: I2894deb5dfafdfe107ae3e6b2d8bce68969a2d46
related-bug: 1420900

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

Reviewed: https://review.opencontrail.org/15605
Committed: http://github.org/Juniper/contrail-controller/commit/5c6e0ffcab9fb30ae805f666ea70275abede0ca1
Submitter: Zuul
Branch: master

commit 5c6e0ffcab9fb30ae805f666ea70275abede0ca1
Author: Hari <email address hidden>
Date: Fri Dec 4 14:50:55 2015 +0530

Accept both destination ports for MPLSoverUDP encapsulation.

The old destination port (51234) and the new IANA assigned destination
port (6635) should be accepted by the agent.

Change-Id: I5571e9c5ccca24c24619e323ce15d77001287542
related-bug: 1420900

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

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

commit e63ed051be314039a6b21a177f32ea1479ec9a17
Author: Divakar <email address hidden>
Date: Mon Feb 8 14:56:00 2016 +0530

Reverting from IANA MplsoUdp port

The MplsoUdp port is moved to IANA port (6635) while transmitting, with
the view that receiving vrouter would be able to handle both old (51234)
and new ports even across releases. But new port reception is fine
only in R3.0 and not in other releases.

So the port is moved back to old value and reception is handled well for
both old and new ports in 3.0. Transmitting port would be moved back to
new in later releases.

Change-Id: Ib9c5151702c1254df4b21c370031a19b3874b0eb
partial-bug: #1420900

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.