External network route target limited to 16 bit instead of 32 bit

Bug #1584078 reported by Daniel Roesen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel Contrail plugin
Fix Released
Medium
Unassigned
3.0.x
Fix Released
Medium
Unassigned
4.x.x
Fix Released
Medium
Unassigned

Bug Description

FUEL 8.0, Contrail 3.0.1, plugin git master branch checkout on 2016-05-13.

Trying to enter route target local part in Fuel UI for the Contrail plugin, it refuses to take any values bigger than 2^16. This would be correct for standard BGP communities, but extended communities (e.g. route targets) support 32 bits for the local part.

Bug is in environment_config.yaml:

  contrail_route_target:
    value: '10000'
    label: 'Route Target'
    description: 'The route target number for Contrail external network'
    weight: 60
    type: "text"
    regex:
      source: '^(?:(6553[0-5])|(655[0-2]\d)|(65[0-4]\d{2})|(6[0-4]\d{3})|([1-5]\d{4})|([1-9]\d{1,3})|([1-9]))$'
      error: "Invalid target"

regex needs to be adapted to 32 bits.

Revision history for this message
Oleksandr Martsyniuk (omartsyniuk) wrote :

Hello,

Contrail uses a notation of as-number:community-value, where each value is a decimal number.
Based on that, plugin setting for route target refers to community-value only, using predefined as-number.

Illia Polliul (ipolliul)
Changed in fuel-plugin-contrail:
status: New → Invalid
Revision history for this message
Daniel Roesen (daniel-roesen) wrote :

Hi,

I'm talking only about the "community-value" part. This is a 32bit value, not a 16bit value!
The FUEL plugin currently limits the "community-value" part to 16bit, and this is a bug.

Best regards,
Daniel

Changed in fuel-plugin-contrail:
status: Invalid → New
Illia Polliul (ipolliul)
Changed in fuel-plugin-contrail:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → Oleksandr Martsyniuk (omartsyniuk)
milestone: none → 4.0.0
Changed in fuel-plugin-contrail:
status: Confirmed → In Progress
Illia Polliul (ipolliul)
Changed in fuel-plugin-contrail:
status: In Progress → Fix Committed
assignee: Oleksandr Martsyniuk (omartsyniuk) → nobody
Revision history for this message
Olesia Tsvigun (otsvigun) wrote :

Verified at fuel-plugin-contrail 4.0

Changed in fuel-plugin-contrail:
status: Fix Committed → Fix Released
Revision history for this message
Daniel Roesen (daniel-roesen) wrote :

Fix verified, works for us. Thanks!

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.