squidient 3.0.4__tar.gz → 3.0.6__tar.gz

{squidient-3.0.4 → squidient-3.0.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: squidient
-Version: 3.0.4
+Version: 3.0.6
 Summary: CI/CD framework for alya
 Project-URL: Homepage, https://alya.gitlab.bsc.es/alya/ci-cd/squidient
 Project-URL: Bug Tracker, https://alya.gitlab.bsc.es/alya/ci-cd/squidient/issues

{squidient-3.0.4 → squidient-3.0.6}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "squidient"
-version = "3.0.4"
+version = "3.0.6"
 authors = [
   { name="Damien Dosimont", email="damien.dosimont@gmail.com" },
 ]

{squidient-3.0.4 → squidient-3.0.6}/src/squidient/cli/monitoring.py

@@ -82,7 +82,7 @@ def _run_cli(argv: Sequence[str]) -> int:
         monitoring.init()
 
         if "validate" == argv[0]:
-            monitoring.validate()
+            return 0 if monitoring.validate() else 1
 
         elif "build" == argv[0]:
             monitoring.lock(terraform)

{squidient-3.0.4 → squidient-3.0.6}/src/squidient/cli/staging.py

@@ -137,7 +137,7 @@ def _run_cli(argv: Sequence[str]) -> int:
             cicd.init()
 
             if "validate" == argv[0]:
-                cicd.validate()
+                return 0 if cicd.validate() else 1
 
             elif "report" == argv[0]:
                 if len(argv) >= 2:

{squidient-3.0.4 → squidient-3.0.6}/src/squidient/monitoring/monitoring.py

@@ -139,7 +139,7 @@ class Monitoring(Staging):
             console.error(message + "\nDetails written to " + log_path, level=1)
         if len(failures) > 0:
             raise WorkflowFailure("Benchmark notification email could not be sent")
-        super().validate()
+        return super().validate()
 
     def write_notification_failure_log(self, failures):
         log_dir = os.path.dirname(benchmark_notification_failure_log)
@@ -197,8 +197,6 @@ class Monitoring(Staging):
             console.report("web/monitoring_test.html", level=1)
             print_line()
             self.clean_locks()
-            if not status:
-                raise WorkflowFailure("Monitoring test phase failed")
 
     def prepare_monitoring_test_builds(self, fake=False):
         if not fake:

{squidient-3.0.4 → squidient-3.0.6}/src/squidient/staging/staging.py

@@ -327,8 +327,6 @@ class Staging:
             console.report("web/staging_test.html", level=1)
             print_line()
             self.clean_locks()
-            if not status:
-                raise WorkflowFailure("Test phase failed")
 
     def create_test_orchestrator(self):
         return TestOrchestrator(
@@ -568,10 +566,12 @@ class Staging:
         self.clean_locks()
 
     def validate(self):
+        console.section("Validation result")
         if self._report.get_valid():
+            console.success("Report is valid", level=1)
             return True
-        else:
-            raise WorkflowFailure("Report is not valid")
+        console.error("Report is not valid", level=1)
+        return False
 
     def report_push(self):
         with self.timer("staging.report_push", local=False, system="bsc"):

{squidient-3.0.4 → squidient-3.0.6}/src/squidient/testing/benchmarks/failure_notifier.py

@@ -20,6 +20,7 @@
 from email.message import EmailMessage
 from ...definitions import benchmark_failure_log_files
 import os
+import re
 import shutil
 import smtplib
 import subprocess
@@ -68,9 +69,24 @@ class BenchmarkFailureNotifier:
         self._from_address = from_address
         if copy_recipients is None:
             copy_recipients = []
-        self._copy_recipients = copy_recipients
+        self._copy_recipients = self.normalize_recipients(copy_recipients)
         self._mail_sender = mail_sender or self.send_mail
 
+    @staticmethod
+    def normalize_recipients(recipients):
+        if recipients is None:
+            return []
+        if isinstance(recipients, str):
+            candidates = re.split(r"[,;]", recipients)
+        else:
+            candidates = recipients
+        normalized = []
+        for candidate in candidates:
+            candidate = str(candidate).strip()
+            if "@" in candidate and candidate not in normalized:
+                normalized.append(candidate)
+        return normalized
+
     def notify(self, report):
         recipients = self.recipients(report.get("authors", []))
         if len(recipients) == 0: