Skip to content

docs(external): add documentation for validate_json_schema() function support in VRL#23359

Merged
pront merged 14 commits intovectordotdev:masterfrom
jlambatl:add_validate_json_schema_support
Jul 18, 2025
Merged

docs(external): add documentation for validate_json_schema() function support in VRL#23359
pront merged 14 commits intovectordotdev:masterfrom
jlambatl:add_validate_json_schema_support

Conversation

@jlambatl
Copy link
Contributor

Summary

This is a documentation sibling PR to add the validate_json_schema() VRL function to vector. The other PR can be found vectordotdev/vrl#1443

Vector configuration

N/A

How did you test this PR?

I have run through the various check scripts in the repository to ensure that the cue files are formatted correctly etc.

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.

References

@jlambatl jlambatl requested a review from a team as a code owner July 11, 2025 06:32
@bits-bot
Copy link

bits-bot commented Jul 11, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added the domain: external docs Anything related to Vector's external, public documentation label Jul 11, 2025
@jlambatl jlambatl requested a review from a team as a code owner July 11, 2025 06:37
pront and others added 2 commits July 11, 2025 13:27
Co-authored-by: Brett Blue <84536271+brett0000FF@users.noreply.github.com>
@pront pront added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Jul 11, 2025
Copy link
Member

@pront pront left a comment

Choose a reason for hiding this comment

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

Thanks @jlambatl!

@pront pront enabled auto-merge July 11, 2025 17:29
@thomasqueirozb
Copy link
Contributor

json-schema_definition.json should be an actual file in lib/vector-vrl/tests/resources/. Without this file tests won't pass

@thomasqueirozb thomasqueirozb added the meta: awaiting author Pull requests that are awaiting their author. label Jul 15, 2025
auto-merge was automatically disabled July 15, 2025 22:40

Head branch was pushed to by a user without write access

@github-actions github-actions bot removed the meta: awaiting author Pull requests that are awaiting their author. label Jul 15, 2025
@jlambatl
Copy link
Contributor Author

added the json-schema_definition.json file to the path as mentioned by @thomasqueirozb.

@pront pront enabled auto-merge July 16, 2025 13:39
auto-merge was automatically disabled July 16, 2025 23:13

Head branch was pushed to by a user without write access

@jlambatl
Copy link
Contributor Author

Added the newline that was blocking the vdev fmt check test.

@pront pront enabled auto-merge July 17, 2025 13:51
@thomasqueirozb
Copy link
Contributor

Your json was incorrectly using {{ which is for template literals. See string reference

@pront pront added this pull request to the merge queue Jul 18, 2025
auto-merge was automatically disabled July 18, 2025 21:33

Pull Request is not mergeable

Merged via the queue into vectordotdev:master with commit d654fdd Jul 18, 2025
54 checks passed
5Dev24 pushed a commit to 5Dev24/vector that referenced this pull request Jul 21, 2025
… support in VRL (vectordotdev#23359)

* add documentation for validate_json_schema vrl function

* doc(validate_json_schema) docs update

* add documentation changelog fragment

* Update website/cue/reference/remap/functions/validate_json_schema.cue

Co-authored-by: Brett Blue <84536271+brett0000FF@users.noreply.github.com>

* cargo update -p vrl

* remove changelog

* update licenses

* add json-schema_definition for tests to pass

* add newline to json-schema_definition.json so cargo vdec check fmt passes

* Fix invalid json

---------

Co-authored-by: Pavlos Rontidis <pavlos.rontidis@gmail.com>
Co-authored-by: Brett Blue <84536271+brett0000FF@users.noreply.github.com>
Co-authored-by: Thomas <thomas.schneider@datadoghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: external docs Anything related to Vector's external, public documentation no-changelog Changes in this PR do not need user-facing explanations in the release changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants