]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
NSX: port status must reflect fabric, not link status
authorSalvatore Orlando <salv.orlando@gmail.com>
Thu, 27 Feb 2014 00:43:59 +0000 (16:43 -0800)
committerThomas Goirand <thomas@goirand.fr>
Thu, 13 Mar 2014 07:20:34 +0000 (15:20 +0800)
This patch ensures the status attribute for the port resource is
semantically consistent with the reference plugin when using the
NSX plugin.

Change-Id: Ifb9c7664da926d3e6d5c6ec7c0e375b53a97582a
Closes-Bug: 1285426

neutron/plugins/nicira/common/sync.py

index f6e0c3f5a9414f2da94abbd97ccee57737297e62..0e70a6d8703f29cab527a1ccd179f6d8398ac3e7 100644 (file)
@@ -203,7 +203,7 @@ class NvpSynchronizer():
     LP_URI = nvplib._build_uri_path(
         nvplib.LSWITCHPORT_RESOURCE,
         parent_resource_id='*',
-        fields='uuid,tags,fabric_status,link_status_up',
+        fields='uuid,tags,fabric_status_up',
         relations='LogicalPortStatus')
 
     def __init__(self, plugin, cluster, state_sync_interval,
@@ -423,7 +423,7 @@ class NvpSynchronizer():
         if lswitchport:
             lp_status = (lswitchport['_relations']
                          ['LogicalPortStatus']
-                         ['link_status_up'])
+                         ['fabric_status_up'])
             status = (lp_status and
                       constants.PORT_STATUS_ACTIVE
                       or constants.PORT_STATUS_DOWN)