MiniLangstream should error out if no NodePort exists for a service (#698)

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

Closed Gagravarr closed 8 months ago

Gagravarr commented 8 months ago

Add error handling to the shell script function start_port_forward to avoid the current hanging if no node port is defined for the service.

Doesn't attempt any alternate forwarding (eg kubectl), just gives a moderately helpful message and errors out.

eolivelli commented 8 months ago

Please fill in https://cla.datastax.com then we can merge your patch. Thanks

Gagravarr commented 8 months ago

CLA filed (with my Apache email)