Skip to content

Conversation

@gwenaskell
Copy link
Contributor

@gwenaskell gwenaskell commented Dec 2, 2025

Summary

Following vectordotdev/vrl#1542, some datadog search queries are parsed differently by VRL. This PR bumps VRL and fixes the impacted unit tests.

Note that the VRL dependency update will cause Vector to interpret some datadog queries differently, which can impact some customers.

Vector configuration

NA

How did you test this PR?

CI

Change Type

  • Bug fix
  • New feature
  • 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.

Those changes are already mentioned in the VRL changelog

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 here.

@gwenaskell gwenaskell requested a review from a team as a code owner December 2, 2025 10:07
@github-actions
Copy link

github-actions bot commented Dec 2, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@gwenaskell gwenaskell changed the title Bump vrl and fix datadog search tests chore(vrl): Bump vrl hash and fix datadog search tests Dec 2, 2025
@bruceg bruceg changed the title chore(vrl): Bump vrl hash and fix datadog search tests chore(vrl)!: Bump vrl hash and fix datadog search tests Dec 2, 2025
@bruceg bruceg added type: bug A code related bug. meta: breaking change Anything that breaks backward compatibility. provider: datadog Anything `datadog` service provider related domain: vrl Anything related to the Vector Remap Language dependencies Pull requests that update a dependency file vrl: stdlib changes to VRL's standard library. labels Dec 2, 2025
Copy link
Contributor

@thomasqueirozb thomasqueirozb left a comment

Choose a reason for hiding this comment

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

Thanks!

@thomasqueirozb thomasqueirozb added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Dec 2, 2025
@gwenaskell
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@thomasqueirozb thomasqueirozb added this pull request to the merge queue Dec 2, 2025
Merged via the queue into master with commit 80fc73b Dec 2, 2025
62 of 70 checks passed
@thomasqueirozb thomasqueirozb deleted the yoenn.burban/fix-datadog-search-tests branch December 2, 2025 17:28
@github-actions github-actions bot locked and limited conversation to collaborators Dec 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

dependencies Pull requests that update a dependency file domain: vrl Anything related to the Vector Remap Language meta: breaking change Anything that breaks backward compatibility. no-changelog Changes in this PR do not need user-facing explanations in the release changelog provider: datadog Anything `datadog` service provider related type: bug A code related bug. vrl: stdlib changes to VRL's standard library.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants