Merge pull request #50 from openlit/fix/controller-service-name-resolution

fix(controller): resolve correct parent service name in auto-derived …

Author: amanagarwal042

charts/openlit-controller/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: openlit-controller
 description: OpenLIT Controller - Zero-code LLM and Agent observability using eBPF
 type: application
-version: 0.1.0
+version: 0.1.1
 appVersion: "0.1.0"
 home: https://openlit.io
 sources:

charts/openlit-controller/templates/_helpers.tpl

@@ -63,29 +63,65 @@ ServiceAccount name
 {{- end }}
 {{- end }}
 
+{{/*
+Resolve the OpenLIT Kubernetes service name.
+
+Priority (highest to lowest):
+  1. config.openlitServiceName — explicit override for standalone deployments.
+  2. Auto-derive — mirrors the parent "openlit" chart's fullname logic so the
+     name is correct when deployed as a subchart regardless of release name.
+*/}}
+{{- define "openlit-controller.parentServiceName" -}}
+{{- if .Values.config.openlitServiceName }}
+{{- .Values.config.openlitServiceName | trunc 63 | trimSuffix "-" }}
+{{- else }}
+{{- $parentName := "openlit" }}
+{{- if contains $parentName .Release.Name }}
+{{- .Release.Name | trunc 63 | trimSuffix "-" }}
+{{- else }}
+{{- printf "%s-%s" .Release.Name $parentName | trunc 63 | trimSuffix "-" }}
+{{- end }}
+{{- end }}
+{{- end }}
+
+{{/*
+Build a host string for the parent OpenLIT service.
+
+When config.openlitNamespace is set (standalone / cross-namespace), produce a
+fully-qualified DNS name.  Otherwise use the short service name (same namespace).
+*/}}
+{{- define "openlit-controller.parentServiceHost" -}}
+{{- $svc := include "openlit-controller.parentServiceName" . }}
+{{- if .Values.config.openlitNamespace }}
+{{- printf "%s.%s.svc.cluster.local" $svc .Values.config.openlitNamespace }}
+{{- else }}
+{{- $svc }}
+{{- end }}
+{{- end }}
+
 {{/*
 Resolve the OpenLIT dashboard URL.
-When config.openlitUrl is set, use it directly.
-Otherwise, derive from the parent chart's release name (subchart mode).
+
+Priority: explicit config.openlitUrl > auto-derived from service name + namespace.
 */}}
 {{- define "openlit-controller.openlitUrl" -}}
 {{- if .Values.config.openlitUrl }}
 {{- .Values.config.openlitUrl }}
 {{- else }}
-{{- printf "http://%s-openlit:%s" .Release.Name "3000" }}
+{{- printf "http://%s:%s" (include "openlit-controller.parentServiceHost" .) "3000" }}
 {{- end }}
 {{- end }}
 
 {{/*
 Resolve the OTLP endpoint.
-When config.otlpEndpoint is set, use it directly.
-Otherwise, derive from the parent chart's release name (subchart mode).
+
+Priority: explicit config.otlpEndpoint > auto-derived from service name + namespace.
 */}}
 {{- define "openlit-controller.otlpEndpoint" -}}
 {{- if .Values.config.otlpEndpoint }}
 {{- .Values.config.otlpEndpoint }}
 {{- else }}
-{{- printf "http://%s-openlit:%s" .Release.Name "4318" }}
+{{- printf "http://%s:%s" (include "openlit-controller.parentServiceHost" .) "4318" }}
 {{- end }}
 {{- end }}
 

charts/openlit-controller/values.yaml

@@ -19,10 +19,31 @@ fullnameOverride: ""
 # are auto-derived from the parent release name. Override only when using an
 # external OpenLIT instance or deploying the controller standalone.
 config:
-  # URL of the OpenLIT dashboard (required when standalone)
+  # Full URL of the OpenLIT dashboard.  Highest priority override — when set,
+  # openlitServiceName and openlitNamespace are ignored for this field.
+  # Example: "http://openlit.monitoring.svc.cluster.local:3000"
   openlitUrl: ""
-  # OTLP HTTP endpoint for traces and metrics (required when standalone)
+  # Full OTLP HTTP endpoint.  Same override semantics as openlitUrl.
+  # Example: "http://openlit.monitoring.svc.cluster.local:4318"
   otlpEndpoint: ""
+
+  # --- Standalone deployment helpers ---
+  # When deploying the controller chart on its own (not as a subchart of
+  # openlit), set openlitServiceName to the Kubernetes Service name of your
+  # existing OpenLIT deployment.  Optionally set openlitNamespace if OpenLIT
+  # is in a different namespace.
+  #
+  # Examples (standalone in same namespace):
+  #   config.openlitServiceName: openlit
+  #
+  # Examples (standalone, cross-namespace):
+  #   config.openlitServiceName: openlit
+  #   config.openlitNamespace: monitoring
+  #
+  # When deployed as a subchart of the openlit chart, leave both empty —
+  # the service name is auto-derived from the release.
+  openlitServiceName: ""
+  openlitNamespace: ""
   # How often the controller polls the dashboard
   pollInterval: "60s"
   # Deployment environment label

charts/openlit/Chart.lock

@@ -1,6 +1,6 @@
 dependencies:
 - name: openlit-controller
   repository: file://../openlit-controller
-  version: 0.1.0
-digest: sha256:c38e4d551a6786cf6f90da491c3ff6bb7e5511389972fbadd52021dc5ef83398
-generated: "2026-04-11T16:35:42.016301+05:30"
+  version: 0.1.1
+digest: sha256:4ddad965d3914002694065413b7df1c67b230f326767e2d32367e28382e2e9b5
+generated: "2026-05-04T17:03:06.057969+05:30"

charts/openlit/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: openlit
 description: A Helm chart for OpenLIT
 type: application
-version: 1.19.0
+version: 1.19.1
 appVersion: 1.19.0
 icon: https://avatars.githubusercontent.com/u/149867240?s=200&v=4
 home: https://openlit.io
@@ -22,7 +22,7 @@ maintainers:
     url: https://openlit.io
 dependencies:
   - name: openlit-controller
-    version: "0.1.0"
+    version: "0.1.1"
     repository: "file://../openlit-controller"
     condition: openlit-controller.enabled
     tags: