Skip to content

Fix LogBuilder tagsRawData to include LAL-added tags#13748

Merged
wu-sheng merged 1 commit intomasterfrom
fix/log-tags-raw-data-include-lal-tags
Mar 18, 2026
Merged

Fix LogBuilder tagsRawData to include LAL-added tags#13748
wu-sheng merged 1 commit intomasterfrom
fix/log-tags-raw-data-include-lal-tags

Conversation

@wu-sheng
Copy link
Member

Summary

  • LogBuilder.toLog() only serialized original LogData tags into tagsRawData, causing LAL-added tags (via tag 'key': value in extractor) to be missing from the UI tag display and Kafka log export.
  • Merge both original and LAL-added tags into a single LogTags protobuf for tagsRawData, so all tags are visible in query results and exports.

Test plan

  • Verify compile passes (mvnw compile -pl oap-server/server-core)
  • Verify LAL-added tags appear in log query results (UI tag list)
  • Verify LAL-added tags appear in Kafka log export

🤖 Generated with Claude Code

tagsRawData only contained original LogData tags from the agent,
causing LAL-added tags (via `tag 'key': value`) to be missing from
the UI tag display and Kafka export. Merge both original and
LAL-added tags into a single LogTags protobuf for tagsRawData.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@wu-sheng wu-sheng added this to the 10.4.0 milestone Mar 18, 2026
@wu-sheng wu-sheng added bug Something isn't working and you are sure it's a bug! backend OAP backend related. labels Mar 18, 2026
@wu-sheng wu-sheng merged commit fbcaf23 into master Mar 18, 2026
372 of 374 checks passed
@wu-sheng wu-sheng deleted the fix/log-tags-raw-data-include-lal-tags branch March 18, 2026 02:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend OAP backend related. bug Something isn't working and you are sure it's a bug!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants