Merge pull request #284 from cta-observatory/check-closer

Change "&" by "and" in the while checking the status of autocloser jobs
cta-observatory · Mar 8, 2024 · b8767ae · b8767ae
2 parents 501ada5 + f5e2512
commit b8767ae
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/osa/scripts/closer.py b/src/osa/scripts/closer.py
@@ -193,7 +193,7 @@ def post_process(seq_tuple):
     # before creating the NightFinished.txt file
     n_max = 6
     n = 0
-    while not all_closer_jobs_finished_correctly() & n <= n_max:
+    while not all_closer_jobs_finished_correctly() and n <= n_max:
         log.info(
             "All jobs launched by autocloser did not finished correctly yet. "
             "Checking again in 10 minutes..."
@@ -202,7 +202,7 @@ def post_process(seq_tuple):
         n += 1
 
     if n > n_max:
-        send_warning_mail(date=options.date)
+        send_warning_mail(date=date_to_iso(options.date))
         return False
 
     if options.seqtoclose is None: