Skip to content

Commit

Permalink
[gha/docker] remove deprecated indexer build (#10405)
Browse files Browse the repository at this point in the history
* Revert "Revert "[gha/docker] remove deprecated indexer build (#9836)""

This reverts commit 21c1317.

* [gha] remove indexer docker build deps on forge
  • Loading branch information
rustielin authored Oct 14, 2023
1 parent 3da4597 commit f031750
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 26 deletions.
21 changes: 0 additions & 21 deletions .github/workflows/docker-build-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -134,22 +134,6 @@ jobs:
BUILD_ADDL_TESTING_IMAGES: true
TARGET_REGISTRY: ${{ needs.determine-docker-build-metadata.outputs.targetRegistry }}

rust-images-indexer:
needs: [permission-check, determine-docker-build-metadata]
uses: aptos-labs/aptos-core/.github/workflows/workflow-run-docker-rust-build.yaml@main
if: |
github.event_name == 'push' ||
github.event_name == 'workflow_dispatch' ||
contains(github.event.pull_request.labels.*.name, 'CICD:build-indexer-images')
secrets: inherit
with:
GIT_SHA: ${{ needs.determine-docker-build-metadata.outputs.gitSha }}
TARGET_CACHE_ID: ${{ needs.determine-docker-build-metadata.outputs.targetCacheId }}
PROFILE: release
FEATURES: indexer
BUILD_ADDL_TESTING_IMAGES: true
TARGET_REGISTRY: ${{ needs.determine-docker-build-metadata.outputs.targetRegistry }}

rust-images-failpoints:
needs: [permission-check, determine-docker-build-metadata]
uses: aptos-labs/aptos-core/.github/workflows/workflow-run-docker-rust-build.yaml@main
Expand Down Expand Up @@ -263,7 +247,6 @@ jobs:
- permission-check
- determine-docker-build-metadata
- rust-images
- rust-images-indexer
- rust-images-failpoints
- rust-images-performance
- rust-images-consensus-only-perf-test
Expand Down Expand Up @@ -296,7 +279,6 @@ jobs:
- permission-check
- determine-docker-build-metadata
- rust-images
- rust-images-indexer
- rust-images-failpoints
- rust-images-performance
- rust-images-consensus-only-perf-test
Expand Down Expand Up @@ -326,7 +308,6 @@ jobs:
- permission-check
- determine-docker-build-metadata
- rust-images
- rust-images-indexer
- rust-images-failpoints
- rust-images-performance
- rust-images-consensus-only-perf-test
Expand Down Expand Up @@ -355,7 +336,6 @@ jobs:
- permission-check
- determine-docker-build-metadata
- rust-images
- rust-images-indexer
- rust-images-failpoints
- rust-images-performance
- rust-images-consensus-only-perf-test
Expand All @@ -378,7 +358,6 @@ jobs:
- permission-check
- determine-docker-build-metadata
- rust-images
- rust-images-indexer
- rust-images-failpoints
- rust-images-performance
- rust-images-consensus-only-perf-test
Expand Down
5 changes: 3 additions & 2 deletions docker/builder/docker-bake-rust-all.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,9 @@
# You need to execute this from the repository root as working directory
# E.g. docker/docker-bake-rust-all.sh
# If you want to build a specific target only, run:
# docker/docker-bake-rust-all.sh <target>
# E.g. docker/docker-bake-rust-all.sh indexer
# docker/docker-bake-rust-all.sh <target>
# See targets in docker/builder/docker-bake-rust-all.hcl
# e.g. docker/docker-bake-rust-all.sh forge-images

set -ex

Expand Down
3 changes: 0 additions & 3 deletions docker/release-images.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@

const Features = {
Default: "default",
Indexer: "indexer",
};

const IMAGES_TO_RELEASE_ONLY_INTERNAL = ["validator-testing"];
Expand All @@ -40,7 +39,6 @@ const IMAGES_TO_RELEASE = {
],
release: [
Features.Default,
Features.Indexer,
],
},
"validator-testing": {
Expand All @@ -49,7 +47,6 @@ const IMAGES_TO_RELEASE = {
],
release: [
Features.Default,
Features.Indexer,
],
},
faucet: {
Expand Down

0 comments on commit f031750

Please sign in to comment.