Skip to content

Commit

Permalink
feat(observability-pipeline): Add default refinery telemetry api key (#…
Browse files Browse the repository at this point in the history
…410)

## Which problem is this PR solving?

Sets a default `REFINERY_HONEYCOMB_API_KEY` env var so that refinery can
export telemetry using a default install.

## Short description of the changes

- Add a default env var named REFINERY_HONEYCOMB_API_KEY that looks for
the secret that users are instructed to create in the README

## How to verify that this has the expected result

Local testing.
  • Loading branch information
TylerHelmuth authored Dec 20, 2024
1 parent 23cfe9c commit b35c3af
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
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.5-alpha
version: 0.0.6-alpha
appVersion: 0.0.1-alpha
keywords:
- refinery
Expand Down
6 changes: 6 additions & 0 deletions charts/observability-pipeline/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,12 @@ refinery:
Enabled: false
OTelMetrics:
Enabled: true
environment:
- name: REFINERY_HONEYCOMB_API_KEY
valueFrom:
secretKeyRef:
name: honeycomb-observability-pipeline
key: api-key
extraCommandArgs:
- "-c"
- 'http://{{ include "honeycomb-observability-pipeline.name" . }}-observability-pipeline-control-plane:4321/config?kind=refinery_config'
Expand Down

0 comments on commit b35c3af

Please sign in to comment.