Skip to content

Conversation

Xunzhuo
Copy link
Member

@Xunzhuo Xunzhuo commented Sep 29, 2025

What type of PR is this?

chore: move all dockerfile into tools

Release Notes: Yes

@Xunzhuo Xunzhuo requested a review from rootfs as a code owner September 29, 2025 04:14
Copy link

netlify bot commented Sep 29, 2025

Deploy Preview for vllm-semantic-router ready!

Name Link
🔨 Latest commit 4c241e2
🔍 Latest deploy log https://app.netlify.com/projects/vllm-semantic-router/deploys/68da7f65f099a7000907d8a6
😎 Deploy Preview https://deploy-preview-274--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.

Copy link

github-actions bot commented Sep 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/docker-publish.yml
  • .github/workflows/docker-release.yml
  • .github/workflows/precommit-publish.yml
  • docker-compose.yml

📁 tools

Owners: @yuluo-yx, @rootfs, @Xunzhuo
Files changed:

  • tools/make/docker.mk
  • tools/docker/Dockerfile
  • tools/docker/Dockerfile.extproc
  • tools/docker/Dockerfile.llm-katan
  • tools/docker/Dockerfile.mock-vllm
  • tools/docker/Dockerfile.precommit

📁 website

Owners: @Xunzhuo
Files changed:

  • website/docs/troubleshooting/network-tips.md

vLLM

🎉 Thanks for your contributions!

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

@Xunzhuo
Copy link
Member Author

Xunzhuo commented Sep 29, 2025

/hold

@github-actions github-actions bot added the hold label Sep 29, 2025
@JaredforReal
Copy link
Contributor

There are some configs for Grafana and Prometheus for Docker Compose Path in config/prometheus.yaml and config/grafana/, since Observability config will be put into deploy/kubernetes/observability/(said in #279 ), maybe you can pack the configs in config/ into tools/docker/ too. @Xunzhuo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants