Skip to content

feat(tracing-apps): Bump otel collector#1645

Open
isantospardo wants to merge 1 commit into
mainfrom
feat/bump-otel-collector-to-0.142.1
Open

feat(tracing-apps): Bump otel collector#1645
isantospardo wants to merge 1 commit into
mainfrom
feat/bump-otel-collector-to-0.142.1

Conversation

@isantospardo

Copy link
Copy Markdown
Contributor

Description

Related to #1603

Bump opentelemetryCollector from 0.135.0 to 0.142.1

Issues

logging exporter deprecated to debug

exporters:
  debug: {}

Checklist

  • This PR contains a description of the changes I'm making
  • I updated the version in Chart.yaml
  • I updated the changelog with an artifacthub.io/changes annotation in Chart.yaml, check the example in the documentation.
  • I updated applicable README.md files using pre-commit run
  • I documented any high-level concepts I'm introducing in docs/
  • CI is currently green and this is ready for review
  • I am ready to test changes after they are applied and released

Signed-off-by: Iago Santos <iago.santos.pardo@adfinis.com>
@isantospardo isantospardo requested a review from a team as a code owner January 30, 2026 14:14
@github-actions github-actions Bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jan 30, 2026

@hairmare hairmare left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not use this one from tracing-apps:

# -- [opentelemetry-collector](https://opentelemetry.io/docs/collector/) ([example](./examples/opentelemetryCollector.yaml))
# @default -- -
opentelemetryCollector:
# -- Enable otel-exporter
enabled: false
name: otel-collector
destination:
# -- Namespace
namespace: "infra-otel-operator"
# -- Repo URL
# @default -- [repo](https://open-telemetry.github.io/opentelemetry-helm-charts)
repoURL: "https://open-telemetry.github.io/opentelemetry-helm-charts"
# -- Chart
chart: "opentelemetry-collector"
# -- [opentelemetry-collector Helm chart](https://github.com/open-telemetry/opentelemetry-helm-charts/tree/main/charts/opentelemetry-collector)
targetRevision: "0.135.0"
# -- Helm values
# @default -- [upstream values](https://github.com/open-telemetry/opentelemetry-helm-charts/blob/main/charts/opentelemetry-collector/values.yaml)
values: {}

@isantospardo

Copy link
Copy Markdown
Contributor Author

Why not use this one from tracing-apps:

# -- [opentelemetry-collector](https://opentelemetry.io/docs/collector/) ([example](./examples/opentelemetryCollector.yaml))
# @default -- -
opentelemetryCollector:
# -- Enable otel-exporter
enabled: false
name: otel-collector
destination:
# -- Namespace
namespace: "infra-otel-operator"
# -- Repo URL
# @default -- [repo](https://open-telemetry.github.io/opentelemetry-helm-charts)
repoURL: "https://open-telemetry.github.io/opentelemetry-helm-charts"
# -- Chart
chart: "opentelemetry-collector"
# -- [opentelemetry-collector Helm chart](https://github.com/open-telemetry/opentelemetry-helm-charts/tree/main/charts/opentelemetry-collector)
targetRevision: "0.135.0"
# -- Helm values
# @default -- [upstream values](https://github.com/open-telemetry/opentelemetry-helm-charts/blob/main/charts/opentelemetry-collector/values.yaml)
values: {}

Just to differentiate between traces and logs, what do you think?

@hairmare

hairmare commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

Just to differentiate between traces and logs, what do you think?

I think we should mainly install one otel-collector per cluster and that there shoul donly be one upstream path for that. In the big picture logs are just traces with less metadata.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants