-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs(vrl): update documentation for parse_nginx_log
with the main
log format
#22119
docs(vrl): update documentation for parse_nginx_log
with the main
log format
#22119
Conversation
… log format Signed-off-by: VirtualTam <[email protected]>
84415a0
to
0a484f3
Compare
Thanks @pront for merging this PR and fixing the typos in the Cue files 👍 Though I followed the CONTRIBUTING / Running other checks, I didn't catch these issues locally, and the Github CI seems to require maintainer approval before jobs are started. Looking at one of the failed CI runs, these errors are caught in the Maybe this command could be added to the CONTRIBUTING guide in a section for documentation-related PRs, as well as any relevant command to check the documentation generation locally? |
We had the same thought, already added: https://github.com/vectordotdev/vector/pull/22159/files. |
… log format (vectordotdev#22119) * docs(vrl): update documentation for `parse_nginx_log` with the `main` log format Signed-off-by: VirtualTam <[email protected]> * fix * more fixes --------- Signed-off-by: VirtualTam <[email protected]> Co-authored-by: Pavlos Rontidis <[email protected]>
Summary
This PR updates the documentation for the
parse_nginx_log
VRL remap function with an example for parsing a log line formatted using themain
format (see VRL issue and PR referenced below for more information).Change Type
Is this a breaking change?
How did you test this PR?
I have built the website documentation locally with:
$ cd rust-docs $ make docs
Does this PR include user facing changes?
Checklist
Cargo.lock
), pleaserun
dd-rust-license-tool write
to regenerate the license inventory and commit the changes (if any). More details here.References
main
log format forparse_nginx_log
vrl#1201main
log format toparse_nginx_log
function vrl#1202Notes for reviewers