Skip to content

Commit

Permalink
feat(observability-pipeline): add support for environments and volumes (
Browse files Browse the repository at this point in the history
#420)

## Which problem is this PR solving?

Add ability to set environment variables and volumes on the collector
pods.

## Short description of the changes

- add env support
- add volume support
- add volumeMount support
- add default env var

## How to verify that this has the expected result

tested in sippycup
  • Loading branch information
TylerHelmuth authored Jan 24, 2025
1 parent 9ed8204 commit 835f65c
Show file tree
Hide file tree
Showing 4 changed files with 20 additions and 2 deletions.
2 changes: 1 addition & 1 deletion charts/observability-pipeline/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: observability-pipeline
description: Chart to deploy both OpenTelemetry Collector and Honeycomb Refinery
type: application
version: 0.0.9-alpha
version: 0.0.10-alpha
appVersion: 0.0.1-alpha
keywords:
- refinery
Expand Down
10 changes: 10 additions & 0 deletions charts/observability-pipeline/templates/collector-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,18 +28,28 @@ spec:
args:
- --config
- /etc/opampsupervisor/config.yaml
{{- with .Values.collector.environment }}
env:
{{- toYaml . | nindent 12 }}
{{- end }}
{{- with .Values.collector.resources }}
resources:
{{- toYaml . | nindent 12 }}
{{- end }}
volumeMounts:
- name: config
mountPath: /etc/opampsupervisor
{{- with .Values.controlPlane.volumeMounts }}
{{- toYaml . | nindent 12 }}
{{- end }}
volumes:
- name: config
configMap:
name: {{ include "honeycomb-observability-pipeline.fullname" . }}-collector
items:
- key: config
path: config.yaml
{{- with .Values.collector.volumes }}
{{- toYaml . | nindent 8 }}
{{- end }}
{{- end }}
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ spec:
{{- with .Values.controlPlane.volumeMounts }}
{{- toYaml . | nindent 12 }}
{{- end }}
{{- end }}
{{- end }}
ports:
- name: config
containerPort: 4321
Expand Down
8 changes: 8 additions & 0 deletions charts/observability-pipeline/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,14 @@ collector:
repository: ""
pullPolicy: IfNotPresent
tag: ""
environment:
- name: HONEYCOMB_API_KEY
valueFrom:
secretKeyRef:
name: honeycomb-observability-pipeline
key: api-key
volumes: []
volumeMounts: []


otelcol-deployment:
Expand Down

0 comments on commit 835f65c

Please sign in to comment.