moving time.sleep to begin of pull topology routine to avoid unnecessary requests when topology is not ready (by all means)

Author: Italo Valcy

sdx_lc/jobs/pull_topo_changes.py

@@ -29,6 +29,7 @@ def main():
 
 def process_domain_controller_topo(db_instance):
     while True:
+        time.sleep(int(OXP_PULL_INTERVAL))
         latest_topology_exists = False
         latest_topology = db_instance.read_from_db("latest_topology")
 
@@ -52,11 +53,9 @@ def process_domain_controller_topo(db_instance):
             pulled_topology = urllib.request.urlopen(OXP_PULL_URL).read()
         except (urllib.request.URLError, ConnectionResetError):
             logger.debug("Error connecting to domain controller...")
-            time.sleep(int(OXP_PULL_INTERVAL))
             continue
 
         if not pulled_topology:
-            time.sleep(int(OXP_PULL_INTERVAL))
             continue
 
         logger.debug("Pulled request from domain controller")
@@ -90,8 +89,6 @@ def process_domain_controller_topo(db_instance):
         # Signal to end keep alive pings.
         rpc_producer.stop()
 
-        time.sleep(int(OXP_PULL_INTERVAL))
-
 
 if __name__ == "__main__":
     main()