Skip to content

[Build]: Update clean-env-dev-kind target in Makefile and fix vllm-sim image version #155

Merged
kfirtoledo merged 2 commits intollm-d:mainfrom
kfirtoledo:dockerfile
Jun 3, 2025
Merged

[Build]: Update clean-env-dev-kind target in Makefile and fix vllm-sim image version #155
kfirtoledo merged 2 commits intollm-d:mainfrom
kfirtoledo:dockerfile

Conversation

@kfirtoledo
Copy link
Collaborator

@kfirtoledo kfirtoledo commented May 29, 2025

This PR includes:

  • Add clean-env-dev-kind target to the makefile
  • Fix vllm-sim image version (use the official one-dev is not compiled from this git)
    That makes the target env-dev-kind run properly.

@kfirtoledo kfirtoledo requested review from elevran and shmuelk May 29, 2025 12:02
@kfirtoledo kfirtoledo linked an issue May 29, 2025 that may be closed by this pull request
@elevran
Copy link
Collaborator

elevran commented May 29, 2025 via email

Signed-off-by: Kfir Toledo <kfir.toledo@ibm.com>
@kfirtoledo kfirtoledo changed the title [Build] Change base image to ubi9-minimal [Build]: Update lean-env-dev-kind target in Makefile and fix vllm-sim image version May 29, 2025
@kfirtoledo
Copy link
Collaborator Author

@elevran PTAL

@elevran elevran changed the title [Build]: Update lean-env-dev-kind target in Makefile and fix vllm-sim image version [Build]: Update clean-env-dev-kind target in Makefile and fix vllm-sim image version May 29, 2025
elevran
elevran previously approved these changes May 30, 2025
Copy link
Collaborator

@elevran elevran left a comment

Choose a reason for hiding this comment

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

probably want to consider setting images based on latest available instead of hard coding defaults. There is an issue tracking that

@kfirtoledo kfirtoledo requested a review from mayabar June 3, 2025 06:06
Signed-off-by: Kfir Toledo <kfir.toledo@ibm.com>
@kfirtoledo
Copy link
Collaborator Author

@shmuelk PTAL

@kfirtoledo kfirtoledo merged commit c6e7690 into llm-d:main Jun 3, 2025
2 checks passed
@kfirtoledo kfirtoledo deleted the dockerfile branch June 3, 2025 12:09
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.

Reduce image size for inference scheduler

3 participants