yahoo-eng-team team mailing list archive

[Tom Stappaerts <1718385@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

When supplying a subport for a trunk with segmenation_id 0 Neutron
reports:

Invalid subport details '{u'segmentation_type': u'vlan', u'port_id':
u'449451c4-f134-4fbc-b405-df9109746e51', u'segmentation_id': 0}':
missing segmentation information. Must specify both segmentation_id and
segmentation_type from (pid=16359) validate_subports
/Ocata/local/lib/python2.7/site-
packages/neutron_lib/api/validators.py:923

This is due to the following lines: 
        segmentation_id = subport.get("segmentation_id")
        segmentation_type = subport.get("segmentation_type")
        if (not segmentation_id or not segmentation_type) and len(subport) > 1:

Can we allow segmentation_id 0 to pass or provide a more suitable error
message?

** Affects: neutron
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1718385

Title:
  neutron-lib validate_subports incorrectly blocks segmentation_id 0

Status in neutron:
  New

Bug description:
  When supplying a subport for a trunk with segmenation_id 0 Neutron
  reports:

  Invalid subport details '{u'segmentation_type': u'vlan', u'port_id':
  u'449451c4-f134-4fbc-b405-df9109746e51', u'segmentation_id': 0}':
  missing segmentation information. Must specify both segmentation_id
  and segmentation_type from (pid=16359) validate_subports
  /Ocata/local/lib/python2.7/site-
  packages/neutron_lib/api/validators.py:923

  This is due to the following lines: 
          segmentation_id = subport.get("segmentation_id")
          segmentation_type = subport.get("segmentation_type")
          if (not segmentation_id or not segmentation_type) and len(subport) > 1:

  Can we allow segmentation_id 0 to pass or provide a more suitable
  error message?

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1718385/+subscriptions