From 5c68bbf0415b4e437e62491bc236770a56d54b52 Mon Sep 17 00:00:00 2001
From: Gustavo Valverde <g.valverde02@gmail.com>
Date: Tue, 11 Jun 2024 15:05:55 -0400
Subject: [PATCH] fix(build): remove extra build steps

---
 .../sub-ci-integration-tests-gcp.yml          | 21 -------------------
 .../workflows/sub-ci-unit-tests-docker.yml    | 21 -------------------
 2 files changed, 42 deletions(-)

diff --git a/.github/workflows/sub-ci-integration-tests-gcp.yml b/.github/workflows/sub-ci-integration-tests-gcp.yml
index aebcf5fefd4..27f11f1f37d 100644
--- a/.github/workflows/sub-ci-integration-tests-gcp.yml
+++ b/.github/workflows/sub-ci-integration-tests-gcp.yml
@@ -61,27 +61,6 @@ jobs:
     with:
       network: 'Testnet'
 
-  # Build the docker image used by the tests.
-  #
-  # The default network in the Zebra config in the image is mainnet, unless a manually triggered
-  # workflow or repository variable is configured differently. Testnet jobs change that config to
-  # testnet when running the image.
-  build:
-    name: Build CI Docker
-    if: ${{ !startsWith(github.event_name, 'pull') || !github.event.pull_request.head.repo.fork }}
-    uses: ./.github/workflows/sub-build-docker-image.yml
-    with:
-      dockerfile_path: ./docker/Dockerfile
-      dockerfile_target: tests
-      image_name: ${{ vars.CI_IMAGE_NAME }}
-      no_cache: ${{ inputs.no_cache || false }}
-      rust_backtrace: full
-      rust_lib_backtrace: full
-      rust_log: info
-    # This step needs access to Docker Hub secrets to run successfully
-    secrets: inherit
-
-
   # zebrad cached checkpoint state tests
 
   # Regenerate mandatory checkpoint Zebra cached state disks.
diff --git a/.github/workflows/sub-ci-unit-tests-docker.yml b/.github/workflows/sub-ci-unit-tests-docker.yml
index 9893e32fc9c..3f80d24ebbd 100644
--- a/.github/workflows/sub-ci-unit-tests-docker.yml
+++ b/.github/workflows/sub-ci-unit-tests-docker.yml
@@ -37,27 +37,6 @@ env:
 #! The job names in `ci-unit-tests-docker.yml`, `ci-unit-tests-docker.patch.yml` and
 #! `ci-unit-tests-docker.patch-external.yml` must be kept in sync.
 jobs:
-  # Build the docker image used by the tests.
-  #
-  # The default network in the Zebra config in the image is mainnet, unless a manually triggered
-  # workflow or repository variable is configured differently. Testnet jobs change that config to
-  # testnet when running the image.
-  build:
-    name: Build CI Docker
-    # Skip PRs from external repositories, let them pass, and then Mergify will check them
-    if: ${{ !startsWith(github.event_name, 'pull') || !github.event.pull_request.head.repo.fork }}
-    uses: ./.github/workflows/sub-build-docker-image.yml
-    with:
-      dockerfile_path: ./docker/Dockerfile
-      dockerfile_target: tests
-      image_name: ${{ vars.CI_IMAGE_NAME }}
-      no_cache: ${{ inputs.no_cache || false }}
-      rust_backtrace: full
-      rust_lib_backtrace: full
-      rust_log: info
-    # This step needs access to Docker Hub secrets to run successfully
-    secrets: inherit
-
   # Run all the zebra tests, including tests that are ignored by default.
   #
   # - We activate the gRPC feature to avoid recompiling `zebrad`, but we don't actually run any gRPC tests.