docker-shim: fail fast when shared BuildKit builder unavailable#1588
Merged
lukemarsden merged 1 commit intomainfrom Feb 5, 2026
Merged
docker-shim: fail fast when shared BuildKit builder unavailable#1588lukemarsden merged 1 commit intomainfrom
lukemarsden merged 1 commit intomainfrom
Conversation
When /buildkit-cache exists, the shared BuildKit builder is REQUIRED. Previously, the shim would inject cache flags even when the builder wasn't available, causing "Cache export is not supported" errors. Now the shim fails fast with a clear error message when: - /buildkit-cache directory exists (expecting cached builds) - The helix-buildkit container is missing or unreachable - The helix-shared buildx builder cannot be created Error message explains what's wrong and how to fix it: "shared BuildKit container 'helix-buildkit' not found. This container should be started by Hydra. Check that Hydra is running and has set up the BuildKit container." This follows the fail-fast principle: builds fail immediately with actionable errors rather than proceeding with broken cache flags. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
When
/buildkit-cacheexists, the shared BuildKit builder is REQUIRED for docker builds. Previously, the shim would inject cache flags even when the builder wasn't available, causing "Cache export is not supported for the docker driver" errors.Changes
Fail-fast behavior: The shim now fails immediately with a clear error when:
/buildkit-cachedirectory exists (expecting cached builds)helix-buildkitcontainer is missing or unreachablehelix-sharedbuildx builder cannot be createdError message example:
Error before fix
This error was confusing because it didn't explain what was wrong or how to fix it.
Principle
Follows the fail-fast principle: builds fail immediately with actionable error messages rather than proceeding with broken cache configuration.
🤖 Generated with Claude Code