YACHT-584: Fixed small bug

ocadotechnology · Jul 12, 2018 · bf60986 · bf60986
1 parent 46d5f07
commit bf60986
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/datastore_export/export_datastore_backups_to_gcs_service.py b/src/datastore_export/export_datastore_backups_to_gcs_service.py
@@ -47,11 +47,10 @@ def export(self, gcs_output_url, kinds):
         return self.__is_finished_with_success(response)
 
     def __is_finished_with_success(self, response):
-        operation_id = response["name"]
         finish_time = time.time() + TIMEOUT
-        self.__wait_till_done(operation_id)
+        self.__wait_till_done(response["name"])
 
-        if time.time() < finish_time:
+        if time.time() > finish_time:
             ErrorReporting().report(
                 "Timeout (%d seconds) exceeded !!!" % TIMEOUT)
             return False

diff --git a/src/datastore_export/load_datastore_backups_to_big_query_service.py b/src/datastore_export/load_datastore_backups_to_big_query_service.py
@@ -70,7 +70,7 @@ def __is_finished_with_success(self, load_job_id):
         finish_time = time.time() + TIMEOUT
         self.__wait_till_done(load_job_id)
 
-        if time.time() < finish_time:
+        if time.time() > finish_time:
             ErrorReporting().report(
                 "Timeout (%d seconds) exceeded !!!" % TIMEOUT)
             return False