PROD-3069: Move build logic to reusable workflow

Since cross compiling requires a couple of extra setup, we decided to
move the build logic into a reusable workflow so there's only one way of
doing things regarding CI.

Signed-off-by: Jonnatan Jossemar Cordero <[email protected]>

Author: jossemargt-cto-ai

.github/workflows/main.yml

@@ -14,11 +14,6 @@ on:
     tags-ignore:
       - '**'
 
-env:
-  DOCKER_DRIVER: overlay
-  CARGO_NET_GIT_FETCH_WITH_CLI: true
-  CARGO_TARGET_AARCH64_UNKNOWN_LINUX_MUSL_LINKER: aarch64-linux-gnu-gcc
-
 jobs:
   test:
     runs-on: ubuntu-latest
@@ -34,38 +29,8 @@ jobs:
           cargo test
 
   build:
-    runs-on: ubuntu-latest
-    container: rust:1.72
     needs: test
-    steps:
-      - uses: actions/checkout@v3
-      - name: Setup
-        run: |
-          rustup target add x86_64-unknown-linux-musl
-          rustup target add aarch64-unknown-linux-musl
-
-          apt-get update
-          apt-get install -y musl-tools musl-dev
-
-          if [ '${{ contains(inputs.arch, 'arm64') }}' = 'true' ]; then
-            apt-get install -y gcc-aarch64-linux-gnu
-          fi
-
-      - name: Build & package amd64
-        if: ${{ contains(inputs.arch, 'amd64') }}
-        run: |
-          cargo build --release --target x86_64-unknown-linux-musl
-          ./scripts/package ./dist ./target/x86_64-unknown-linux-musl/release
-
-      - name: Build & package arm64
-        if: ${{ contains(inputs.arch, 'arm64') }}
-        run: |
-          cargo build --release --target aarch64-unknown-linux-musl
-          ./scripts/package ./dist ./target/aarch64-unknown-linux-musl/release
-
-      - name: Archive
-        uses: actions/upload-artifact@v3
-        with:
-          name: package
-          retention-days: 1
-          path: dist/*.tar.gz
+    uses: ./.github/workflows/wc_build_binaries.yml
+    with:
+      arch: ${{ inputs.arch }}
+      artifact_retention: 1

.github/workflows/release.yml

@@ -1,9 +1,6 @@
 name: Release
 on:
-  push:
-    tags:
-      - "v*"
-  # Workflow dispatch should be used only as fallback method
+  # Remember to select a Git tag otherwise the workflow will fail
   workflow_dispatch:
     inputs:
       arch:
@@ -20,7 +17,7 @@ env:
   CARGO_NET_GIT_FETCH_WITH_CLI: true
 
 jobs:
-  build:
+  check:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
@@ -29,28 +26,24 @@ jobs:
           VERSION_IN_CARGO=$(find . -type f -name 'Cargo.toml' -exec grep -oP '^\W*version\W*\K[0-9\.]+' {} \; | sort -nru | head -1)
           GIT_TAG='${{ github.ref_name }}'
 
-          if [ "${GIT_TAG/#v/}" != "${VERSION_IN_CARGO}" ]; then
-            echo 'git tag and Cargo.toml version mismatch'
-            echo "${GIT_TAG/#v/} != ${VERSION_IN_CARGO}"
+          if [[ "${GIT_TAG/#v/}" != "${VERSION_IN_CARGO}" ]]; then
+            echo 'Version mismatch between git tag and Cargo.toml'
+              echo "git:   ${GIT_TAG/#v/}"
+              echo "cargo: ${VERSION_IN_CARGO}"
             echo ''
-            echo 'REMINDER: Git tags must start with "v"'
+            echo 'REMINDERS:'
+            echo '  - This workflow must be triggered from a Git tag'
+            echo '  - Git tags must start with "v"'
             exit 1
           fi
-      - name: Setup
-        run: |
-          ./scripts/build-setup
-      - name: Build
-        run: |
-          ./scripts/build ${{ inputs.arch }}
-      - name: Package
-        run: |
-          ./scripts/package
-      - name: Archive
-        uses: actions/upload-artifact@v3
-        with:
-          name: package
-          retention-days: 7
-          path: dist/*.tar.gz
+        shell: bash
+
+  build:
+    needs: check
+    uses: ./.github/workflows/wc_build_binaries.yml
+    with:
+      arch: ${{ inputs.arch }}
+      artifact_retention: 7
 
   release:
     runs-on: ubuntu-latest
@@ -64,5 +57,6 @@ jobs:
       - name: Publish release
         uses: softprops/action-gh-release@v1
         with:
+          prerelease: true
           files: |
             dist/*.tar.gz

.github/workflows/wc_build_binaries.yml

@@ -0,0 +1,54 @@
+name: build binaries
+on:
+  workflow_call:
+    inputs:
+      arch:
+        required: false
+        type: string
+        default: amd64
+      artifact_retention:
+        required: false
+        type: number
+        default: 1
+
+env:
+  DOCKER_DRIVER: overlay
+  CARGO_NET_GIT_FETCH_WITH_CLI: true
+  CARGO_TARGET_AARCH64_UNKNOWN_LINUX_MUSL_LINKER: aarch64-linux-gnu-gcc
+
+jobs:
+  binaries:
+    runs-on: ubuntu-latest
+    container: rust:1.72
+    steps:
+      - uses: actions/checkout@v3
+      - name: Setup
+        run: |
+          rustup target add x86_64-unknown-linux-musl
+          rustup target add aarch64-unknown-linux-musl
+
+          apt-get update
+          apt-get install -y musl-tools musl-dev
+
+          if [ '${{ contains(inputs.arch, 'arm64') }}' = 'true' ]; then
+            apt-get install -y gcc-aarch64-linux-gnu
+          fi
+
+      - name: Build & package amd64
+        if: ${{ contains(inputs.arch, 'amd64') }}
+        run: |
+          cargo build --release --target x86_64-unknown-linux-musl
+          ./scripts/package ./target/x86_64-unknown-linux-musl/release
+
+      - name: Build & package arm64
+        if: ${{ contains(inputs.arch, 'arm64') }}
+        run: |
+          cargo build --release --target aarch64-unknown-linux-musl
+          ./scripts/package ./target/aarch64-unknown-linux-musl/release
+
+      - name: Archive
+        uses: actions/upload-artifact@v3
+        with:
+          name: package
+          retention-days: ${{ inputs.artifact_retention }}
+          path: dist/*.tar.gz