Skip to content

Conversation

@mohit-singlestore
Copy link
Contributor

@mohit-singlestore mohit-singlestore commented Dec 10, 2025

This PR has changes to add internalConnectionURL in InferenceAPI. This internalConnectionURL will be used by managed functions to communicate with UMG over private network

@kesmit13
Copy link
Collaborator

Just curious, what exactly is this for?

@mohit-singlestore mohit-singlestore changed the title changes for adding internalConnectionURL Add internalConnectionURL to inferenceAPI info Dec 10, 2025
@mohit-singlestore
Copy link
Contributor Author

Just curious, what exactly is this for?

The internalConnectionURL will be used by managed functions to communicate with UMG using private network instead of public internet.
cc: @mgiannakopoulos

@kesmit13 kesmit13 merged commit b4c65d0 into main Dec 11, 2025
11 checks passed
@kesmit13 kesmit13 deleted the mohit-verma/add-internalConnectionURL-inferenceAPIs branch December 11, 2025 14:38
Copy link
Contributor

@mgiannakopoulos mgiannakopoulos left a comment

Choose a reason for hiding this comment

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

🚢 Ship IT!

@mgiannakopoulos
Copy link
Contributor

mgiannakopoulos commented Dec 11, 2025

@kesmit13 I was about to ask , let's not create a release before this one gets merged and deployed to production: https://github.com/singlestore/helios/pull/21671

It should have been mentioned in the description of the PR by @mohit-singlestore

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.

4 participants