commit f229101b929802e10280f20486ec4b350fec79f8
author Scott Baker <smbaker@gmail.com> Thu Sep 08 13:19:45 2016 -0700
committer Scott Baker <smbaker@gmail.com> Thu Sep 08 13:19:45 2016 -0700
tree 9abbf44cba3f014c8451352d3048eaf77a75c962
parent 90f780de641e60f0018eb468170d75b17747c111

don't do VTN APIs on networks/ports with no id

Change-Id: I7d48133c52bb9554965e1b7749ffdbcd665fe080

xos/synchronizer/steps/sync_vtn_service.py

diff --git a/xos/synchronizer/steps/sync_vtn_service.py b/xos/synchronizer/steps/sync_vtn_service.py
index 000f303..a4016e5 100644
--- a/xos/synchronizer/steps/sync_vtn_service.py
+++ b/xos/synchronizer/steps/sync_vtn_service.py
@@ -116,7 +116,12 @@
         valid_ids = []
         for network in Network.objects.all():
             network = VTNNetwork(network)
+
+            if not network.id:
+                continue
+
             valid_ids.append(network.id)
+
             if (glo_saved_networks.get(network.id, None) != network.to_dict()):
                 (exists, url, method, req_func) = self.get_method("http://" + self.get_vtn_addr() + ":8181/onos/cordvtn/serviceNetworks", network.id)
 
@@ -156,7 +161,12 @@
         valid_ids = []
         for port in Port.objects.all():
             port = VTNPort(port)
+
+            if not port.id:
+                continue
+
             valid_ids.append(port.id)
+
             if (glo_saved_ports.get(port.id, None) != port.to_dict()):
                 (exists, url, method, req_func) = self.get_method("http://" + self.get_vtn_addr() + ":8181/onos/cordvtn/servicePorts", port.id)
 
@@ -170,7 +180,7 @@
                 logger.info("DATA: %s" % str(data))
 
                 r=req_func(url, json=data, auth=self.get_vtn_auth() )
-                if (r.status_code not in [200,201]):
+                if (r.status_code in [200,201]):
                     glo_saved_ports[port.id] = port.to_dict()
                 else:
                     logger.error("Received error from vtn service (%d)" % r.status_code)