Skip to content

Commit

Permalink
[demo] Use k8sattributesprocessor instead of env vars (open-telemetry…
Browse files Browse the repository at this point in the history
…#817)

* [demo] Use k8sattributesprocessor instead of env vars

Signed-off-by: Goutham <gouthamve@gmail.com>

* Add 0.24 to upgrading.md

Signed-off-by: Goutham <gouthamve@gmail.com>

* Update charts/opentelemetry-demo/UPGRADING.md

---------

Signed-off-by: Goutham <gouthamve@gmail.com>
Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
  • Loading branch information
gouthamve and TylerHelmuth authored Sep 12, 2023
1 parent a4e2d58 commit d408404
Show file tree
Hide file tree
Showing 33 changed files with 594 additions and 2,194 deletions.
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 kubernetes metadata as resource attributes. If you override the processors array in your config, you will
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

0 comments on commit d408404

Please sign in to comment.