Skip to content

[monitoring] Add OpenTelemetry spans in scd/operational_intents_handler #1366

Closed
the-glu wants to merge 2 commits intointeruss:masterfrom
Orbitalize:opentelemetry_scd_oih
Closed

[monitoring] Add OpenTelemetry spans in scd/operational_intents_handler #1366
the-glu wants to merge 2 commits intointeruss:masterfrom
Orbitalize:opentelemetry_scd_oih

Conversation

@the-glu
Copy link
Contributor

@the-glu the-glu commented Jan 29, 2026

This is a set of 3 PRs, proposition to add OpenTelemetry to generate optional traces.
You can check the base PR, the one for handlers and the one for datastores.

This PR add as an example spans in pkg/scd/operational_intents_handler.go

Spans look like that:

image_2026-01-29_12-46-54

Idea would be to have that in all handlers.

Copy link
Contributor

@mickmis mickmis left a comment

Choose a reason for hiding this comment

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

Reviewed f99b7de

Do we really want to manually add this tracer to all methods manually? This is quite noisy and error-prone. We cannot have a middleware doing so automatically?

@the-glu
Copy link
Contributor Author

the-glu commented Mar 11, 2026

Following discussions I close that one (that has been partially overwritten by #1391 ).

I do think we want as much inside as possible, but since there is no easy way without having it everywhere, that probably not the correct approach; and if needed we could still add specific span in the future.

@the-glu the-glu closed this Mar 11, 2026
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