Skip to content

test: make Docker suites hermetic#438

Draft
sbalabanov wants to merge 1 commit into
mainfrom
e2e-hermetic
Draft

test: make Docker suites hermetic#438
sbalabanov wants to merge 1 commit into
mainfrom
e2e-hermetic

Conversation

@sbalabanov

Copy link
Copy Markdown
Contributor

Summary

  • cross-compile container service binaries through Bazel and expose explicit Docker test-context filegroups
  • resolve Compose files only from declared runfiles and stage minimal temporary Docker build contexts
  • remove repository-root discovery plus MODULE.bazel / go.mod marker data dependencies
  • make Docker test targets declare network access and remove pre-build Make prerequisites
  • derive test image names from declared input contents to keep caching deterministic

Sandbox evaluation

The Docker suites already run under Bazel's sandboxed strategy (processwrapper-sandbox locally). I did not enable Bazel's experimental hermetic Linux sandbox globally: it omits the host root and would require nonportable mounts for the Docker CLI, plugin, socket, shared libraries, and certificate/config paths. The tests now get source inputs only through Bazel runfiles while retaining the host Docker boundary they inherently require.

Validation

  • make build
  • make test
  • make integration-test
  • make e2e-test
  • make lint
  • make check-tidy
  • make check-gazelle

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@sbalabanov
sbalabanov marked this pull request as ready for review July 23, 2026 23:51
@sbalabanov
sbalabanov requested review from a team and behinddwalls as code owners July 23, 2026 23:51
@sbalabanov
sbalabanov marked this pull request as draft July 23, 2026 23:53
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.

4 participants