squad 1.92__py3-none-any.whl → 1.93.1__py3-none-any.whl

squad/ci/backend/tuxsuite.py

@@ -10,6 +10,7 @@ from requests.adapters import HTTPAdapter, Retry
 from functools import reduce
 from urllib.parse import urljoin
 
+from cryptography.exceptions import InvalidSignature
 from cryptography.hazmat.primitives.asymmetric import ec
 from cryptography.hazmat.primitives import (
     hashes,
@@ -481,22 +482,18 @@ class Backend(BaseBackend):
         if public_key is None:
             raise Exception("missing tuxsuite public key for this project")
 
-        payload = json.loads(request.body)
         signature = base64.urlsafe_b64decode(signature)
         key = serialization.load_ssh_public_key(public_key.encode("ascii"))
-        key.verify(
-            signature,
-            payload.encode("utf-8"),
-            ec.ECDSA(hashes.SHA256()),
-        )
+        try:
+            key.verify(
+                signature,
+                request.body,
+                ec.ECDSA(hashes.SHA256()),
+            )
+        except InvalidSignature:
+            raise Exception("Failed to verify signature against payload")
 
     def process_callback(self, json_payload, build, environment, backend):
-        # The payload coming from Tuxsuite is formatted as bytes,
-        # so after the first json.loads(request.body), the result
-        # will still be a string containing the actual json document
-        # We need to call json.loads() once more to get the actual
-        # python dict containing all the information we need
-        json_payload = json.loads(json_payload)
         if "kind" not in json_payload or "status" not in json_payload:
             raise Exception("`kind` and `status` are required in the payload")
 

squad/ci/models.py

@@ -138,10 +138,8 @@ class Backend(models.Model):
                 completed=completed,
             )
             test_job.testrun = testrun
-        except InvalidMetadata as exception:
+        except (DuplicatedTestJob, InvalidMetadata) as exception:
             test_job.failure = str(exception)
-        except DuplicatedTestJob as exception:
-            logger.error('Failed to fetch test_job(%d): "%s"' % (test_job.id, str(exception)))
 
         if test_job.needs_postprocessing():
             # Offload postprocessing plugins to a new task

squad/core/management/commands/run_log_parser.py

@@ -0,0 +1,44 @@
+from django.core.management.base import BaseCommand
+
+from squad.plugins.linux_log_parser import Plugin as BootTestLogParser
+from squad.plugins.linux_log_parser_build import Plugin as BuildLogParser
+
+
+class FakeTestRun:
+    log_file = None
+    id = None
+
+
+log_parsers = {
+    'linux_log_parser_boot_test': BootTestLogParser(),
+    "linux_log_parser_build": BuildLogParser(),
+}
+
+
+class Command(BaseCommand):
+
+    help = """Run a log parser and print the outputs to the stdout."""
+
+    def add_arguments(self, parser):
+
+        parser.add_argument(
+            "LOG_FILE",
+            help="Log file to parser",
+        )
+
+        parser.add_argument(
+            "LOG_PARSER",
+            choices=log_parsers.keys(),
+            help="Which log parser to run"
+        )
+
+    def handle(self, *args, **options):
+        self.options = options
+
+        with open(options["LOG_FILE"], "r") as f:
+            log_file = f.read()
+
+        testrun = FakeTestRun()
+        testrun.log_file = log_file
+        parser = log_parsers[options["LOG_PARSER"]]
+        parser.postprocess_testrun(testrun, squad=False, print=True)

squad/core/models.py

@@ -493,6 +493,9 @@ class Build(models.Model):
         ordering = ['datetime']
 
     def save(self, *args, **kwargs):
+        # Initialize this to timezone.now(), then if a testrun is seen with an
+        # earlier datetime, keep this value up to date with the earliest
+        # testrun.datetime (handled in ReceiveTestRun.__call__).
         if not self.datetime:
             self.datetime = timezone.now()
         with transaction.atomic():
