Skip to content

Conversation

@skudasov
Copy link
Contributor

@skudasov skudasov commented Sep 18, 2025


Below is a summarization created by an LLM (gpt-4-0125-preview). Be mindful of hallucinations and verify accuracy.

Why

The changes enhance the OpenTelemetry (OTel) configuration to improve observability within the framework. Specifically, enabling resource_to_telemetry_conversion improves the integration and fidelity of telemetry data by making it more consistent and easier to query.

What

  • framework/observability/compose/otel.yaml
    • Added resource_to_telemetry_conversion under prometheus exporter with two new settings: enabled: true and add_metric_suffixes: false. This configuration enables the conversion of resource attributes to telemetry attributes, making the exported data more uniform and directly usable without additional transformation. It also opts not to add metric suffixes, which can simplify metric names and improve readability.

@skudasov skudasov requested a review from a team as a code owner September 18, 2025 11:13
@github-actions
Copy link

👋 skudasov, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

ettec
ettec previously approved these changes Sep 18, 2025
sebawo
sebawo previously approved these changes Sep 18, 2025
@skudasov skudasov dismissed stale reviews from sebawo and ettec via 599dac4 September 18, 2025 11:39
@skudasov skudasov requested a review from sebawo September 18, 2025 11:39
@skudasov skudasov enabled auto-merge (squash) September 18, 2025 11:45
@skudasov skudasov merged commit 4431a65 into main Sep 18, 2025
68 of 70 checks passed
@skudasov skudasov deleted the update-prom-otel-metrics-settings branch September 18, 2025 11:46
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.

3 participants