Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[demo] Use k8sattributesprocessor instead of env vars #817

Merged
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion charts/opentelemetry-demo/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v2
type: application
name: opentelemetry-demo
version: 0.23.0
version: 0.24.0
description: opentelemetry demo helm chart
home: https://opentelemetry.io/
sources:
Expand Down
7 changes: 7 additions & 0 deletions charts/opentelemetry-demo/UPGRADING.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
# Upgrade guidelines

## To 0.24

This release uses the [kubernetes attributes processor](https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/processor/k8sattributesprocessor)
to add pod labels. If you override the processors array in your config, you will
TylerHelmuth marked this conversation as resolved.
Show resolved Hide resolved
need to add the k8s attributes processor manually to restore `service.instance.id`
resource attribute.

## To 0.23

The Prometheus sub-chart dependency made updates to pod labels. You may need to
Expand Down
Loading