@@ -577,12 +580,17 @@ class Build(models.Model):
         List of attachments from all testruns
         """
         if self.__attachments__ is None:
+            test_run_ids = self.test_runs.values_list('id', flat=True)
+            all_attachments = Attachment.objects.filter(test_run_id__in=test_run_ids).values(
+                'test_run_id', 'filename'
+            )
+
             attachments = {}
-            for test_run in self.test_runs.all():
-                attachments[test_run.pk] = []
-                for attachment in test_run.attachments.all():
-                    attachments[test_run.pk].append(attachment.filename)
+            for attachment in all_attachments:
+                attachments.setdefault(attachment['test_run_id'], []).append(attachment['filename'])
+
             self.__attachments__ = attachments
+
         return self.__attachments__
 
     @property
@@ -801,6 +809,9 @@ class TestRun(models.Model):
         unique_together = ('build', 'job_id')
 
     def save(self, *args, **kwargs):
+        # testrun.datetime will take datetime from the metadata if it exists
+        # (during ReceiveTestRun.__call__). If datetime is not in the metadata,
+        # set it to timezone.now()
         if not self.datetime:
             self.datetime = timezone.now()
         if self.__metadata__:

squad/core/tasks/__init__.py

@@ -178,6 +178,8 @@ class ReceiveTestRun(object):
 
         testrun.refresh_from_db()
 
+        # This keeps the datetime of the build in line with the earliest
+        # observed testrun.datetime.
         if not build.datetime or testrun.datetime < build.datetime:
             build.datetime = testrun.datetime
             build.save()

squad/plugins/lib/base_log_parser.py

@@ -16,17 +16,44 @@ square_brackets_and_contents = r"\[[^\]]+\]"
 
 class BaseLogParser:
     def compile_regexes(self, regexes):
-        combined = [r"(%s)" % r[REGEX_BODY] for r in regexes]
-        return re.compile(r"|".join(combined), re.S | re.M)
+        with_brackets = [r"(%s)" % r[REGEX_BODY] for r in regexes]
+        combined = r"|".join(with_brackets)
+
+        # In the case where there is only one regex, we need to add extra
+        # bracket around it for it to behave the same as the multiple regex
+        # case
+        if len(regexes) == 1:
+            combined = f"({combined})"
+
+        return re.compile(combined, re.S | re.M)
 
     def remove_numbers_and_time(self, snippet):
-        # [ 1067.461794][  T132] BUG: KCSAN: data-race in do_page_fault spectre_v4_enable_task_mitigation
-        # -> [ .][  T] BUG: KCSAN: data-race in do_page_fault spectre_v_enable_task_mitigation
-        without_numbers = re.sub(r"(0x[a-f0-9]+|[<\[][0-9a-f]+?[>\]]|\d+)", "", snippet)
+        # [   92.236941] CPU: 1 PID: 191 Comm: kunit_try_catch Tainted: G        W         5.15.75-rc1 #1
+        # <4>[   87.925462] CPU: 0 PID: 135 Comm: (crub_all) Not tainted 6.7.0-next-20240111 #14
+        # Remove '(Not t|T)ainted', to the end of the line.
+        without_tainted = re.sub(r"(Not t|T)ainted.*", "", snippet)
+
+        # x23: ffff9b7275bc6f90 x22: ffff9b7275bcfb50 x21: fff00000cc80ef88
+        # x20: 1ffff00010668fb8 x19: ffff8000800879f0 x18: 00000000805c0b5c
+        # Remove words with hex numbers.
+        # <3>[    2.491276][    T1] BUG: KCSAN: data-race in console_emit_next_record / console_trylock_spinning
+        # -> <>[    .][    T1] BUG: KCSAN: data-race in console_emit_next_record / console_trylock_spinning
+        without_hex = re.sub(r"\b(?:0x)?[a-fA-F0-9]+\b", "", without_tainted)
+
+        # <>[ 1067.461794][  T132] BUG: KCSAN: data-race in do_page_fault spectre_v4_enable_task_mitigation
+        # -> <>[ .][  T132] BUG: KCSAN: data-race in do_page_fault spectre_v_enable_task_mitigation
+        # But should not remove numbers from functions.
+        without_numbers = re.sub(
+            r"(0x[a-f0-9]+|[<\[][0-9a-f]+?[>\]]|\b\d+\b(?!\s*\())", "", without_hex
+        )
 
-        # [ .][  T] BUG: KCSAN: data-race in do_page_fault spectre_v_enable_task_mitigation
+        # <>[ .][  T132] BUG: KCSAN: data-race in do_page_fault spectre_v_enable_task_mitigation
         # ->  BUG: KCSAN: data-race in do_page_fault spectre_v_enable_task_mitigation
-        without_time = re.sub(f"^{square_brackets_and_contents}({square_brackets_and_contents})?", "", without_numbers) # noqa
+        without_time = re.sub(
+            f"^<?>?{square_brackets_and_contents}({square_brackets_and_contents})?",
+            "",
+            without_numbers,
+        )  # noqa
 
         return without_time
 
@@ -41,10 +68,7 @@ class BaseLogParser:
         snippet = matches[0]
         without_numbers_and_time = self.remove_numbers_and_time(snippet)
 
-        # Limit the name length to 191 characters, since the max name length
-        # for SuiteMetadata in SQUAD is 256 characters. The SHA and "-" take 65
-        # characters: 256-65=191
-        return slugify(without_numbers_and_time)[:191]
+        return slugify(without_numbers_and_time)
 
     def create_shasum(self, snippet):
         sha = hashlib.sha256()
@@ -52,7 +76,7 @@ class BaseLogParser:
         sha.update(without_numbers_and_time.encode())
         return sha.hexdigest()
 
-    def create_name_log_dict(self, test_name, lines, test_regex=None):
+    def create_name_log_dict(self, test_name, lines, test_regex=None, create_shas=True):
         """
         Produce a dictionary with the test names as keys and the extracted logs
         for that test name as values. There will be at least one test name per
@@ -64,31 +88,43 @@ class BaseLogParser:
         # have any output for a particular regex, just use the default name
         # (for example "check-kernel-oops").
         tests_without_shas_to_create = defaultdict(set)
-        tests_with_shas_to_create = defaultdict(set)
+        tests_with_shas_to_create = None
 
         # If there are lines, then create the tests for these.
         for line in lines:
             extracted_name = self.create_name(line, test_regex)
             if extracted_name:
-                extended_test_name = f"{test_name}-{extracted_name}"
+                max_name_length = 256
+                # If adding SHAs, limit the name length to 191 characters,
+                # since the max name length for SuiteMetadata in SQUAD is 256
+                # characters. The SHA and "-" take 65 characters: 256-65=191
+                if create_shas:
+                    max_name_length -= 65
+                extended_test_name = f"{test_name}-{extracted_name}"[:max_name_length]
             else:
                 extended_test_name = test_name
             tests_without_shas_to_create[extended_test_name].add(line)
 
