Skip to content

Conversation

@JaredforReal
Copy link
Collaborator

Which issue(s) this PR fixes:
Fixes #499

@netlify
Copy link

netlify bot commented Oct 29, 2025

Deploy Preview for vllm-semantic-router ready!

Name Link
🔨 Latest commit dc89d32
🔍 Latest deploy log https://app.netlify.com/projects/vllm-semantic-router/deploys/69021bf7f823640008f424e5
😎 Deploy Preview https://deploy-preview-553--vllm-semantic-router.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link

github-actions bot commented Oct 29, 2025

👥 vLLM Semantic Team Notification

The following members have been identified for the changed files in this PR and have been automatically assigned:

📁 Root Directory

Owners: @rootfs, @Xunzhuo
Files changed:

  • .github/workflows/k8s-kind-integration-test.yml

📁 deploy

Owners: @rootfs, @Xunzhuo
Files changed:

  • deploy/kubernetes/deployment.yaml

vLLM

🎉 Thanks for your contributions!

This comment was automatically generated based on the OWNER files in the repository.

@JaredforReal
Copy link
Collaborator Author

cc @rootfs

@rootfs
Copy link
Collaborator

rootfs commented Oct 29, 2025

@JaredforReal you probably also need a CI specific readiness probe too
https://github.com/vllm-project/semantic-router/blob/main/deploy/kubernetes/deployment.yaml#L144

@JaredforReal
Copy link
Collaborator Author

@rootfs PTAL, thanks!

@rootfs rootfs merged commit 75523aa into vllm-project:main Oct 29, 2025
15 checks passed
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.

3 participants