From: Salvatore Orlando <salv.orlando@gmail.com>
Date: Thu, 27 Feb 2014 00:43:59 +0000 (-0800)
Subject: NSX: port status must reflect fabric, not link status
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=67aae424118c5eb79f666e53cc56d3be1caaced6;p=openstack-build%2Fneutron-build.git

NSX: port status must reflect fabric, not link status

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
---

diff --git a/neutron/plugins/nicira/common/sync.py b/neutron/plugins/nicira/common/sync.py
index f6e0c3f5a..0e70a6d87 100644
--- a/neutron/plugins/nicira/common/sync.py
+++ b/neutron/plugins/nicira/common/sync.py
@@ -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)