Skip to content

Conversation

dougbtv
Copy link
Collaborator

@dougbtv dougbtv commented Sep 22, 2025

Otherwise, falls back to building image and wheel.

DOCKER_IMAGE_OVERRIDE="$IMAGE_TAG"
echo "Using Docker image for LCA commit: $DOCKER_IMAGE_OVERRIDE"
else
DOCKER_IMAGE_OVERRIDE="public.ecr.aws/q9t5s3a7/vllm-ci-postmerge-repo:latest"
Copy link
Collaborator

Choose a reason for hiding this comment

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

we don't need this if we don't enable SKIP_IMAGE_BUILD right

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

good eye got it, another push incoming...

@dougbtv dougbtv force-pushed the bugfix/validate-image-for-commit branch 2 times, most recently from 456a78e to a9e8e80 Compare September 22, 2025 19:22
@dougbtv
Copy link
Collaborator Author

dougbtv commented Sep 22, 2025

test build in: https://buildkite.com/vllm/ci/builds/31858

@dougbtv dougbtv force-pushed the bugfix/validate-image-for-commit branch from a9e8e80 to 3a7f26e Compare September 22, 2025 19:34
Otherwise, falls back to building image and wheel.

Signed-off-by: dougbtv <[email protected]>
@dougbtv dougbtv force-pushed the bugfix/validate-image-for-commit branch from 3a7f26e to 1550cd6 Compare September 22, 2025 19:36
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