Skip to content

[receiver/kafka] Fix deprecated field migration logic for metrics, traces, and profiles topic configuration#45215

Merged
songy23 merged 4 commits intoopen-telemetry:mainfrom
paulojmdias:fix/kafka_receiver_migration_fields
Jan 5, 2026
Merged

[receiver/kafka] Fix deprecated field migration logic for metrics, traces, and profiles topic configuration#45215
songy23 merged 4 commits intoopen-telemetry:mainfrom
paulojmdias:fix/kafka_receiver_migration_fields

Conversation

@paulojmdias
Copy link
Member

Description

Fixed bug where deprecated topic and exclude_topic fields for metrics, traces, and profiles were incorrectly checking the logs configuration instead of their respective signal type's configuration. This prevented proper migration from deprecated fields unless logs.topics was empty.

Also fixed validation error message typo for traces.exclude_topic and corrected profiles validation to check ExcludeTopic fields instead of Topic fields.

…aces, and profiles topic configuration

Signed-off-by: Paulo Dias <paulodias.gm@gmail.com>
Signed-off-by: Paulo Dias <paulodias.gm@gmail.com>
Copy link
Contributor

@axw axw left a comment

Choose a reason for hiding this comment

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

😬 thanks @paulojmdias!

@paulojmdias paulojmdias added the ready to merge Code review completed; ready to merge by maintainers label Jan 5, 2026
@paulojmdias paulojmdias removed the ready to merge Code review completed; ready to merge by maintainers label Jan 5, 2026
@paulojmdias paulojmdias added the ready to merge Code review completed; ready to merge by maintainers label Jan 5, 2026
@paulojmdias
Copy link
Member Author

/rerun

@songy23 songy23 merged commit 0ce888b into open-telemetry:main Jan 5, 2026
207 of 211 checks passed
@github-actions github-actions bot added this to the next release milestone Jan 5, 2026
@paulojmdias paulojmdias deleted the fix/kafka_receiver_migration_fields branch January 5, 2026 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Code review completed; ready to merge by maintainers receiver/kafka

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants