Skip to content

[chore] Add in Attribute Bundle pages #2382 #2470

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

thompson-tomo
Copy link
Contributor

@thompson-tomo thompson-tomo commented Jul 3, 2025

Progresses #2382

Changes

Adds in a log record registry to enable easy discovery of log record definitions

Note: if the PR is touching an area that is not listed in the existing areas, or the area does not have sufficient domain experts coverage, the PR might be tagged as experts needed and move slowly until experts are identified.

Merge requirement checklist

@thompson-tomo
Copy link
Contributor Author

Currently blocked by #2471 after which build should be resolved.

@thompson-tomo thompson-tomo marked this pull request as draft July 9, 2025 12:58
@jsuereth
Copy link
Contributor

Commented on the Event registry - We don't want logs vs. records split in semconv. Let's drop this PR and update the Event PR to include all event group types.

@thompson-tomo
Copy link
Contributor Author

As per the filter these are looking at attribute groups rather than events.

What we do need is a way to capture a group/bundle of attributes which can be attached to log records, events or spans.

If we take the example of https://opentelemetry.io/docs/specs/semconv/cloudevents/cloudevents-spans/ that documentation is not a span but rather a grouping of attributes which could be added to a signal.

Let me rework this tomorrow.

@thompson-tomo thompson-tomo force-pushed the feature/#2382_LogRecordRegistry branch from bb4fd82 to d4259e0 Compare July 15, 2025 04:41
@thompson-tomo thompson-tomo changed the title [chore] Add in logrecords registry #2382 [chore] Add in Attribute Bundle pages #2382 Jul 15, 2025
@thompson-tomo thompson-tomo force-pushed the feature/#2382_LogRecordRegistry branch 3 times, most recently from bba10f4 to f357a67 Compare July 15, 2025 04:52
@thompson-tomo thompson-tomo force-pushed the feature/#2382_LogRecordRegistry branch from f357a67 to d935b26 Compare July 15, 2025 04:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants