Merge pull request #1023 from Altinity/builds/25.6/add_build_report_artifact

25.6 Add build report artifact

Author: MyroTk

.github/workflows/backport_branches.yml

@@ -265,6 +265,12 @@ jobs:
             python3 -m praktika run 'Build (amd_release)' --workflow "BackportPR" --ci |& tee ./ci/tmp/job.log
           fi
 
+      - name: Upload artifact build_report_package_release
+        uses: actions/upload-artifact@v4
+        with:
+          name: build_report_package_release
+          path: ./ci/tmp/artifact_report_build_amd_release.json
+
   build_amd_asan:
     runs-on: [self-hosted, altinity-on-demand, altinity-builder]
     needs: [config_workflow, dockers_build_amd, dockers_build_arm]

.github/workflows/master.yml

@@ -308,6 +308,12 @@ jobs:
             python3 -m praktika run 'Build (amd_release)' --workflow "MasterCI" --ci |& tee ./ci/tmp/job.log
           fi
 
+      - name: Upload artifact build_report_package_release
+        uses: actions/upload-artifact@v4
+        with:
+          name: build_report_package_release
+          path: ./ci/tmp/artifact_report_build_amd_release.json
+
   build_amd_asan:
     runs-on: [self-hosted, altinity-on-demand, altinity-builder]
     needs: [config_workflow, dockers_build_amd, dockers_build_arm, dockers_build_multiplatform_manifest]

.github/workflows/pull_request.yml

@@ -265,6 +265,12 @@ jobs:
             python3 -m praktika run 'Build (amd_release)' --workflow "PR" --ci |& tee ./ci/tmp/job.log
           fi
 
+      - name: Upload artifact build_report_package_release
+        uses: actions/upload-artifact@v4
+        with:
+          name: build_report_package_release
+          path: ./ci/tmp/artifact_report_build_amd_release.json
+
   build_amd_asan:
     runs-on: [self-hosted, altinity-on-demand, altinity-builder]
     needs: [config_workflow, dockers_build_amd, dockers_build_arm]

.github/workflows/release_branches.yml

@@ -263,6 +263,12 @@ jobs:
             python3 -m praktika run 'Build (amd_release)' --workflow "ReleaseBranchCI" --ci |& tee ./ci/tmp/job.log
           fi
 
+      - name: Upload artifact build_report_package_release
+        uses: actions/upload-artifact@v4
+        with:
+          name: build_report_package_release
+          path: ./ci/tmp/artifact_report_build_amd_release.json
+
   build_amd_asan:
     runs-on: [self-hosted, altinity-on-demand, altinity-builder]
     needs: [config_workflow, dockers_build_amd, dockers_build_arm]

ci/defs/defs.py

@@ -397,6 +397,8 @@ class ArtifactNames:
     FUZZERS = "FUZZERS"
     FUZZERS_CORPUS = "FUZZERS_CORPUS"
 
+    BUILD_REPORT_AMD_RELEASE = "build_report_package_release"
+
     PERF_REPORTS_AMD_1 = "PERF_REPORTS_AMD_1"
     PERF_REPORTS_AMD_2 = "PERF_REPORTS_AMD_2"
     PERF_REPORTS_AMD_3 = "PERF_REPORTS_AMD_3"
@@ -505,6 +507,11 @@ class ArtifactConfigs:
         type=Artifact.Type.S3,
         path=f"{TEMP_DIR}/build/programs/*_seed_corpus.zip",
     )
+    build_reports = Artifact.Config(
+        name=ArtifactNames.BUILD_REPORT_AMD_RELEASE,
+        type=Artifact.Type.GH,
+        path="./ci/tmp/artifact_report_build_amd_release.json"
+    )
     performance_reports = Artifact.Config(
         name="*",
         type=Artifact.Type.S3,

ci/defs/job_configs.py

@@ -180,6 +180,7 @@ class JobConfigs:
                 ArtifactNames.DEB_AMD_RELEASE,
                 ArtifactNames.RPM_AMD_RELEASE,
                 ArtifactNames.TGZ_AMD_RELEASE,
+                ArtifactNames.BUILD_REPORT_AMD_RELEASE,
             ],
             [
                 ArtifactNames.CH_AMD_ASAN,

ci/praktika/validator.py

@@ -160,11 +160,12 @@ def validate(cls):
                     in Settings.S3_BUCKET_TO_HTTP_ENDPOINT
                 ), f"S3_BUCKET_TO_HTTP_ENDPOINT Setting must include bucket name [{Settings.HTML_S3_PATH}] from HTML_S3_PATH, workflow [{workflow.name}]"
 
-            if workflow.enable_cache:
-                for artifact in workflow.artifacts or []:
-                    assert (
-                        artifact.is_s3_artifact()
-                    ), f"All artifacts must be of S3 type if enable_cache|enable_html=True, artifact [{artifact.name}], type [{artifact.type}], workflow [{workflow.name}]"
+            # NOTE(myrotk): the only GH artifact is build_reports and it is necessary
+            # if workflow.enable_cache:
+            #     for artifact in workflow.artifacts or []:
+            #         assert (
+            #             artifact.is_s3_artifact()
+            #         ), f"All artifacts must be of S3 type if enable_cache|enable_html=True, artifact [{artifact.name}], type [{artifact.type}], workflow [{workflow.name}]"
 
             if workflow.dockers and not workflow.disable_dockers_build:
                 assert (

ci/workflows/backport_branches.py

@@ -46,6 +46,7 @@
         *ArtifactConfigs.clickhouse_debians,
         *ArtifactConfigs.clickhouse_rpms,
         *ArtifactConfigs.clickhouse_tgzs,
+        ArtifactConfigs.build_reports,
     ],
     dockers=DOCKERS,
     secrets=SECRETS,

ci/workflows/master.py

@@ -49,6 +49,7 @@
         *ArtifactConfigs.clickhouse_tgzs,
         ArtifactConfigs.fuzzers,
         ArtifactConfigs.fuzzers_corpus,
+        ArtifactConfigs.build_reports,
         *ArtifactConfigs.performance_reports,
     ],
     dockers=DOCKERS,

ci/workflows/pull_request.py

@@ -68,6 +68,7 @@
         *ArtifactConfigs.clickhouse_tgzs,
         ArtifactConfigs.fuzzers,
         ArtifactConfigs.fuzzers_corpus,
+        ArtifactConfigs.build_reports,
         # *ArtifactConfigs.performance_reports,
     ],
     dockers=DOCKERS,