fix: avoid referencing loki subchart when disabled (#60)

Author: ethanndickson

README.md

@@ -22,7 +22,7 @@ Logs will be scraped from all pods in the Kubernetes cluster.
 
 ```bash
 helm repo add coder-observability https://helm.coder.com/observability
-helm upgrade --install coder-observability coder-observability/coder-observability --version 0.4.0 --namespace coder-observability --create-namespace
+helm upgrade --install coder-observability coder-observability/coder-observability --version 0.4.1 --namespace coder-observability --create-namespace
 ```
 
 ## Requirements

coder-observability/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: coder-observability
 description: Gain insights into your Coder deployment
 type: application
-version: 0.4.0
+version: 0.4.1
 dependencies:
   - name: pyroscope
     condition: pyroscope.enabled

coder-observability/templates/_collector-config.tpl

@@ -197,11 +197,13 @@ loki.process "pod_logs" {
 {{ if $agent.extraBlocks -}}
 {{ $agent.extraBlocks }}
 {{- end }}
+{{ if .Values.loki.enabled -}}
 loki.write "loki" {
   endpoint {
     url = "http://{{ include "loki.fullname" .Subcharts.loki }}-gateway.{{ .Release.Namespace }}.{{ .Values.global.zone }}/loki/api/v1/push"
   }
 }
+{{- end }}
 
 {{ if .Values.pyroscope.enabled }}
 pyroscope.scrape "pods" {
@@ -407,4 +409,4 @@ prometheus.scrape "coder_metrics" {
 {{- range $key, $val := . -}}
 {{ $key }} = "{{ $val }}",
 {{- end -}}
-{{ end }}
+{{ end }}