Skip to content

ci: remove sandbox docker build from publish and e2e workflows#275

Merged
drew merged 6 commits intomainfrom
remove-publish-sandbox-action
Mar 13, 2026
Merged

ci: remove sandbox docker build from publish and e2e workflows#275
drew merged 6 commits intomainfrom
remove-publish-sandbox-action

Conversation

@drew
Copy link
Copy Markdown
Collaborator

@drew drew commented Mar 13, 2026

Summary

  • Remove the build-sandbox job from publish.yml and e2e.yml workflows
  • Remove build-sandbox from needs arrays in downstream jobs (tag-ghcr-latest, build-python-wheels, e2e)
  • Remove sandbox from the GHCR latest-tagging loop in publish.yml
  • Update docker-build.yml component description to reflect only gateway and cluster

Sandbox images are maintained in the community repo and not built from this repository.

Sandbox images are maintained in the community repo and not built here.
Remove the build-sandbox job and all downstream references from
publish.yml, e2e.yml, and docker-build.yml.
@drew drew added the test:e2e Requires end-to-end coverage label Mar 13, 2026
@drew drew self-assigned this Mar 13, 2026
The hardcoded /app/.venv/bin/python path was specific to the old
repo-built sandbox image. Community sandbox images install Python at
standard system paths, so resolve via PATH instead. Update e2e policy
tests and docs to use /usr/bin/python* glob for binary matching.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 13, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-03-13 07:57 UTC

drew added 4 commits March 12, 2026 23:32
The community base sandbox image installs Python via uv at
/sandbox/.uv/python/**/python*, not /usr/bin/python*. Update policy
binary globs in e2e tests and docs to match the actual resolved path.
Update Python version across the project:
- mise.toml: 3.12.12 -> 3.13.12
- pyproject.toml: requires-python >= 3.13
- Dockerfiles, CI workflows, examples, and docs
- Regenerate uv.lock for Python 3.13

Add Authority Key Identifier extension to L7 TLS leaf certificates
to satisfy Python 3.13's stricter ssl module certificate validation.
The published wheel must support Python >=3.12 even though the dev
toolchain and sandbox runtime now use 3.13. Revert wheel build
Dockerfiles to Python 3.12, restore requires-python >= 3.12 in
pyproject.toml, and regenerate uv.lock accordingly.
The --bootstrap flag has hide = true in the CLI definition (it is the
default behaviour), so it correctly does not appear in --help output.
Remove it from the expected flags list in the smoke test.
@drew drew merged commit d98c928 into main Mar 13, 2026
10 checks passed
@drew drew deleted the remove-publish-sandbox-action branch March 13, 2026 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:e2e Requires end-to-end coverage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant