NSX: create_port should return empty list instead of null for allowed-address-pair

Bug #1366921 reported by Aaron Rosen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
Aaron Rosen
Icehouse
Fix Released
High
Aaron Rosen

Bug Description

ft133.5: tempest.api.network.test_ports.PortsTestJSON.test_show_port[gate,smoke]_StringException: pythonlogging:'': {{{2014-09-07 18:53:43,165 17979 INFO [tempest.common.rest_client] Request (PortsTestJSON:test_show_port): 200 GET http://localhost:9696/v2.0/ports/2827a27a-dee1-4013-b90f-cf2aeeae5f4f 0.030s}}}

Traceback (most recent call last):
  File "tempest/api/network/test_ports.py", line 81, in test_show_port
    (port, excluded_keys=['extra_dhcp_opts']))
  File "/opt/stack/tempest/.tox/smoke-serial/local/lib/python2.7/site-packages/testtools/testcase.py", line 433, in assertThat
    raise mismatch_error
MismatchError: Only in actual:
  {'binding:vnic_type': normal}
Differences:
  allowed_address_pairs: expected [], actual None

Traceback (most recent call last):
_StringException: Empty attachments:
  stderr
  stdout

pythonlogging:'': {{{2014-09-07 18:53:43,165 17979 INFO [tempest.common.rest_client] Request (PortsTestJSON:test_show_port): 200 GET http://localhost:9696/v2.0/ports/2827a27a-dee1-4013-b90f-cf2aeeae5f4f 0.030s}}}

Traceback (most recent call last):
  File "tempest/api/network/test_ports.py", line 81, in test_show_port
    (port, excluded_keys=['extra_dhcp_opts']))
  File "/opt/stack/tempest/.tox/smoke-serial/local/lib/python2.7/site-packages/testtools/testcase.py", line 433, in assertThat
    raise mismatch_error
MismatchError: Only in actual:
  {'binding:vnic_type': normal}
Differences:
  allowed_address_pairs: expected [], actual None

Traceback (most recent call last):
_StringException: Empty attachments:
  stderr
  stdout

pythonlogging:'': {{{2014-09-07 18:53:43,165 17979 INFO [tempest.common.rest_client] Request (PortsTestJSON:test_show_port): 200 GET http://localhost:9696/v2.0/ports/2827a27a-dee1-4013-b90f-cf2aeeae5f4f 0.030s}}}

Traceback (most recent call last):
  File "tempest/api/network/test_ports.py", line 81, in test_show_port
    (port, excluded_keys=['extra_dhcp_opts']))
  File "/opt/stack/tempest/.tox/smoke-serial/local/lib/python2.7/site-packages/testtools/testcase.py", line 433, in assertThat
    raise mismatch_error
MismatchError: Only in actual:
  {'binding:vnic_type': normal}
Differences:
  allowed_address_pairs: expected [], actual None

Aaron Rosen (arosen)
Changed in neutron:
assignee: nobody → Aaron Rosen (arosen)
importance: Undecided → High
tags: added: vmware
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/119864

Changed in neutron:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/icehouse)

Fix proposed to branch: stable/icehouse
Review: https://review.openstack.org/120160

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/119864
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=1361ad99969d2a729599720f06f8373db230e0ed
Submitter: Jenkins
Branch: master

commit 1361ad99969d2a729599720f06f8373db230e0ed
Author: Aaron Rosen <email address hidden>
Date: Mon Sep 8 11:23:50 2014 -0700

    NSX: Correct allowed_address_pair return value on create_port

    This patch fixes the NSX plugin to return [] rather than None for a
    port does not have any allowed_address_pairs when a create_port api
    call is made. Currently the NSX plugin is failing every patch because
    now tempest is checking for this value to be [].

    Change-Id: Ic82c564ff62000c367c544f44a76694be9375790
    Closes-bug: 1366921

Changed in neutron:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/icehouse)

Reviewed: https://review.openstack.org/120160
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=255df456334a9a888f92ad22b2e53fef8492a091
Submitter: Jenkins
Branch: stable/icehouse

commit 255df456334a9a888f92ad22b2e53fef8492a091
Author: Aaron Rosen <email address hidden>
Date: Mon Sep 8 11:23:50 2014 -0700

    NSX: Correct allowed_address_pair return value on create_port

    This patch fixes the NSX plugin to return [] rather than None for a
    port does not have any allowed_address_pairs when a create_port api
    call is made. Currently the NSX plugin is failing every patch because
    now tempest is checking for this value to be [].

    Change-Id: Ic82c564ff62000c367c544f44a76694be9375790
    Closes-bug: 1366921
    (cherry picked from commit 1361ad99969d2a729599720f06f8373db230e0ed)

tags: added: in-stable-icehouse
Thierry Carrez (ttx)
Changed in neutron:
milestone: none → juno-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: juno-rc1 → 2014.2
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.