Skip to content

Added page on traces and grouped it with metrics in observability sec… - #193

Merged
MartinaeyNL merged 3 commits into
mainfrom
feature/observability
Sep 21, 2026
Merged

MartinaeyNL merged 3 commits into
mainfrom
feature/observability

Conversation

@ebariaux

@ebariaux ebariaux commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

…tion

@MartinaeyNL MartinaeyNL left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks OK to me.
Let's align with the changes necessary in the openremote/openremote pull request.

Blocked by openremote/openremote#3218

@wborn wborn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is an additional AI-assisted review.

The observability grouping and tracing documentation look good overall. One OpenTelemetry configuration detail should be aligned with openremote/openremote#3218.

Comment thread docs/user-guide/120-observability/30-traces.md

@MartinaeyNL MartinaeyNL left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's merge this. We can resolve the deprecation notice later 👍

@MartinaeyNL
MartinaeyNL enabled auto-merge (squash) September 21, 2026 11:55
@MartinaeyNL
MartinaeyNL merged commit e516e22 into main Sep 21, 2026
2 checks passed
@MartinaeyNL
MartinaeyNL deleted the feature/observability branch September 21, 2026 12:15
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