Skip to content

Conversation

JaredforReal
Copy link
Contributor

@JaredforReal JaredforReal commented Oct 2, 2025

What type of PR is this?
feat: add CI test for k8s core deployment

What this PR does / why we need it:
This workflow tests the CORE semantic-router Kubernetes deployment.

Test Scope:

  • Core deployment (namespace, pvc, deployment, service, configmap)
  • Manifest validation (kubeconform)
  • Service connectivity (gRPC, metrics, API ports)
  • Security scanning (Trivy, Checkov)
  • Basic syntax validation for observability and AI-Gateway configs

Out of Scope (planned for follow-up PRs):

  • Observability stack deployment (Prometheus + Grafana)
  • AI Gateway end-to-end testing (Envoy Gateway + InferencePool)

For Reviewer:
I'm new to setting up CI tests for Kubernetes on GitHub Actions, and would really appreciate any suggestions!

Copy link

netlify bot commented Oct 2, 2025

Deploy Preview for vllm-semantic-router ready!

Name Link
🔨 Latest commit 196961c
🔍 Latest deploy log https://app.netlify.com/projects/vllm-semantic-router/deploys/68deb463d6e651000896f080
😎 Deploy Preview https://deploy-preview-317--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-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Copy link

github-actions bot commented Oct 2, 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-integration-test.yml

vLLM

🎉 Thanks for your contributions!

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

@rootfs
Copy link
Collaborator

rootfs commented Oct 2, 2025

@JaredforReal this is great! Left a comment here.

@JaredforReal JaredforReal marked this pull request as draft October 2, 2025 16:18
@JaredforReal JaredforReal marked this pull request as ready for review October 2, 2025 17:20
@rootfs
Copy link
Collaborator

rootfs commented Oct 2, 2025

@JaredforReal thanks for making this happen! Would be great to have the e2e test integrated in the following up PRs!
cc @yossiovadia

@JaredforReal
Copy link
Contributor Author

@JaredforReal thanks for making this happen! Would be great to have the e2e test integrated in the following up PRs! cc @yossiovadia

I would love to move things forward!

@rootfs rootfs merged commit 5ec06fd into vllm-project:main Oct 2, 2025
15 checks passed
@JaredforReal JaredforReal deleted the gha branch October 2, 2025 17:31
Aias00 pushed a commit to Aias00/semantic-router that referenced this pull request Oct 4, 2025
* add k8s integration github action test

Signed-off-by: JaredforReal <[email protected]>

* add syntax validation for observability and ai-gateway

Signed-off-by: JaredforReal <[email protected]>

* fix image conflict & kustomize path error

Signed-off-by: JaredforReal <[email protected]>

* fix network error in kind

Signed-off-by: JaredforReal <[email protected]>

* fix https response to http error

Signed-off-by: JaredforReal <[email protected]>

* fix model init error

Signed-off-by: JaredforReal <[email protected]>

* change hf-cli to hf download for models

Signed-off-by: JaredforReal <[email protected]>

* change image loading strategy & models init

Signed-off-by: JaredforReal <[email protected]>

---------

Signed-off-by: JaredforReal <[email protected]>
Signed-off-by: liuhy <[email protected]>
Aias00 pushed a commit to Aias00/semantic-router that referenced this pull request Oct 4, 2025
* add k8s integration github action test

Signed-off-by: JaredforReal <[email protected]>

* add syntax validation for observability and ai-gateway

Signed-off-by: JaredforReal <[email protected]>

* fix image conflict & kustomize path error

Signed-off-by: JaredforReal <[email protected]>

* fix network error in kind

Signed-off-by: JaredforReal <[email protected]>

* fix https response to http error

Signed-off-by: JaredforReal <[email protected]>

* fix model init error

Signed-off-by: JaredforReal <[email protected]>

* change hf-cli to hf download for models

Signed-off-by: JaredforReal <[email protected]>

* change image loading strategy & models init

Signed-off-by: JaredforReal <[email protected]>

---------

Signed-off-by: JaredforReal <[email protected]>
Signed-off-by: liuhy <[email protected]>
Aias00 pushed a commit to Aias00/semantic-router that referenced this pull request Oct 4, 2025
* add k8s integration github action test

Signed-off-by: JaredforReal <[email protected]>

* add syntax validation for observability and ai-gateway

Signed-off-by: JaredforReal <[email protected]>

* fix image conflict & kustomize path error

Signed-off-by: JaredforReal <[email protected]>

* fix network error in kind

Signed-off-by: JaredforReal <[email protected]>

* fix https response to http error

Signed-off-by: JaredforReal <[email protected]>

* fix model init error

Signed-off-by: JaredforReal <[email protected]>

* change hf-cli to hf download for models

Signed-off-by: JaredforReal <[email protected]>

* change image loading strategy & models init

Signed-off-by: JaredforReal <[email protected]>

---------

Signed-off-by: JaredforReal <[email protected]>
Signed-off-by: liuhy <[email protected]>
Aias00 pushed a commit to Aias00/semantic-router that referenced this pull request Oct 4, 2025
* add k8s integration github action test

Signed-off-by: JaredforReal <[email protected]>

* add syntax validation for observability and ai-gateway

Signed-off-by: JaredforReal <[email protected]>

* fix image conflict & kustomize path error

Signed-off-by: JaredforReal <[email protected]>

* fix network error in kind

Signed-off-by: JaredforReal <[email protected]>

* fix https response to http error

Signed-off-by: JaredforReal <[email protected]>

* fix model init error

Signed-off-by: JaredforReal <[email protected]>

* change hf-cli to hf download for models

Signed-off-by: JaredforReal <[email protected]>

* change image loading strategy & models init

Signed-off-by: JaredforReal <[email protected]>

---------

Signed-off-by: JaredforReal <[email protected]>
Signed-off-by: liuhy <[email protected]>
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