logdetective 0.11.2__tar.gz → 1.0.0__tar.gz

{logdetective-0.11.2 → logdetective-1.0.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: logdetective
-Version: 0.11.2
+Version: 1.0.0
 Summary: Log using LLM AI to search for build/test failures and provide ideas for fixing these.
 License: Apache-2.0
 Author: Jiri Podivin

{logdetective-0.11.2 → logdetective-1.0.0}/logdetective/server/gitlab.py

@@ -93,7 +93,7 @@ async def process_gitlab_job_event(
     preprocessed_log.close()
 
     # check if this project is on the opt-in list for posting comments.
-    if project.name not in SERVER_CONFIG.general.packages:
+    if not is_eligible_package(project.name):
         LOG.info("Not publishing comment for unrecognized package %s", project.name)
         return
 
@@ -111,6 +111,31 @@ async def process_gitlab_job_event(
     return staged_response
 
 
+def is_eligible_package(project_name: str):
+    """Check whether the provided package name is eligible for posting
+    comments to the merge request"""
+
+    # First check the allow-list. If it's not allowed, we deny.
+    allowed = False
+    for pattern in SERVER_CONFIG.general.packages:
+        print(f"include {pattern}")
+        if re.search(pattern, project_name):
+            allowed = True
+            break
+    if not allowed:
+        # The project did not match any of the permitted regular expressions
+        return False
+
+    # Next, check the deny-list. If it was allowed before, but denied here, we deny.
+    for pattern in SERVER_CONFIG.general.excluded_packages:
+        print(f"Exclude {pattern}")
+        if re.search(pattern, project_name):
+            return False
+
+    # It was allowed and not denied, so return True to indicate it is eligible
+    return True
+
+
 class LogsTooLargeError(RuntimeError):
     """The log archive exceeds the configured maximum size"""
 
@@ -174,16 +199,17 @@ async def retrieve_and_preprocess_koji_logs(
             with artifacts_zip.open(zipinfo.filename) as task_failed_log:
                 contents = task_failed_log.read().decode("utf-8")
                 match = FAILURE_LOG_REGEX.search(contents)
-                if not match:
-                    LOG.error(
-                        "task_failed.log does not indicate which log contains the failure."
-                    )
-                    raise SyntaxError(
+                if match:
+                    failure_log_name = match.group(1)
+                    failed_arches[architecture] = PurePath(path.parent, failure_log_name)
+                else:
+                    LOG.info(
                         "task_failed.log does not indicate which log contains the failure."
                     )
-                failure_log_name = match.group(1)
-
-            failed_arches[architecture] = PurePath(path.parent, failure_log_name)
+                    # The best thing we can do at this point is return the
+                    # task_failed.log, since it will probably contain the most
+                    # relevant information
+                    failed_arches[architecture] = path
 
     if not failed_arches:
         # No failed task found in the sub-tasks.

{logdetective-0.11.2 → logdetective-1.0.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "logdetective"
-version = "0.11.2"
+version = "1.0.0"
 description = "Log using LLM AI to search for build/test failures and provide ideas for fixing these."
 authors = ["Jiri Podivin <jpodivin@gmail.com>"]
 license = "Apache-2.0"