Skip to content

Skip unused Bazel install in benchmark Docker image - #872

Draft
Matthew Parkinson (mjp41) with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-benchmark-job
Draft

Skip unused Bazel install in benchmark Docker image#872
Matthew Parkinson (mjp41) with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-benchmark-job

Conversation

Copilot AI commented Aug 13, 2026

Copy link
Copy Markdown

The benchmark GitHub Actions job fails while building benchmark/Dockerfile because the pinned mimalloc-bench setup script downloads a Bazel GPG key from a URL that now returns 404.

  • Benchmark image setup

    • Patches the pinned mimalloc-bench script during the Docker build.
    • Removes the standalone aptinstallbazel call before running ./build-bench-env.sh packages.
    • Keeps the pinned mimalloc-bench revision unchanged.
  • Scope of change

    • Bazel is only needed for the tcg benchmark path.
    • This Dockerfile builds and runs the configured benchmark workloads against snmalloc only, so the Bazel install is unused in this job.
RUN sed -i '/^[[:space:]]*aptinstallbazel[[:space:]]*$/d' build-bench-env.sh && \
    if grep -q '^[[:space:]]*aptinstallbazel[[:space:]]*$' build-bench-env.sh; then \
      echo "failed to remove unused Bazel install"; \
      exit 1; \
    fi

Co-authored-by: mjp41 <270363+mjp41@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job benchmark Skip unused Bazel install in benchmark Docker image Aug 13, 2026
…PLAN.md

Co-authored-by: mjp41 <270363+mjp41@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants