Skip to content

Fix docker image list behavior for ITs#2231

Merged
nnarayen merged 3 commits intomainfrom
nikhil/fix-integration-tests
Feb 13, 2026
Merged

Fix docker image list behavior for ITs#2231
nnarayen merged 3 commits intomainfrom
nikhil/fix-integration-tests

Conversation

@nnarayen
Copy link
Contributor

@nnarayen nnarayen commented Feb 13, 2026

🚀 What

Context: https://basetenlabs.slack.com/archives/C0A6XLUS3NK/p1771010736066349

Theory is that the docker version in GHA recently updated to V29, which changed the default behavior of which images are exposed via docker image ls.

💻 How

🔬 Testing

IT run: https://github.com/basetenlabs/truss/actions/runs/22001631192

@nnarayen nnarayen force-pushed the nikhil/fix-integration-tests branch from 2a06932 to 3261977 Compare February 13, 2026 20:13
cache: yarn
cache-dependency-path: baseten-performance-client/node_bindings/yarn.lock
architecture: x86
# We use `docker run` directly instead of a third-party action to avoid
Copy link
Contributor Author

Choose a reason for hiding this comment

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

cc @michaelfeil, I haven't fully investigated, but I think docker on GHA runners is slowly updating, and seems like this action bundled an incompatible cli version. This just uses the cli that comes with docker directly, since the action hasn't been updated in a while

@nnarayen nnarayen force-pushed the nikhil/fix-integration-tests branch from 93ea686 to 155ac56 Compare February 13, 2026 20:26
@nnarayen nnarayen force-pushed the nikhil/fix-integration-tests branch from 155ac56 to 977fec3 Compare February 13, 2026 21:00
Copy link
Contributor

@ndeepak-baseten ndeepak-baseten left a comment

Choose a reason for hiding this comment

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

Too bad the GHA doesn't allow us to pin docker. Thanks for the fix!

@nnarayen nnarayen merged commit 3a69383 into main Feb 13, 2026
61 checks passed
@nnarayen nnarayen deleted the nikhil/fix-integration-tests branch February 13, 2026 22:49
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