ci: write artifact collection script, compress using zstd

.github/workflows/build.yml

@@ -176,19 +176,22 @@ jobs:
       - name: Build source and run tests
         run: |
           git config --global --add safe.directory "$PWD"
-          CCACHE_SIZE="400M"
-          CACHE_DIR="/cache"
-          mkdir /output
-          BASE_OUTDIR="/output"
-          BUILD_TARGET="${{ matrix.build_target }}"
+          export ARTIFACT_PATH="/output"
+          export BUILD_TARGET="${{ matrix.build_target }}"
+          export CACHE_DIR="/cache"
+          export CCACHE_SIZE="400M"
           source ./ci/dash/matrix.sh
           ./ci/dash/build_src.sh
           ./ci/dash/test_unittests.sh
+          ./ci/dash/create-artifacts.sh
         shell: bash
 
       - name: Upload build artifacts
         uses: actions/upload-artifact@v4
         with:
-          name: build-artifacts-${{ matrix.build_target }}
+          name: artifacts_${{ matrix.build_target }}
           path: |
-            /output
+            /output/artifacts_${{ matrix.build_target }}.tar.zst
+            /output/artifacts_${{ matrix.build_target }}.tar.zst.sha256
+          compression-level: 0
+          retention-days: 3

ci/dash/create-artifacts.sh

@@ -0,0 +1,87 @@
+#!/usr/bin/env bash
+# Copyright (c) 2024 The Dash Core developers
+# Distributed under the MIT software license, see the accompanying
+# file COPYING or http://www.opensource.org/licenses/mit-license.php.
+
+export LC_ALL=C.UTF-8
+
+set -ex;
+
+if [ -z "${BUILD_TARGET}" ]; then
+    echo "BUILD_TARGET not defined, cannot continue!";
+    exit 1;
+elif [ -z "${ARTIFACT_PATH}" ]; then
+    echo "ARTIFACT_PATH not defined, cannot continue!";
+    exit 1;
+elif  [ ! "$(command -v zstd)" ]; then
+    echo "zstd not found, cannot continue!";
+    exit 1;
+fi
+
+FILE_EXTENSION=""
+if [[ "${BUILD_TARGET}" == "win"* ]]; then
+    FILE_EXTENSION=".exe"
+fi
+
+# The extra comma is so that brace expansion works as expected, otherwise
+# it's just interpreted as literal braces
+ARTIFACT_DIRS=",bin"
+if [[ "${BUILD_TARGET}" == "mac"* ]]; then
+    ARTIFACT_DIRS="${ARTIFACT_DIRS},dist"
+fi
+
+# If there are new binaries, these need to be updated
+BIN_NAMES="d,-cli,-tx"
+if [[ ${BUILD_TARGET} != *"nowallet" ]]; then
+    BIN_NAMES="${BIN_NAMES},-wallet"
+fi
+if [[ ${BUILD_TARGET} == *"multiprocess" ]]; then
+    BIN_NAMES="${BIN_NAMES},-node"
+fi
+
+# Since brace expansion happens _before_ variable substitution, we need to
+# do the variable substitution in this bash instance, then use a fresh bash
+# instance to do the brace expansion, then take the word split output.
+#
+# This needs us to suppress SC2046.
+
+ARTIFACT_BASE_PATH="${ARTIFACT_PATH}/${BUILD_TARGET}"
+# shellcheck disable=SC2046
+mkdir -p $(echo -n $(bash -c "echo ${ARTIFACT_BASE_PATH}/{${ARTIFACT_DIRS}}"))
+
+# We aren't taking binaries from "release" as they're stripped and therefore,
+# impede debugging.
+BUILD_BASE_PATH="build-ci/dashcore-${BUILD_TARGET}/src"
+
+if [ -f "${BUILD_BASE_PATH}/dashd${FILE_EXTENSION}" ]; then
+    # shellcheck disable=SC2046
+    cp $(echo -n $(bash -c "echo ${BUILD_BASE_PATH}/dash{${BIN_NAMES}}${FILE_EXTENSION}")) \
+       "${BUILD_BASE_PATH}/bench/bench_dash${FILE_EXTENSION}" \
+       "${BUILD_BASE_PATH}/test/test_dash${FILE_EXTENSION}" \
+       "${ARTIFACT_BASE_PATH}/bin";
+fi
+
+if [ -f "${BUILD_BASE_PATH}/qt/dash-qt${FILE_EXTENSION}" ]; then
+    cp "${BUILD_BASE_PATH}/qt/dash-qt${FILE_EXTENSION}" \
+       "${BUILD_BASE_PATH}/qt/test/test_dash-qt${FILE_EXTENSION}" \
+       "${ARTIFACT_BASE_PATH}/bin";
+fi
+
+if [ -f "${BUILD_BASE_PATH}/test/fuzz/fuzz${FILE_EXTENSION}" ]; then
+    cp "${BUILD_BASE_PATH}/test/fuzz/fuzz${FILE_EXTENSION}" \
+       "${ARTIFACT_BASE_PATH}/bin";
+fi
+
+# TODO: After bitcoin#28432, it will be Dash-Core.zip. Remember to change this!
+if [[ "${ARTIFACT_DIRS}" == *"dist"* ]] && [[ "${BUILD_TARGET}" == "mac"* ]]; then
+    cp "${BUILD_BASE_PATH}/../Dash-Core.dmg" \
+       "${ARTIFACT_BASE_PATH}/dist"
+fi
+
+ARTIFACT_ARCHIVE="artifacts_${BUILD_TARGET}.tar.zst"
+# We have to cd into ARTIFACT_PATH so that the archive doesn't have the
+# directory structure ARTIFACT_PATH/BUILD_TARGET when we want BUILD_TARGET
+# as the root directory. `tar` offers no easy way to do this.
+cd "${ARTIFACT_PATH}"
+tar -cvf - "${BUILD_TARGET}" | zstd -T"$(nproc)" -5 > "${ARTIFACT_ARCHIVE}"
+sha256sum "${ARTIFACT_ARCHIVE}" > "${ARTIFACT_ARCHIVE}.sha256"

contrib/containers/ci/Dockerfile

@@ -134,7 +134,8 @@ RUN apt-get update && apt-get install $APT_ARGS \
     valgrind \
     wine-stable \
     wine64 \
-    xorriso
+    xorriso \
+    zstd
 
 ARG CPPCHECK_VERSION=2.13.0
 RUN curl -sL "https://github.com/danmar/cppcheck/archive/${CPPCHECK_VERSION}.tar.gz" | tar -xvzf - --directory /tmp/