Skip to content

docs: clarify metrics not included in Coralogix integration#157

Merged
tdehan86 merged 3 commits intomainfrom
docs/coralogix-metrics-clarification
Feb 11, 2026
Merged

docs: clarify metrics not included in Coralogix integration#157
tdehan86 merged 3 commits intomainfrom
docs/coralogix-metrics-clarification

Conversation

@tdehan86
Copy link
Contributor

Added a "What's included" section to clarify that the Coralogix OTEL integration only sends traces, not metrics.

Users who want metrics in Coralogix can use the Prometheus integration + OTEL Collector.

Add 'What's included' section listing traces, HTTP instrumentation,
and correlation metadata. Add note clarifying that metrics are not
included and linking to Prometheus integration for metrics export.
Copy link
Contributor

@guolau guolau left a comment

Choose a reason for hiding this comment

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

Thanks Thomas! Good thing to clarify for this integration. I left a few small nits 🙂

Comment on lines 70 to 71
If you are using a different trace header than the standard `traceparent`, you can configure this header in the *custom
trace header* field in the integration settings.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this line be moved to above the "What's included" section (to line 57)? Seems like it's separated from the relevant section currently

Copy link
Contributor

Choose a reason for hiding this comment

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

This still needs to be moved up, I think?

Make sens to keep names closer to what we have

Co-authored-by: Laura Guo <48332483+guolau@users.noreply.github.com>
Comment on lines 70 to 71
If you are using a different trace header than the standard `traceparent`, you can configure this header in the *custom
trace header* field in the integration settings.
Copy link
Contributor

Choose a reason for hiding this comment

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

This still needs to be moved up, I think?

@tdehan86 tdehan86 requested a review from guolau February 3, 2026 15:24
@tdehan86 tdehan86 merged commit 6659489 into main Feb 11, 2026
3 checks passed
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.

2 participants