Skip to content

perf: cache conformance image builds - #127

Merged
lucarlig merged 4 commits into
mainfrom
user/luca/conformance-build-cache
Sep 1, 2026
Merged

perf: cache conformance image builds#127
lucarlig merged 4 commits into
mainfrom
user/luca/conformance-build-cache

Conversation

@lucarlig

@lucarlig lucarlig commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • persist the Linux Rust target/ directory across source-mode Docker builds
  • keep the built binary outside the cache mount for the final runtime image
  • exclude Git and integration runtime state from the Docker context
  • pin conformance CI to cf-integration 0.2.1

Normal Linux CI still uses the prebuilt artifact path. The cache accelerates local/source builds without trying to copy an incompatible macOS target/ binary into Linux.

Results

  • warm Docker image build: 0.90s
  • external-dataplane make conformance: 136.35s, down from 364.88s
  • the strict-modern routed registration failure is still printed as a failed scenario and returns nonzero; completed fixture and client results are preserved

Verification

  • actionlint
  • docker buildx build --check --build-context prebuilt=target/debug --file docker/Dockerfile .
  • runtime image --help smoke test
  • make -n conformance
  • make -n conformance-bless
  • end-to-end make conformance with the locally built 0.2.1 CLI

Depends on contextforge-org/contextforge-dev-tools#12. The remaining routed failure is the control-plane strict-modern resource-sync issue tracked in IBM/mcp-context-forge#6485.

Signed-off-by: lucarlig <luca.carlig@ibm.com>
Signed-off-by: lucarlig <luca.carlig@ibm.com>
Signed-off-by: lucarlig <luca.carlig@ibm.com>
@lucarlig

lucarlig commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

/conformance

Signed-off-by: lucarlig <luca.carlig@ibm.com>
@lucarlig
lucarlig requested a review from cafalchio September 1, 2026 14:19

@cafalchio cafalchio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lucarlig
lucarlig merged commit 0608067 into main Sep 1, 2026
9 checks passed
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