Skip to content

Conversation

@mariomac
Copy link
Contributor

@mariomac mariomac commented Dec 5, 2025

Updates the documentation to some changes in OBI

  • A common metrics > features configuration section was added to OBI, and deprecated the per-exporter (OTEL or Prometheus) features section.
  • The OTEL_EBPF_ENABLE_CONTEXT_PROPAGATION option was totally removed from OBI but still some references to it persisted.
  • Some examples in the documentation still use the OBI_ prefix, but it should be OTEL_EBPF_

@mariomac mariomac requested a review from a team as a code owner December 5, 2025 15:56
@otelbot-docs otelbot-docs bot requested a review from a team December 5, 2025 15:56
Copy link
Contributor

@grcevski grcevski left a comment

Choose a reason for hiding this comment

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

Nice! LGTM!

Copy link
Member

@tiffany76 tiffany76 left a comment

Choose a reason for hiding this comment

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

LGTM with a few small copy edits!

- `application`: Application-level metrics
- `application_host`: Application-level host metrics for host-based pricing.
- `application_span`: Application-level trace span metrics in legacy format
(like `traces_spanmetrics_latency`, `spanmetrics` is not separate).
Copy link
Member

Choose a reason for hiding this comment

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

You can add this line to the front matter of this page. (I'd leave a suggestion there, but it's outside the commentable range.)

cSpell:ignore: spanmetrics

The OBI metrics exporter can export the following metrics data groups for
processes matching entries in the [metrics discovery](./) configuration.

- `application`: Application-level metrics
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- `application`: Application-level metrics
- `application`: Application-level metrics.

- `application`: Application-level metrics
- `application_host`: Application-level host metrics for host-based pricing.
- `application_span`: Application-level trace span metrics in legacy format
(like `traces_spanmetrics_latency`, `spanmetrics` is not separate).
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
(like `traces_spanmetrics_latency`, `spanmetrics` is not separate).
(like `traces_spanmetrics_latency`); `spanmetrics` is not separate.

- `application_span`: Application-level trace span metrics in legacy format
(like `traces_spanmetrics_latency`, `spanmetrics` is not separate).
- `application_span_otel`: Application-level trace span metrics in OpenTelemetry
format (like `traces_span_metrics_calls_total`), `span_metrics` is separate.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
format (like `traces_span_metrics_calls_total`), `span_metrics` is separate.
format (like `traces_span_metrics_calls_total`); `span_metrics` is separate.

Comment on lines +51 to +53
[network metrics](../../network) configuration documentation to learn more
- `network_inter_zone`: Network inter-zone metrics, refer to the
[network metrics](../../network/) configuration documentation to learn more
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
[network metrics](../../network) configuration documentation to learn more
- `network_inter_zone`: Network inter-zone metrics, refer to the
[network metrics](../../network/) configuration documentation to learn more
[network metrics](../../network) configuration documentation to learn more.
- `network_inter_zone`: Network inter-zone metrics, refer to the
[network metrics](../../network/) configuration documentation to learn more.

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

Labels

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants