Add "pipeline error" to TestingFarmWatchResult

This is to be able to handle this result ``` 🔎 api https://api.dev.testing-farm.io/v0.1/requests/72efbafc-7db9-4e05-856c-873114a92435 📛 pipeline error Guest couldn't be provisioned: Artemis resource ended in 'error' state ```
fedora-llvm-team · Mar 31, 2024 · c7d92d8 · c7d92d8
1 parent 4aaec74
commit c7d92d8
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 2 deletions.
diff --git a/snapshot_manager/snapshot_manager/snapshot_manager.py b/snapshot_manager/snapshot_manager/snapshot_manager.py
@@ -167,6 +167,11 @@ def check_todays_builds(self):
                         chroot,
                         f'{chroot}<br /><a href="{artifacts_url}">{watch_result.to_icon()} {watch_result}{vpn}</a>',
                     )
+                else:
+                    build_status_matrix = build_status_matrix.replace(
+                        chroot,
+                        f"{chroot}<br />{watch_result.to_icon()} {watch_result}",
+                    )
 
                 if watch_result.is_error:
                     failed_test_cases.extend(

diff --git a/snapshot_manager/snapshot_manager/testing_farm_util.py b/snapshot_manager/snapshot_manager/testing_farm_util.py
@@ -38,6 +38,7 @@ class TestingFarmWatchResult(enum.StrEnum):
     TESTS_FAILED = "tests failed"  # Successfully built.
     TESTS_ERROR = "tests error"  # Build has been forked from another build.
     TESTS_UNKNOWN = "tests unknown"  # This package was skipped, see the reason for each chroot separately.
+    PIPELINE_ERROR = "pipeline error"
 
     def to_icon(self) -> str:
         """Get a github markdown icon for the given testing-farm watch result.
@@ -57,6 +58,8 @@ def to_icon(self) -> str:
             return ":x:"
         if self == self.TESTS_UNKNOWN:
             return ":grey_question:"
+        if self == self.PIPELINE_ERROR:
+            return ":warning:"
 
     @classmethod
     def all_watch_results(cls) -> list["TestingFarmWatchResult"]:
@@ -102,7 +105,11 @@ def is_error(self) -> bool:
     @property
     def expect_artifacts_url(self) -> bool:
         """Returns True if for the watch result we can expect and artifacts URL in the watch output."""
-        if self.value in [self.REQUEST_WAITING_TO_BE_QUEUED, self.REQUEST_QUEUED]:
+        if self.value in [
+            self.REQUEST_WAITING_TO_BE_QUEUED,
+            self.REQUEST_QUEUED,
+            self.PIPELINE_ERROR,
+        ]:
             return False
         return True
 
@@ -288,7 +295,7 @@ def parse_for_watch_result(string: str) -> tuple[TestingFarmWatchResult, str]:
     Returns:
         tuple[str, TestingFarmWatchResult]: _description_
 
-    Example:
+    Examples:
     >>> s='''8J+UjiBhcGkgaHR0cHM6Ly9hcGkuZGV2LnRlc3RpbmctZmFybS5pby92MC4xL3JlcXVlc3RzLzI3
     ... MWE3OWU4LWZjOWEtNGUxZC05NWZlLTU2N2NjOWQ2MmFkNArwn5qiIGFydGlmYWN0cyBodHRwOi8v
     ... YXJ0aWZhY3RzLm9zY2kucmVkaGF0LmNvbS90ZXN0aW5nLWZhcm0vMjcxYTc5ZTgtZmM5YS00ZTFk
@@ -297,6 +304,13 @@ def parse_for_watch_result(string: str) -> tuple[TestingFarmWatchResult, str]:
     >>> s = base64.b64decode(s).decode()
     >>> parse_for_watch_result(s)
     (<TestingFarmWatchResult.TESTS_ERROR: 'tests error'>, 'http://artifacts.osci.redhat.com/testing-farm/271a79e8-fc9a-4e1d-95fe-567cc9d62ad4')
+    >>> s='''8J+UjiBhcGkgaHR0cHM6Ly9hcGkuZGV2LnRlc3RpbmctZmFybS5pby92MC4xL3JlcXVlc3RzLzcy
+    ... ZWZiYWZjLTdkYjktNGUwNS04NTZjLTg3MzExNGE5MjQzNQrwn5ObIHBpcGVsaW5lIGVycm9yCkd1
+    ... ZXN0IGNvdWxkbid0IGJlIHByb3Zpc2lvbmVkOiBBcnRlbWlzIHJlc291cmNlIGVuZGVkIGluICdl
+    ... cnJvcicgc3RhdGUKCg=='''
+    >>> s = base64.b64decode(s).decode()
+    >>> parse_for_watch_result(s)
+    (<TestingFarmWatchResult.PIPELINE_ERROR: 'pipeline error'>, None)
     """
     string = clean_testing_farm_output(string)
     for watch_result in TestingFarmWatchResult.all_watch_results():