Skip to content

Conversation

srampal
Copy link

@srampal srampal commented Sep 26, 2025

What type of PR is this?

  • "feat(translator): add new feature"

What this PR does / why we need it:
This PR enables Semantic Router to work with Istio Envoy gateway via the ExtProc interface between Istio and Semantic Router

Which issue(s) this PR fixes:
This is one of multiple fixes needed for the issue #39. There will also be additional PRs needed in order to complete that issue/ Feature.

Release Notes: Yes

Copy link

netlify bot commented Sep 26, 2025

Deploy Preview for vllm-semantic-router ready!

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

@srampal
Copy link
Author

srampal commented Sep 26, 2025

I have tested .. this is working correctly with Istio based Envoy gateway.

Copy link
Member

@Xunzhuo Xunzhuo left a comment

Choose a reason for hiding this comment

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

  1. can you create a sparate dir under kubernetes like kubernetes/istio/xxx.yaml to put resource related istio?
  2. can you provide steps you install in kuberenetes with istio?

@Xunzhuo Xunzhuo changed the title Feat(Istio): add support for Istio-Envoy gateway via ExtProc to Semantic Router feat(Istio): integrate with Istio gateway via extproc Sep 26, 2025
@srampal
Copy link
Author

srampal commented Sep 29, 2025

  1. can you create a sparate dir under kubernetes like kubernetes/istio/xxx.yaml to put resource related istio?
  2. can you provide steps you install in kuberenetes with istio?

I was planning to add the istio docs as a separate PR .. anyway .. but sure .. will add that now via this PR.

@srampal
Copy link
Author

srampal commented Oct 1, 2025

@Xunzhuo @rootfs fyi I'm still working on updating this commit and running into some issues so am working on troubleshooting those. Should have an update in a couple of days.

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.

2 participants