Skip to content

fix: fix Go formatting and flaky torch_baseimage_fallback integration test#2817

Merged
michaeldwan merged 2 commits intomainfrom
md/fix-main
Mar 9, 2026
Merged

fix: fix Go formatting and flaky torch_baseimage_fallback integration test#2817
michaeldwan merged 2 commits intomainfrom
md/fix-main

Conversation

@michaeldwan
Copy link
Member

@michaeldwan michaeldwan commented Mar 6, 2026

Summary

  • Removes trailing whitespace in integration-tests/harness/harness.go that was failing the "Format Go" CI job.
  • Converts torch_baseimage_fallback integration test from a real cog build (which failed because pip install torch==1.13.0 is no longer reliable) to cog debug, which just verifies the generated Dockerfile text falls back to nvidia/cuda instead of cog-base. Uses an empty local registry for deterministic behavior across environments.

…f cog build

The test was doing a real Docker build with torch==1.13.0 which failed
because pip can no longer reliably install this old package. Since the
test's purpose is to verify Dockerfile generation falls back to
nvidia/cuda (not cog-base), use cog debug to check the generated
Dockerfile text. Uses an empty local registry for deterministic behavior.
@michaeldwan michaeldwan changed the title fix: remove trailing whitespace in integration test harness fix: fix Go formatting and flaky torch_baseimage_fallback integration test Mar 9, 2026
@michaeldwan michaeldwan marked this pull request as ready for review March 9, 2026 16:01
@michaeldwan michaeldwan requested a review from a team as a code owner March 9, 2026 16:01
@michaeldwan michaeldwan merged commit 65da700 into main Mar 9, 2026
31 checks passed
@michaeldwan michaeldwan deleted the md/fix-main branch March 9, 2026 16:07
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