Skip to content

fix(ci): restore Docker image builds#223

Merged
MiaoDX merged 1 commit into
mainfrom
fix/docker-ci-build
May 21, 2026
Merged

fix(ci): restore Docker image builds#223
MiaoDX merged 1 commit into
mainfrom
fix/docker-ci-build

Conversation

@MiaoDX
Copy link
Copy Markdown
Owner

@MiaoDX MiaoDX commented May 21, 2026

Summary

  • Replace the removed Debian libgl1-mesa-glx package with libgl1 in the CPU Docker image build.
  • Copy README.md into both Docker metadata install layers so Hatchling can validate readme = "README.md" during editable installs.
  • Add contract tests to keep the Dockerfile metadata assumptions from regressing.

Root Cause

The failing Build CI Docker Image workflow was resolving python:3.12-slim to Debian trixie, where libgl1-mesa-glx has no package candidate. The GPU Dockerfile also installed from pyproject.toml without copying README.md, which breaks Hatchling metadata validation.

Test Coverage

All new behavior is covered by Dockerfile contract tests.

Validation

  • docker run --rm python:3.12-slim ... apt-cache policy ... confirmed libgl1-mesa-glx is unavailable and libgl1 is available.
  • Focused Docker build using the corrected CPU apt layer and metadata copy completed through uv pip install -e ".[demo,dev]".
  • Metadata-only Hatchling editable install check passed with pyproject.toml, README.md, and package stub present.
  • uv run pytest --no-cov tests/contract/test_dockerfiles.py -q passed: 2 passed.
  • uv run ruff check . passed.
  • uv run ruff format --check . passed.
  • uv run mypy src/ passed.
  • MUJOCO_GL=osmesa uv run pytest -q passed: 572 passed, 3 skipped, coverage 91.88%.

Test Plan

  • Docker package availability checked against python:3.12-slim.
  • Focused Docker build validated past the original failing layers.
  • Contract tests, lint, format check, type check, and full pytest passed locally.

Replace the obsolete Debian GL package in the CPU image and copy README.md into metadata-only Docker build contexts so Hatchling editable installs can validate package metadata.

Co-authored-by: Codex <codex@users.noreply.github.com>
@MiaoDX MiaoDX merged commit 12e40ae into main May 21, 2026
12 checks passed
@MiaoDX MiaoDX deleted the fix/docker-ci-build branch May 21, 2026 03:02
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.

1 participant