Skip to content

feat: Capture EventName for logs#285

Open
brightsparc wants to merge 2 commits intostreamfold:mainfrom
brightsparc:julian/logs-event-name
Open

feat: Capture EventName for logs#285
brightsparc wants to merge 2 commits intostreamfold:mainfrom
brightsparc:julian/logs-event-name

Conversation

@brightsparc
Copy link
Contributor

The otel logs schema was not writing the EventName if it was being passed, so added this to the schema, transformer.

Copy link
Contributor

@rjenkins rjenkins left a comment

Choose a reason for hiding this comment

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

Looks like it needs a cargo fmt.

@brightsparc brightsparc requested a review from rjenkins January 22, 2026 04:44
Copy link
Contributor

@mheffner mheffner left a comment

Choose a reason for hiding this comment

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

Hey @brightsparc! It looks like this column was added only recently to the schema in the CH exporter (open-telemetry/opentelemetry-collector-contrib#43164). It does not exist in the current schema here, so we'll need to perform some upgrades.

It appears that several other columns were added, so we'll need to identify an upgrade path. Obviously this could be a breaking change for existing users. The CH exporter appears to detect the active columns to identify which to send, so we may need to do something similar to avoid breakage.

So I think we'll need to look at this in the larger upgrade scope to bring the full schema up to sync and support users in hybrid status mode.

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