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

Fix: Expose OpenTelemetry Collector Metrics Endpoint on Port 8888 #1934

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

alexkroman
Copy link

This PR updates the OpenTelemetry Collector configuration to explicitly expose the metrics endpoint on port 8888. Previously, the metrics endpoint was not accessible due to missing configuration settings.

Changes

  • Updated otelcol-config.yaml to ensure the metrics endpoint is enabled.
  • Verified the configuration aligns with OpenTelemetry's best practices.
  • Confirmed that the metrics are now accessible via http://localhost:8888/metrics.

Merge Requirements

For new features contributions, please make sure you have completed the following
essential items:

  • CHANGELOG.md updated to document new feature additions
  • Appropriate documentation updates in the docs
  • Appropriate Helm chart updates in the helm-charts

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

@alexkroman alexkroman requested a review from a team as a code owner January 21, 2025 20:17
Copy link

linux-foundation-easycla bot commented Jan 21, 2025

CLA Signed


The committers listed above are authorized under a signed CLA.

@github-actions github-actions bot added the helm-update-required Requires an update to the Helm chart when released label Jan 21, 2025
@whuanle
Copy link

whuanle commented Jan 22, 2025

This issue does not appear to be resolved and a new error has appeared.

image

@alexkroman
Copy link
Author

I'm attaching the screenshot of the error I get when the patch is removed and the successful message I get when it is applied. I haven't been able to reproduce the "server misbehaving" message you're seeing.

nopatch patch

@whuanle
Copy link

whuanle commented Jan 23, 2025

Maybe I can retest it after the code is merged.
I am currently using version v1.5.1.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
helm-update-required Requires an update to the Helm chart when released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants