Expose the copr build-id to testing-farm (#2857)

Expose the copr build-id to testing-farm

TODO:

 Write new tests or update the old ones to cover new functionality.
 Update or write new documentation in packit/packit.dev.



Related to packit/packit.dev#1068

RELEASE NOTES BEGIN
The copr build id is now exposed as PACKIT_COPR_BUILD_ID in the testing-farm tests.
RELEASE NOTES END

Reviewed-by: gemini-code-assist[bot]
Reviewed-by: Nikola Forró
Reviewed-by: Cristian Le <[email protected]>
Reviewed-by: Matej Focko

Author: softwarefactory-project-zuul[bot]

packit_service/worker/helpers/testing_farm.py

@@ -483,8 +483,9 @@ def _payload(
             else:
                 nvr = None
             packit_copr_projects.append(f"{build.owner}/{build.project_name}")
+            build_id = int(build.build_id)
         else:
-            build_log_url = nvr = srpm_url = None
+            build_log_url = nvr = srpm_url = build_id = None
 
         if additional_build is not None:
             packit_copr_projects.append(
@@ -528,6 +529,7 @@ def _payload(
             "PACKIT_COPR_PROJECT": (
                 " ".join(packit_copr_projects) if packit_copr_projects else None
             ),
+            "PACKIT_COPR_BUILD_ID": build_id,
             "PACKIT_COPR_RPMS": (" ".join(packit_copr_rpms) if packit_copr_rpms else None),
         }
         predefined_environment = {k: v for k, v in predefined_environment.items() if v is not None}

tests/integration/test_listen_to_fedmsg.py

@@ -711,6 +711,7 @@ def test_copr_build_end_testing_farm(copr_build_end, copr_build_pr):
                     "PACKIT_TARGET_URL": "https://github.com/target/bar",
                     "PACKIT_PR_ID": 24,
                     "PACKIT_COPR_PROJECT": "some-owner/some-project",
+                    "PACKIT_COPR_BUILD_ID": 1,
                     "PACKIT_COPR_RPMS": "hello-world-0.1-1.noarch",
                 },
             },

tests/integration/test_pr_comment.py

@@ -2146,6 +2146,7 @@ def test_pr_test_command_handler_multiple_builds(
                     "PACKIT_TARGET_URL": "https://github.com/packit-service/hello-world",
                     "PACKIT_PR_ID": 9,
                     "PACKIT_COPR_PROJECT": "mf/tree another-owner/another-repo",
+                    "PACKIT_COPR_BUILD_ID": 123456,
                     "PACKIT_COPR_RPMS": "repo-0:0.1-1.noarch another-repo-0:0.1-1.noarch",
                 },
                 "artifacts": [

tests/unit/test_testing_farm.py

@@ -959,8 +959,10 @@ def test_payload(
     # URLs shortened for clarity
     log_url = "https://copr-be.cloud.fedoraproject.org/results/.../builder-live.log"
     srpm_url = f"https://download.copr.fedorainfracloud.org/results/.../{repo}-0.1-1.src.rpm"
+    copr_build_id = "42"
     copr_build = flexmock(
-        id=build_id,
+        id=int(build_id),
+        build_id=copr_build_id,
         built_packages=[
             {
                 "name": repo,
@@ -1022,6 +1024,7 @@ def test_payload(
                 "PACKIT_TARGET_URL": "https://github.com/packit/packit",
                 "PACKIT_PR_ID": 123,
                 "PACKIT_COPR_PROJECT": "builder/some_package",
+                "PACKIT_COPR_BUILD_ID": int(copr_build_id),
                 "MY_ENV_VARIABLE": "my-value",
             },
         },
@@ -1410,11 +1413,13 @@ def test_test_repo(
     flexmock(TFClient).should_receive("distro2compose").and_return(compose)
 
     build_id = 1
+    copr_build_id = "22"
     # URLs shortened for clarity
     log_url = "https://copr-be.cloud.fedoraproject.org/results/.../builder-live.log"
     srpm_url = f"https://download.copr.fedorainfracloud.org/results/.../{repo}-0.1-1.src.rpm"
     copr_build = flexmock(
         id=build_id,
+        build_id=copr_build_id,
         built_packages=[
             {
                 "name": repo,