scriptworker 63.4.1__tar.gz → 63.4.3__tar.gz

{scriptworker-63.4.1 → scriptworker-63.4.3}/HISTORY.rst

@@ -4,6 +4,23 @@ Change Log
 All notable changes to this project will be documented in this file.
 This project adheres to `Semantic Versioning <http://semver.org/>`__.
 
+63.4.3 - 2026-09-02
+-------------------
+
+Fixed
+~~~~~
+
+- Added 'thunderbird' as a secondary owner for enterprise repos
+
+63.4.2 - 2026-08-12
+-------------------
+
+Fixed
+~~~~~
+
+- Reverted #815 for permanent bustage when uploading logs (#821)
+- Don't raise on mismatched source if no repositories configured (#820)
+
 63.4.1 - 2026-08-11
 -------------------
 

{scriptworker-63.4.1 → scriptworker-63.4.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: scriptworker
-Version: 63.4.1
+Version: 63.4.3
 Summary: Taskcluster scriptworker
 Author-email: Mozilla Release Engineering <releng@mozilla.com>
 License-File: LICENSE

{scriptworker-63.4.1 → scriptworker-63.4.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "scriptworker"
-version = "63.4.1"
+version = "63.4.3"
 description = "Taskcluster scriptworker"
 authors = [
     { name = "Mozilla Release Engineering", email = "releng@mozilla.com" },

{scriptworker-63.4.1 → scriptworker-63.4.3}/src/scriptworker/artifacts.py

@@ -6,7 +6,6 @@ in S3.
 """
 
 import asyncio
-import base64
 import fnmatch
 import gzip
 import logging
@@ -22,15 +21,7 @@ from taskcluster.exceptions import TaskclusterFailure
 from scriptworker.client import validate_artifact_url
 from scriptworker.exceptions import DownloadError, ScriptWorkerRetryException, ScriptWorkerTaskException
 from scriptworker.task import get_decision_task_id, get_run_id, get_task_id
-from scriptworker.utils import (
-    add_enumerable_item_to_dict,
-    download_file,
-    get_hash,
-    get_loggable_url,
-    raise_future_exceptions,
-    retry_async,
-    semaphore_wrapper,
-)
+from scriptworker.utils import add_enumerable_item_to_dict, download_file, get_loggable_url, raise_future_exceptions, retry_async, semaphore_wrapper
 
 log = logging.getLogger(__name__)
 
@@ -172,8 +163,6 @@ async def create_artifact(context, path, target_path, content_type, content_enco
     payload = {"storageType": storage_type, "expires": expires or get_expiration_arrow(context).isoformat(), "contentType": content_type}
     args = [get_task_id(context.claim_task), get_run_id(context.claim_task), target_path, payload]
 
-    content_md5 = await asyncio.to_thread(get_content_md5, path)
-
     tc_response = await context.temp_queue.createArtifact(*args)
     skip_auto_headers = [aiohttp.hdrs.CONTENT_TYPE]
     loggable_url = get_loggable_url(tc_response["putUrl"])
@@ -183,7 +172,7 @@ async def create_artifact(context, path, target_path, content_type, content_enco
             async with context.session.put(
                 tc_response["putUrl"],
                 data=fh,
-                headers=_craft_artifact_put_headers(content_type, content_md5, content_encoding),
+                headers=_craft_artifact_put_headers(content_type, content_encoding),
                 skip_auto_headers=skip_auto_headers,
                 compress=False,
             ) as resp:
@@ -191,12 +180,6 @@ async def create_artifact(context, path, target_path, content_type, content_enco
                 response_text = await resp.text()
                 log.info(response_text)
                 if resp.status not in (200, 204):
-                    if content_md5 is not None and "BadDigest" in (response_text or ""):
-                        log.error(
-                            "{} was corrupted in transit: the storage backend rejected the body as not matching Content-MD5 {}. Retrying.".format(
-                                target_path, content_md5
-                            )
-                        )
                     raise ScriptWorkerRetryException("Bad status {}".format(resp.status))
 
 
@@ -223,22 +206,9 @@ async def create_link_artifact(context, target_path, link_to, content_type, expi
     await context.temp_queue.createArtifact(*args)
 
 
-def get_content_md5(path):
-    """Get the base64-encoded md5 digest of a file, formatted for ``Content-MD5``.
-
-    Args:
-        path (str): the path to the file to digest.
-
-    Returns:
-        str: the base64-encoded md5 digest.
-
-    """
-    return base64.b64encode(bytes.fromhex(get_hash(path, hash_alg="md5"))).decode("ascii")
-
-
-def _craft_artifact_put_headers(content_type, content_md5, encoding=None):
+def _craft_artifact_put_headers(content_type, encoding=None):
     log.debug("{} {}".format(content_type, encoding))
-    headers = {aiohttp.hdrs.CONTENT_TYPE: content_type, aiohttp.hdrs.CONTENT_MD5: content_md5}
+    headers = {aiohttp.hdrs.CONTENT_TYPE: content_type}
 
     if encoding is not None:
         headers[aiohttp.hdrs.CONTENT_ENCODING] = encoding

{scriptworker-63.4.1 → scriptworker-63.4.3}/src/scriptworker/constants.py

@@ -335,7 +335,10 @@ DEFAULT_CONFIG: immutabledict[str, Any] = immutabledict(
                                 {
                                     "schemes": ("https", "ssh"),
                                     "netlocs": ("github.com",),
-                                    "path_regexes": (r"^(?P<path>/mozilla/enterprise-firefox)(/|.git|$)",),
+                                    "path_regexes": (
+                                        r"^(?P<path>/mozilla/enterprise-firefox)(/|.git|$)",
+                                        r"^(?P<path>/thunderbird/thunderbird-desktop)(/|.git|$)",
+                                    ),
                                 }
                             ),
                         ),
@@ -500,7 +503,10 @@ DEFAULT_CONFIG: immutabledict[str, Any] = immutabledict(
                     {
                         "adhoc": "mozilla-releng",
                         "app-services": "mozilla",
-                        "enterprise": "mozilla",
+                        "enterprise": (
+                            "mozilla",
+                            "thunderbird",
+                        ),
                         "firefox": "",
                         "glean": "mozilla",
                         "mobile": "mozilla-mobile",
@@ -650,7 +656,14 @@ DEFAULT_CONFIG: immutabledict[str, Any] = immutabledict(
                             }
                         ),
                         "app-services": immutabledict({"app-services-repo": ("/mozilla/application-services",)}),
-                        "enterprise": immutabledict({"enterprise-firefox-repo": ("/mozilla/enterprise-firefox",)}),
+                        "enterprise": immutabledict(
+                            {
+                                "enterprise-firefox-repo": (
+                                    "/mozilla/enterprise-firefox",
+                                    "/thunderbird/thunderbird-desktop",
+                                )
+                            }
+                        ),
                         "firefox": immutabledict(
                             {
                                 # Which repos can perform release actions?

{scriptworker-63.4.1 → scriptworker-63.4.3}/src/scriptworker/cot/verify.py

@@ -1965,7 +1965,11 @@ def get_source_url(obj):
         comm_repo = get_repo(obj.task, source_env_prefix="COMM", repo_type="head")
         match_comm_repo = comm_repo and verify_repo_matches_url(comm_repo, source)
 
-        if not match_repo and not match_comm_repo:
+        # Enterprise runs two Decision tasks, a gecko one and a comm one. If
+        # the task has COMM_HEAD_REPOSITORY defined, it comes from the comm
+        # Decision task and its metadata.source must match the comm repository.
+        # Otherwise it must match the enterprise-firefox repository.
+        if (comm_repo and not match_comm_repo) or (not comm_repo and repo and not match_repo):
             raise CoTError(
                 "{name} {task_id}: {source_env_prefix} {repo} or {comm_repo} doesn't match source {source}!".format(
                     name=obj.name, task_id=obj.task_id, source_env_prefix=source_env_prefix, repo=repo, comm_repo=comm_repo, source=source

{scriptworker-63.4.1 → scriptworker-63.4.3}/src/scriptworker/github.py

@@ -296,6 +296,8 @@ def is_github_repo_owner_the_official_one(context, repo_owner):
             'This worker does not have a defined owner for official GitHub repositories. Given "official_github_repos_owner": {}'.format(official_repo_owner)
         )
 
+    elif isinstance(official_repo_owner, tuple):
+        return repo_owner in official_repo_owner
     return official_repo_owner == repo_owner