Fix case with no host_id in BigSwitch plugin

This correctly checks for the case where nova does not
provide the host_id in port creation/updates.

Fixes: bug #1195903
Change-Id: Ic0d714a3977810c6b1144c1e25488f75cc95d0f0

quantum/plugins/bigswitch/db/porttracker_db.py

@@ -17,6 +17,7 @@
 
 import sqlalchemy as sa
 
+from quantum.api.v2 import attributes
 from quantum.db import model_base
 from quantum.openstack.common import log as logging
 
@@ -38,6 +39,9 @@ def get_port_hostid(context, port_id):
 
 
 def put_port_hostid(context, port_id, host_id):
+    if not attributes.is_attr_set(host_id):
+        LOG.warning(_("No host_id in port request to track port location."))
+        return
     if port_id == '':
         LOG.warning(_("Received an empty port ID for host '%s'"), host_id)
         return

quantum/tests/unit/bigswitch/test_restproxy_plugin.py

@@ -107,6 +107,23 @@ def setUp(self):
         cfg.CONF.set_override('vif_type', 'ivs', 'NOVA')
 
 
+class TestNoHostIDVIFOverride(test_plugin.TestPortsV2,
+                              BigSwitchProxyPluginV2TestCase,
+                              test_bindings.PortBindingsTestCase):
+    VIF_TYPE = portbindings.VIF_TYPE_OVS
+    HAS_PORT_FILTER = False
+
+    def setUp(self):
+        super(TestNoHostIDVIFOverride, self).setUp()
+        cfg.CONF.set_override('vif_type', 'ovs', 'NOVA')
+
+    def test_port_vif_details(self):
+        kwargs = {'name': 'name', 'device_id': 'override_dev'}
+        with self.port(**kwargs) as port:
+            self.assertEqual(port['port']['binding:vif_type'],
+                             portbindings.VIF_TYPE_OVS)
+
+
 class TestBigSwitchVIFOverride(test_plugin.TestPortsV2,
                                BigSwitchProxyPluginV2TestCase,
                                test_bindings.PortBindingsTestCase):