-        for name, test_lines in tests_without_shas_to_create.items():
-            # Some lines of the matched regex might be the same, and we don't want to create
-            # multiple tests like test1-sha1, test1-sha1, etc, so we'll create a set of sha1sums
-            # then create only new tests for unique sha's
+        if create_shas:
+            tests_with_shas_to_create = defaultdict(set)
+            for name, test_lines in tests_without_shas_to_create.items():
+                # Some lines of the matched regex might be the same, and we don't want to create
+                # multiple tests like test1-sha1, test1-sha1, etc, so we'll create a set of sha1sums
+                # then create only new tests for unique sha's
 
-            for line in test_lines:
-                sha = self.create_shasum(line)
-                name_with_sha = f"{name}-{sha}"
-                tests_with_shas_to_create[name_with_sha].add(line)
+                for line in test_lines:
+                    sha = self.create_shasum(line)
+                    name_with_sha = f"{name}-{sha}"
+                    tests_with_shas_to_create[name_with_sha].add(line)
 
         return tests_without_shas_to_create, tests_with_shas_to_create
 
     def create_squad_tests_from_name_log_dict(
-        self, suite, testrun, tests_without_shas_to_create, tests_with_shas_to_create
+        self,
+        suite_name,
+        testrun,
+        tests_without_shas_to_create,
+        tests_with_shas_to_create=None,
     ):
         # Import SuiteMetadata from SQUAD only when required so BaseLogParser
         # does not require a SQUAD to work. This makes it easier to reuse this
@@ -96,6 +132,8 @@ class BaseLogParser:
         # patterns.
         from squad.core.models import SuiteMetadata
 
+        suite, _ = testrun.build.project.suites.get_or_create(slug=suite_name)
+
         for name, lines in tests_without_shas_to_create.items():
             metadata, _ = SuiteMetadata.objects.get_or_create(
                 suite=suite.slug, name=name, kind="test"
@@ -108,34 +146,72 @@ class BaseLogParser:
                 build=testrun.build,
                 environment=testrun.environment,
             )
-        for name_with_sha, lines in tests_with_shas_to_create.items():
-            metadata, _ = SuiteMetadata.objects.get_or_create(
-                suite=suite.slug, name=name_with_sha, kind="test"
-            )
-            testrun.tests.create(
-                suite=suite,
-                result=False,
-                log="\n---\n".join(lines),
-                metadata=metadata,
-                build=testrun.build,
-                environment=testrun.environment,
-            )
-
-    def create_squad_tests(self, testrun, suite, test_name, lines, test_regex=None):
+        if tests_with_shas_to_create:
+            for name_with_sha, lines in tests_with_shas_to_create.items():
+                metadata, _ = SuiteMetadata.objects.get_or_create(
+                    suite=suite.slug, name=name_with_sha, kind="test"
+                )
+                testrun.tests.create(
+                    suite=suite,
+                    result=False,
+                    log="\n---\n".join(lines),
+                    metadata=metadata,
+                    build=testrun.build,
+                    environment=testrun.environment,
+                )
+
+    def print_squad_tests_from_name_log_dict(
+        self,
+        suite_name,
+        tests_without_shas_to_create,
+        tests_with_shas_to_create=None,
+    ):
+        for name, lines in tests_without_shas_to_create.items():
+            print(f"\nName: {suite_name}/{name}")
+            log = "\n".join(lines)
+            print(f"Log:\n{log}")
+
+        if tests_with_shas_to_create:
+            for name_with_sha, lines in tests_with_shas_to_create.items():
+                print(f"\nName: {suite_name}/{name_with_sha}")
+                log = "\n---\n".join(lines)
+                print(f"Log:\n{log}")
+
+    def create_squad_tests(
+        self,
+        testrun,
+        suite_name,
+        test_name,
+        lines,
+        test_regex=None,
+        create_shas=True,
+        print=False,
+        squad=True,
+    ):
         """
         There will be at least one test per regex. If there were any match for
         a given regex, then a new test will be generated using test_name +
         shasum. This helps comparing kernel logs across different builds
         """
+
         tests_without_shas_to_create, tests_with_shas_to_create = (
-            self.create_name_log_dict(test_name, lines, test_regex)
-        )
-        self.create_squad_tests_from_name_log_dict(
-            suite,
-            testrun,
-            tests_without_shas_to_create,
-            tests_with_shas_to_create,
+            self.create_name_log_dict(
+                test_name, lines, test_regex, create_shas=create_shas
+            )
         )
