andygrove opened a new pull request, #1756: URL: https://github.com/apache/datafusion-ballista/pull/1756
# Which issue does this PR close? Closes #. # Rationale for this change The Docker workflow currently has no path filtering, so every push and PR triggers a full Rust build and rebuild of all three Ballista docker images, even for changes that cannot affect the image contents (docs, Python bindings, other workflows, dev tooling). This is one of the heavier workflows in the repo and burns CI minutes for nothing on those PRs. # What changes are included in this PR? Adds a `paths` allow list to both the `push` and `pull_request` triggers in `.github/workflows/docker.yml`, scoped to what the build actually consumes: - `ballista/**`, `ballista-cli/**`, `benchmarks/**` — the workspace crates baked into the images - Root manifests: `Cargo.toml`, `Cargo.lock` - Pinned toolchain: `rust-toolchain.toml` - `dev/docker/**` — Dockerfiles and entrypoint scripts - `dev/build-ballista-docker.sh` — the build driver - `.github/workflows/docker.yml` — the workflow itself # Are there any user-facing changes? No. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
