MiniLangStream enable NodePort if missing for services

LangStream / langstream

LangStream. Event-Driven Developer Platform for Building and Running LLM AI Apps. Powered by Kubernetes and Kafka.

https://langstream.ai

Apache License 2.0

379 stars 28 forks source link

MiniLangStream enable NodePort if missing for services #701

Closed Gagravarr closed 8 months ago

Gagravarr commented 8 months ago

MiniLangStream should enable NodePort as required

If start_port_forward finds that there's currently no Target Port and hence no external URL for a service, it should patch the service definition to enable a dynamically allocated NodePort.

This fixes #698

eolivelli commented 8 months ago

@Gagravarr can you please solve the conflicts ?

Gagravarr commented 8 months ago

Conflict solved (sorry, took this branch of the wrong one so clashed with other changes)

Also one small fix added - if there are multiple URLs found when querying for a service, only the first is taken

eolivelli commented 8 months ago

Merged! thanks