+        if print:
+            self.print_squad_tests_from_name_log_dict(
+                suite_name,
+                tests_without_shas_to_create,
+                tests_with_shas_to_create,
+            )
+        if squad:
+            self.create_squad_tests_from_name_log_dict(
+                suite_name,
+                testrun,
+                tests_without_shas_to_create,
+                tests_with_shas_to_create,
+            )
 
     def join_matches(self, matches, regexes):
         """

squad/plugins/linux_log_parser.py

@@ -44,8 +44,9 @@ class Plugin(BasePlugin, BaseLogParser):
         kernel_msgs = re.findall(f'({tstamp}{pid}? .*?)$', log, re.S | re.M) # noqa
         return '\n'.join(kernel_msgs)
 
-    def postprocess_testrun(self, testrun):
-        if testrun.log_file is None:
+    def postprocess_testrun(self, testrun, squad=True, print=False):
+        # If running as a SQUAD plugin, only run the boot/test log parser if this is not a build testrun
+        if testrun.log_file is None or (squad and testrun.tests.filter(suite__slug="build").exists()):
             return
 
         boot_log, test_log = self.__cutoff_boot_log(testrun.log_file)
@@ -56,7 +57,7 @@ class Plugin(BasePlugin, BaseLogParser):
 
         for log_type, log in logs.items():
             log = self.__kernel_msgs_only(log)
-            suite, _ = testrun.build.project.suites.get_or_create(slug=f'log-parser-{log_type}')
+            suite_name = f'log-parser-{log_type}'
 
             regex = self.compile_regexes(REGEXES)
             matches = regex.findall(log)
@@ -68,4 +69,4 @@ class Plugin(BasePlugin, BaseLogParser):
                 test_name_regex = None
                 if regex_pattern:
                     test_name_regex = re.compile(regex_pattern, re.S | re.M)
-                self.create_squad_tests(testrun, suite, test_name, snippets[regex_id], test_name_regex)
+                self.create_squad_tests(testrun, suite_name, test_name, snippets[regex_id], test_name_regex, squad=squad, print=print)

squad/plugins/linux_log_parser_build.py

@@ -0,0 +1,334 @@
+import logging
+import re
+
+from django.template.defaultfilters import slugify
+
+from squad.plugins import Plugin as BasePlugin
+from squad.plugins.lib.base_log_parser import (
+    REGEX_EXTRACT_NAME,
+    REGEX_NAME,
+    BaseLogParser,
+)
+
+logger = logging.getLogger()
+
+file_path = r"^(?:[^\n]*?:(?:\d+:){2}|<[^\n]*?>:)"
+gcc_clang_compiler_error_warning = rf"{file_path} (?:error|warning): [^\n]+?\n^(?:\.+\n|^(?!\s+(?:CC|Kernel[^\n]*?is ready))\s+?[^\n]+\n|{file_path} note:[^\n]+\n)*"
+
+MULTILINERS_GCC = [
+    (
+        "gcc-compiler",
+        gcc_clang_compiler_error_warning,
+        r"^[^\n]*(?:error|warning)[^\n]*$",
+    ),
+]
+
+ONELINERS_GCC = []
+
+
+MULTILINERS_CLANG = [
+    (
+        "clang-compiler",
+        gcc_clang_compiler_error_warning,
+        r"^[^\n]*(?:error|warning)[^\n]*$",
+    ),
+]
+
+ONELINERS_CLANG = [
+    (
+        "clang-compiler-single-line",
+        "^clang: (?:error|warning).*?$",
+        r"^[^\n]*(?:error|warning).*?$",
+    ),
+    (
+        "clang-compiler-fatal-error",
+        "^fatal error.*?$",
+        r"^fatal error.*?$",
+    ),
+]
+
+MULTILINERS_GENERAL = [
+    (
+        "general-not-a-git-repo",
+        r"^[^\n]*fatal: not a git repository.*?not set\)\.$",
+        r"^[^\n]*fatal: not a git repository.*?$",
+    ),
+    (
+        "general-unexpected-argument",
+        r"^[^\n]*error: Found argument.*?--help$",
+        r"^[^\n]*error: Found argument.*?$",
+    ),
+    (
+        "general-broken-32-bit",
+        r"^[^\n]*Warning: you seem to have a broken 32-bit build.*?(?:If[^\n]*?try:(?:\n|\s+.+?$)+)+",
+        r"^[^\n]*Warning:.*?$",
+    ),
+    (
+        "general-makefile-overriding",
+        r"^[^\n]*warning: overriding recipe for target.*?ignoring old recipe for target.*?$",
+        r"^[^\n]*warning:.*?$",
+    ),
+    (
+        "general-unmet-dependencies",
+        r"^WARNING: unmet direct dependencies detected for.*?$(?:\n +[^\n]+)*",
+        r"^WARNING: unmet direct dependencies detected for.*?$",
+    ),
+    (
+        "general-ldd",
+        r"^[^\n]*?lld:[^\n]+?(?:warning|error):.*?$(?:\n^>>>[^\n]+)*",
+        r"^[^\n]*?lld:.*?$",
+    ),
+    (
+        "general-ld",
+        r"^[^\n]*?ld:[^\n]+?(?:warning|error):[^\n]*?$(?:\n^[^\n]*?NOTE:[^\n]+)*",
+        r"^[^\n]*?ld:[^\n]+?(?:warning|error):.*?$",
+    ),
+    (
+        "general-objcopy",
+        r"^[^\n]*?objcopy:[^\n]+?(?:warning|error):[^\n]*?$(?:\n^[^\n]*?NOTE:[^\n]+)*",
+        r"^[^\n]*?objcopy:[^\n]+?(?:warning|error):.*?$",
+    ),
+    (
+        "general-ld-undefined-reference",
+        r"^[^\n]*?ld[^\n]*?$\n^[^\n]+undefined reference.*?$",
+        r"^[^\n]+undefined reference.*?$",
+    ),
+    (
+        "general-modpost",
+        r"^[^\n]*?WARNING: modpost:[^\n]*?$(?:\n^To see.*?:$\n^.*?$)?",
+        r"^[^\n]*?WARNING.*?$",
+    ),
+    (
+        "general-python-traceback",
+        r"Traceback.*?^[^\s]+Error: .*?$",
+        r"^[^\s]+Error: .*?$",
+    ),
+]
+
+ONELINERS_GENERAL = [
+    (
+        "general-no-such-file-or-directory",
+        r"^[^\n]+?No such file or directory.*?$",
+        r"^[^\n]+?No such file or directory.*?$",
+    ),
+    (
+        "general-no-targets",
+        r"^[^\n]+?No targets.*?$",
+        r"^[^\n]+?No targets.*?$",
+    ),
+    (
+        "general-no-rule-to-make-target",
+        r"^[^\n]+?No rule to make target.*?$",
+        r"^[^\n]+?No rule to make target.*?$",
+    ),
+    (
+        "general-makefile-config",
+        r"^Makefile.config:\d+:.*?$",
+        r"^Makefile.config:\d+:.*?$",
+    ),
+    (
+        "general-not-found",
+        r"^[^\n]*?not found.*?$",
+        r"^[^\n]*?not found.*?$",
+    ),
+    (
+        "general-kernel-abi",
+        r"^Warning: Kernel ABI header at.*?$",
+        r"^Warning: Kernel ABI header at.*?$",
+    ),
+    (
+        "general-missing",
+        r"^Warning: missing.*?$",
+        r"^Warning: missing.*?$",
+    ),
+    (
+        "general-dtc",
+        r"^[^\n]*?Warning \([^\n]*?\).*?$",
+        r"^[^\n]*?Warning.*?$",
+    ),
+    (
+        "general-register-allocation",
+        r"^[^\n]*?error: register allocation failed.*?$",
+        r"^[^\n]*?error.*?$",
+    ),
+]
+
+# Tip: broader regexes should come first
+REGEXES_GCC = MULTILINERS_GCC + MULTILINERS_GENERAL + ONELINERS_GCC + ONELINERS_GENERAL
+REGEXES_CLANG = (
+    MULTILINERS_CLANG + MULTILINERS_GENERAL + ONELINERS_CLANG + ONELINERS_GENERAL
+)
+
+supported_toolchains = {
+    "gcc": REGEXES_GCC,
+    "clang": REGEXES_CLANG,
+}
+
+make_regex = r"^make .*?$"
+in_file_regex = r"^In file[^\n]*?[:,]$(?:\n^(?:\s+|In file)[^\n]*?[:,]$)*"
+in_function_regex = r"^[^\n]*?In function.*?:$"
+entering_dir_regex = r"^make\[(?:\d+)\]: Entering directory.*?$"
+leaving_dir_regex = r"^make\[(?:\d+)\]: Leaving directory.*?$"
+
+split_regex_gcc = rf"(.*?)({make_regex}|{in_file_regex}|{in_function_regex}|{entering_dir_regex}|{leaving_dir_regex})"
+
+
+class Plugin(BasePlugin, BaseLogParser):
+
+    def post_process_test_name(self, text):
+        # Remove "builds/linux" if there
+        text = re.sub(r"builds/linux", "", text)
+
+        # Change "/" and "." to "_" for readability
+        text = re.sub(r"[/\.]", "_", text)
+
+        # Remove numbers and hex
+        text = re.sub(r"(0x[a-f0-9]+|[<\[][0-9a-f]+?[>\]]|\d+)", "", text)
+
+        # Remove "{...}" and "[...]"
+        text = re.sub(r"\{.+?\}", "", text)
+        text = re.sub(r"\[.+?\]", "", text)
+
+        return text
+
+    def create_name(self, snippet, compiled_regex=None):
+        matches = None
+        if compiled_regex:
+            matches = compiled_regex.findall(snippet)
+        if not matches:
+            # Only extract a name if we provide a regex to extract the name and
+            # there is a match
+            return None
+        snippet = matches[0]
+        without_numbers = re.sub(
+            r"(0x[a-f0-9]+|[<\[][0-9a-f]+?[>\]]|\b\d+\b(?!\s*\())", "", snippet
+        )
+
+        name = slugify(self.post_process_test_name(without_numbers))
+
+        return name
+
+    def split_by_regex(self, log, regex):
+        # Split up the log by the keywords we want to capture
+        s_lines_compiled = re.compile(regex, re.DOTALL | re.MULTILINE)
+        split_by_regex_list = s_lines_compiled.split(log)
+        split_by_regex_list = [
+            f for f in split_by_regex_list if f is not None and f != ""
+        ]
+
+        return split_by_regex_list
+
+    def process_blocks(
+        self,
+        blocks_to_process,
+        regexes,
+        make_regex=make_regex,
+        entering_dir_regex=entering_dir_regex,
+        leaving_dir_regex=leaving_dir_regex,
+        in_file_regex=in_file_regex,
+        in_function_regex=in_function_regex,
+    ):
+        snippets = dict()
+        regex_compiled = self.compile_regexes(regexes)
+        make_regex_compiled = re.compile(make_regex, re.DOTALL | re.MULTILINE)
+        entering_dir_regex_compiled = re.compile(
+            entering_dir_regex, re.DOTALL | re.MULTILINE
+        )
+        leaving_dir_regex_compiled = re.compile(
+            leaving_dir_regex, re.DOTALL | re.MULTILINE
+        )
+        in_file_regex_compiled = re.compile(in_file_regex, re.DOTALL | re.MULTILINE)
+        in_function_regex_compiled = re.compile(
+            in_function_regex, re.DOTALL | re.MULTILINE
+        )
+
+        # For tracking the last piece of information we saw
+        make_command = None
+        entering_dir = None
+        in_file = None
+        in_function = None
+
+        for regex_id in range(len(regexes)):
+            snippets[regex_id] = []
+        for block in blocks_to_process:
+            if make_regex_compiled.match(block):
+                make_command = block
+                entering_dir = None
+                in_file = None
+                in_function = None
+            elif entering_dir_regex_compiled.match(block):
+                entering_dir = block
+                in_file = None
+                in_function = None
+            elif leaving_dir_regex_compiled.match(block):
+                entering_dir = None
+                in_file = None
+                in_function = None
+            elif in_file_regex_compiled.match(block):
+                in_file = block
+                in_function = None
+            elif in_function_regex_compiled.match(block):
+                in_function = block
+            else:
+                matches = regex_compiled.findall(block)
+                sub_snippets = self.join_matches(matches, regexes)
+                prepend = ""
+                if make_command:
+                    prepend += make_command + "\n"
+                if entering_dir:
+                    prepend += entering_dir + "\n"
+                if in_file:
+                    prepend += in_file + "\n"
+                if in_function:
+                    prepend += in_function + "\n"
+                for regex_id in range(len(regexes)):
+                    for s in sub_snippets[regex_id]:
+                        snippets[regex_id].append(prepend + s)
+
+        return snippets
+
+    def postprocess_testrun(self, testrun, squad=True, print=False):
+        """
+        Check:
+            - There is a log file
+            - If running as SQUAD plugin, the testrun contains the "build"
+              suite - this tells us that the testrun's log is a build log
+        """
+        if testrun.log_file is None or (
+            squad and not testrun.tests.filter(suite__slug="build").exists()
+        ):
+            return
+
+        regexes = None
+        for toolchain, toolchain_regexes in supported_toolchains.items():
+            if f"--toolchain={toolchain}" in testrun.log_file:
+                toolchain_name = toolchain
+                regexes = toolchain_regexes
+
+        # If a supported toolchain was not found in the log
+        if regexes is None:
+            return
+
+        # If running in SQUAD, create the suite
+        suite_name = f"log-parser-build-{toolchain_name}"
+
+        blocks_to_process = self.split_by_regex(testrun.log_file, split_regex_gcc)
+
+        snippets = self.process_blocks(blocks_to_process, regexes)
+
+        for regex_id in range(len(regexes)):
+            test_name = regexes[regex_id][REGEX_NAME]
+            regex_pattern = regexes[regex_id][REGEX_EXTRACT_NAME]
+            test_name_regex = None
+            if regex_pattern:
+                test_name_regex = re.compile(regex_pattern, re.S | re.M)
+            self.create_squad_tests(
+                testrun,
+                suite_name,
+                test_name,
+                snippets[regex_id],
+                test_name_regex,
+                create_shas=False,
+                print=print,
+                squad=squad,
+            )

squad/version.py

@@ -1 +1 @@
-__version__ = '1.92'
+__version__ = '1.93.1'

{squad-1.92.dist-info → squad-1.93.1.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: squad
-Version: 1.92
+Version: 1.93.1
 Summary: Software Quality Dashboard
 Home-page: https://github.com/Linaro/squad
 Author: Antonio Terceiro

{squad-1.92.dist-info → squad-1.93.1.dist-info}/RECORD

@@ -10,7 +10,7 @@ squad/manage.py,sha256=Z-LXT67p0R-IzwJ9fLIAacEZmU0VUjqDOSg7j2ZSxJ4,1437
 squad/settings.py,sha256=0MZ48SV_7CTrLMik2ubWf8-ROQiFju6CKnUC3iR8KAc,14800
 squad/socialaccount.py,sha256=vySqPwQ3qVVpahuJ-Snln8K--yzRL3bw4Nx27AsB39A,789
 squad/urls.py,sha256=JiEfVW8YlzLPE52c2aHzdn5kVVKK4o22w8h5KOA6QhQ,2776
-squad/version.py,sha256=WHUX3wrk23jPrWzNUft2dot0QtSqj7kO0c3Q3yKJiDM,21
+squad/version.py,sha256=N1lgYLLJIViE9VAAq7J5l3sFo7_BdyvoOEdr3j9eNaA,23
 squad/wsgi.py,sha256=SF8T0cQ0OPVyuYjO5YXBIQzvSXQHV0M2BTmd4gP1rPs,387
 squad/api/__init__.py,sha256=CJiVakfAlHVN5mIFRVQYZQfuNUhUgWVbsdYTME4tq7U,1349
 squad/api/apps.py,sha256=Trk72p-iV1uGn0o5mdJn5HARUoHGbfgO49jwXvpkmdQ,141
@@ -26,14 +26,14 @@ squad/ci/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 squad/ci/admin.py,sha256=7yB-6F0cvt0NVvzGOTlZCyGPV_YHarmbKJZTTzataT4,2255
 squad/ci/apps.py,sha256=6OVnzTdJkxdqEJnKWYE9dZgUcc29_T1LrDw41cK4EQk,139
 squad/ci/exceptions.py,sha256=a1sccygniTYDSQi7FRn_6doapddFFiMf55AwGUh5Y80,227
-squad/ci/models.py,sha256=Fm-4b3SDgMh9HXzqjOd4iZDRMJ1D9AnZ2cg7i2OR248,16018
+squad/ci/models.py,sha256=wR9FMBdjQgtEP3ga9CY6npFr5fUIeVpnfAhNa2xqM00,15897
 squad/ci/tasks.py,sha256=P0NYjLuyUViTpO1jZMuRVREbFDCccrMCZDw5E4pt928,3882
 squad/ci/utils.py,sha256=38zHpw8xkZDSFlkG-2BwSK6AkcddK9OkN9LXuQ3SHR0,97
 squad/ci/backend/__init__.py,sha256=yhpotXT9F4IdAOXvGQ3-17eOHAFwoaqf9SnMX17ab30,534
 squad/ci/backend/fake.py,sha256=7Rl-JXnBYThDomOBzBsN9XuVkSjSHTZjtZOURdowZbA,2397
 squad/ci/backend/lava.py,sha256=WeOJJNxv42geGf3Y6r-I0WnhWinxpSSgZAFAwfkiXGY,34039
 squad/ci/backend/null.py,sha256=htEd4NbrXLKdPgFfTS0Ixm8PdT6Ghat3BCYi2zjfuv0,5624
-squad/ci/backend/tuxsuite.py,sha256=dTQNgHs5HNrY2LUaSH6jXiBLVjgyMNTqJ6CUWU3yPK4,18914
+squad/ci/backend/tuxsuite.py,sha256=pFcNdcHpFzalHPQhbSY6ryOci_PU3LFsaNjSsgjbqGg,18676
 squad/ci/management/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 squad/ci/management/commands/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 squad/ci/management/commands/create_tuxsuite_boot_tests.py,sha256=JvjNusebLX71eyz9d-kaeCyekYSpzc1eXoeIqWK9ygo,4045
@@ -82,7 +82,7 @@ squad/core/comparison.py,sha256=LR3-Unv0CTmakFCDzF_h8fm2peTJzkv79mQWNau1iwI,2442
 squad/core/data.py,sha256=2zw56v7iYRTUc7wlhuUNgwIIMmK2w84hi-amR9J7EPU,2236
 squad/core/failures.py,sha256=X6lJVghM2fOrd-RfuHeLlezW2pt7owDZ8eX-Kn_Qrt0,918
 squad/core/history.py,sha256=QRSIoDOw6R6vUWMtsPMknsHGM7FaCAeuCYqASCayHTk,3541
-squad/core/models.py,sha256=sXQmgPtl54IZT7rDmJEU3QK6JSPbi0hTUGRsjwL6PIo,60851
+squad/core/models.py,sha256=qSLlxjBwzsZKGoCkPX6T-g48jXg81B1JH3wMXSLLvHQ,61401
 squad/core/notification.py,sha256=rOpO6F63w7_5l9gQgWBBEk-MFBjp7x_hVzoVIVyDze0,10030
 squad/core/plugins.py,sha256=FLgyoXXKnPBYEf2MgHup9M017rHuADHivLhgzmx_cJE,6354
 squad/core/queries.py,sha256=78fhIJZWXIlDryewYAt96beK1VJad66Ufu8cg3dHh4w,7698
@@ -105,6 +105,7 @@ squad/core/management/commands/migrate_test_runs.py,sha256=RHV06tb4gWyv_q-ooC821
 squad/core/management/commands/populate_metric_build_and_environment.py,sha256=DJP9_YLRso0RiERBVsB0GP4-GaiRtJb0rAiUQDfFNQk,3166
 squad/core/management/commands/populate_test_build_and_environment.py,sha256=0yHClC0x_8LSZlvT6Ag0BnipC9Xk-U6lcIaCsqAGEWk,3146
 squad/core/management/commands/prepdump.py,sha256=WM58leVdJj45KhWPw3DGO7vwnNY70ReXrJRSIIzGXkI,518
+squad/core/management/commands/run_log_parser.py,sha256=SeksSD1cnbgl8oRsD3wu12p30_FMw090T6ouQyO4ZsI,1113
 squad/core/management/commands/send-email.py,sha256=wb1o5oKLDyH2ZonnQY-Jw28Y0Mu61OHWP8b1AQGKqbU,1120
 squad/core/management/commands/update_project_statuses.py,sha256=JleCesbVhYOSXr90ntH7s5u9Isknt7EnlX22VC6yI78,2089
 squad/core/management/commands/users.py,sha256=qIp87xRMfKWHymsAft5-gnYajm2mgaiHvVn7z86DCT8,9429
@@ -278,7 +279,7 @@ squad/core/migrations/0167_add_project_datetime.py,sha256=VUBG-qsAhh2f2NXaHOqfX9
 squad/core/migrations/0168_add_group_settings.py,sha256=5UdylfMMNavTL0KXkjPSiEMhSisGWXbhUXQSzfK29Ck,462
 squad/core/migrations/0169_userpreferences.py,sha256=FwYv9RWxMWdQ2lXJMgi-Xc6XBB5Kp-_YTAOr9GVq1To,1098
 squad/core/migrations/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-squad/core/tasks/__init__.py,sha256=pYbEkFzNaat7iQQretRiJQPPF4Sq-5-hBykJYnBM04g,18567
+squad/core/tasks/__init__.py,sha256=wKjyFw0JXiEDY6PEaKx3ureiSNIQFL8lHH4JIOMjlF8,18677
 squad/core/tasks/exceptions.py,sha256=n4cbmJFBdA6KWsGiTbfN9DyYGbJpk0DjR0UneEYw_W0,931
 squad/core/tasks/notification.py,sha256=6ZyTbUQZPITPP-4r9MUON7x-NbwvDBG8YeabM6fsjzA,4915
 squad/core/templates/squad/notification/base.jinja2,sha256=AbtQioEHV5DJBW4Etsu0-DQXd_8tQCnLejzgbDGDW7s,3413
@@ -426,17 +427,18 @@ squad/plugins/__init__.py,sha256=9BSzy2jFIoDpWlhD7odPPrLdW4CC3btBhdFCvB651dM,152
 squad/plugins/example.py,sha256=BKpwd315lHRIuNXJPteibpwfnI6C5eXYHYdFYBtVmsI,89
 squad/plugins/gerrit.py,sha256=CqO2KnFQzu9utr_TQ-sGr1wg3ln0B-bS2-c0_i8T5-c,7009
 squad/plugins/github.py,sha256=pdtLZw_7xNuzkaFvY_zWi0f2rsMlalXjKm7sz0eADz4,2429
-squad/plugins/linux_log_parser.py,sha256=MB8ScFZycq70Rrm7IM_Cw95rMtqVS9w4RhS5HhBSpcE,3292
+squad/plugins/linux_log_parser.py,sha256=HQVreyZLBmLuv-K-MjlN43sQQSkcls4hkUsjJ9_5WfM,3472
+squad/plugins/linux_log_parser_build.py,sha256=42pTj1_inTsiS_-htElNWw5Cod0bxpF8ZAm1qvYVhes,10481
 squad/plugins/lib/__init__.py,sha256=jzazbAvp2_ibblAs0cKZrmo9aR2EL3hKLyRDE008r2I,40
-squad/plugins/lib/base_log_parser.py,sha256=OW6JkZ3PM5RiDkt9UZ7OFFpUIArCxFUaqovynzwBL1Y,6573
+squad/plugins/lib/base_log_parser.py,sha256=Bb3ok6R9_65EYvdWAsm8wcY741duGujTpaDXw1gJ9Yk,9366
 squad/run/__init__.py,sha256=ssE8GPAGFiK6V0WpZYowav6Zqsd63dfDMMYasNa1sQg,1410
 squad/run/__main__.py,sha256=DOl8JOi4Yg7DdtwnUeGqtYBJ6P2k-D2psAEuYOjWr8w,66
 squad/run/listener.py,sha256=jBeOQhPGb4EdIREB1QsCzYuumsfJ-TqJPd3nR-0m59g,200
 squad/run/scheduler.py,sha256=CDJG3q5C0GuQuxwlMOfWTSSJpDdwbR6rzpbJfuA0xuw,277
 squad/run/worker.py,sha256=jtML0h5qKDuSbpJ6_rpWP4MT_rsGA7a24AhwGxBquzk,594
-squad-1.92.dist-info/COPYING,sha256=jOtLnuWt7d5Hsx6XXB2QxzrSe2sWWh3NgMfFRetluQM,35147
-squad-1.92.dist-info/METADATA,sha256=_lF0F4lPQq88zJhPQ7M38zYoiWXjPWDKCOXr10WTKfg,1278
-squad-1.92.dist-info/WHEEL,sha256=eOLhNAGa2EW3wWl_TU484h7q1UNgy0JXjjoqKoxAAQc,92
-squad-1.92.dist-info/entry_points.txt,sha256=J_jG3qnkoOHX4RFNGC0f83eJ4BSvK3pqLFkoF3HWfmA,195
-squad-1.92.dist-info/top_level.txt,sha256=_x9uqE1XppiiytmVTl_qNgpnXus6Gsef69HqfliE7WI,6
-squad-1.92.dist-info/RECORD,,
+squad-1.93.1.dist-info/COPYING,sha256=jOtLnuWt7d5Hsx6XXB2QxzrSe2sWWh3NgMfFRetluQM,35147
+squad-1.93.1.dist-info/METADATA,sha256=BBKV-R_mmv5k6Tujtg0ARX4GSF0LwWdGvbEerx0OHpw,1280
+squad-1.93.1.dist-info/WHEEL,sha256=tZoeGjtWxWRfdplE7E3d45VPlLNQnvbKiYnx7gwAy8A,92
+squad-1.93.1.dist-info/entry_points.txt,sha256=J_jG3qnkoOHX4RFNGC0f83eJ4BSvK3pqLFkoF3HWfmA,195
+squad-1.93.1.dist-info/top_level.txt,sha256=_x9uqE1XppiiytmVTl_qNgpnXus6Gsef69HqfliE7WI,6
+squad-1.93.1.dist-info/RECORD,,

{squad-1.92.dist-info → squad-1.93.1.dist-info}/WHEEL

@@ -1,5 +1,5 @@
 Wheel-Version: 1.0
-Generator: bdist_wheel (0.44.0)
+Generator: bdist_wheel (0.45.1)
 Root-Is-Purelib: true
 Tag: py3-none-any