Skip to content

Conversation

lakkawardhananjay
Copy link

@lakkawardhananjay lakkawardhananjay commented Sep 2, 2025

  • Removed Prometheus receiver configuration from:
    • distributions/otelcol/config.yaml
    • distributions/otelcol-contrib/config.yaml
    • tests/golden/config.yaml
  • Added service.telemetry.metrics to expose internal metrics in OTLP
  • Updated metrics pipeline to use otlp receiver and debug exporter

fixes #836

@lakkawardhananjay lakkawardhananjay requested a review from a team as a code owner September 2, 2025 16:52
Comment on lines 40 to 43
telemetry:
metrics:
level: basic
address: 0.0.0.0:8888
Copy link
Member

@bogdandrutu bogdandrutu Sep 2, 2025

Choose a reason for hiding this comment

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

This does not work. See https://github.com/open-telemetry/opentelemetry-collector/blob/main/CHANGELOG.md#v1340v01280 and the telemetry.disableAddressFieldForInternalTelemetry

Copy link
Author

Choose a reason for hiding this comment

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

Thank you for the clarification @bogdandrutu .
With just level: basic, the metrics will be exposed on the default (localhost) endpoint, as outlined in the latest documentation. I'll recommit with the removed one accordingly.
Much appreciated!

@lakkawardhananjay
Copy link
Author

lakkawardhananjay commented Sep 8, 2025

hey @bogdandrutu please review it.
@ChrsMark plese review it.

@mowies mowies changed the title This PR switches from Prometheus self-scrape to OTLP telemetry and updates the pipeline.(fixes #836 ) Switch from Prometheus self-scrape to OTLP telemetry Sep 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change the config.yaml of distributions to use the OTLP receiver instead of Prometheus
2 participants