Skip to content

add tokenizer directory to simulator deployment yaml#307

Merged
github-actions[bot] merged 2 commits intollm-d:mainfrom
mayabar:update-sim-deploy
Aug 21, 2025
Merged

add tokenizer directory to simulator deployment yaml#307
github-actions[bot] merged 2 commits intollm-d:mainfrom
mayabar:update-sim-deploy

Conversation

@mayabar
Copy link
Contributor

@mayabar mayabar commented Aug 20, 2025

No description provided.

Signed-off-by: Maya Barnea <mayab@il.ibm.com>
- "--block-size=16"
- "--zmq-endpoint=tcp://${EPP_NAME}.default.svc.cluster.local:5557"
- "--event-batch-size=16"
- "--tokenizers-cache-dir=tokenizer-cache"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should be /tokenizer-cache

Signed-off-by: Maya Barnea <mayab@il.ibm.com>
@mayabar mayabar requested a review from shmuelk August 21, 2025 06:59
Copy link
Collaborator

@shmuelk shmuelk left a comment

Choose a reason for hiding this comment

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

/lgtm

/approve

@shmuelk
Copy link
Collaborator

shmuelk commented Aug 21, 2025

/lgtm

/approve

@github-actions github-actions bot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 21, 2025
@github-actions github-actions bot merged commit ddb8aa1 into llm-d:main Aug 21, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm "Looks good to me", indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants