Skip to content

chore(logging): Improve datadog logs semantic meaning warning#25608

Open
gwenaskell wants to merge 2 commits into
masterfrom
yoenn.burban/improve-semantic-meaning-log
Open

chore(logging): Improve datadog logs semantic meaning warning#25608
gwenaskell wants to merge 2 commits into
masterfrom
yoenn.burban/improve-semantic-meaning-log

Conversation

@gwenaskell

Copy link
Copy Markdown
Contributor

Summary

The existing warning log fired when a DD reserved attribute (e.g. status) needed to be relocated based on its semantic meaning, but the destination field already existed. The message was vague — it didn't identify the source path of the semantically-tagged field, the destination it was being moved to, or which field was being renamed and why.

This PR improves the log by adding source_path, destination_path, and renamed_existing_to fields, making it actionable when triaging unexpected field renames in pipelines that route OpenTelemetry or other structured logs through the Datadog logs sink.

Vector configuration

How did you test this PR?

n/a

Change Type

  • Bug fix
  • New feature
  • Dependencies
  • Non-functional (chore, refactoring, docs)
  • Performance

Is this a breaking change?

  • Yes
  • No

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on our guidelines.
  • No. A maintainer will apply the no-changelog label to this PR.

References

Notes

  • Please read our Vector contributor resources.
  • Do not hesitate to use @vectordotdev/vector to reach out to us regarding this PR.
  • Some CI checks run only after we manually approve them.
    • We recommend adding a pre-push hook, please see this template.
    • Alternatively, we recommend running the following locally before pushing to the remote branch:
      • make fmt
      • make check-clippy (if there are failures it's possible some of them can be fixed with make clippy-fix)
      • make test
  • After a review is requested, please avoid force pushes to help us review incrementally.
    • Feel free to push as many commits as you want. They will be squashed into one before merging.
    • For example, you can run git merge origin master and git push.
  • If this PR introduces changes Vector dependencies (modifies Cargo.lock), please
    run make build-licenses to regenerate the license inventory and commit the changes (if any). More details on the dd-rust-license-tool.

gwenaskell and others added 2 commits June 12, 2026 10:50
…ity improvement

Co-authored-by: Cursor <cursoragent@cursor.com>
@gwenaskell gwenaskell requested a review from a team as a code owner June 12, 2026 08:54
@github-actions github-actions Bot added the domain: sinks Anything related to the Vector's sinks label Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: sinks Anything related to the Vector's sinks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant