-
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
chore(vdev): Rewrite scripts in vdev #16661
Conversation
Signed-off-by: jonathanpv <jonpad512@gmail.com>
✅ Deploy Preview for vector-project canceled.
|
✅ Deploy Preview for vrl-playground canceled.
|
Signed-off-by: jonathanpv <jonpad512@gmail.com>
Signed-off-by: jonathanpv <jonpad512@gmail.com>
Signed-off-by: jonathanpv <jonpad512@gmail.com>
Regression Detector ResultsRun ID: d6bed955-ba30-4822-9b34-ee883663d5e4 ExplanationA regression test is an integrated performance test for The table below, if present, lists those experiments that have experienced a statistically significant change in mean optimization goal performance between baseline and comparison SHAs with 90.00% confidence OR have been detected as newly erratic. Negative values mean that baseline is faster, positive comparison. Results that do not exhibit more than a ±5.00% change in their mean optimization goal are discarded. An experiment is erratic if its coefficient of variation is greater than 0.1. The abbreviated table will be omitted if no interesting change is observed. Changes in experiment optimization goals with confidence ≥ 90.00% and |Δ mean %| ≥ 5.00%:
Fine details of change detection per experiment.
|
Signed-off-by: jonathanpv <jonpad512@gmail.com>
Regression Detector ResultsRun ID: 4d983d3b-b9de-4528-ad29-d6227090c406 ExplanationA regression test is an integrated performance test for The table below, if present, lists those experiments that have experienced a statistically significant change in mean optimization goal performance between baseline and comparison SHAs with 90.00% confidence OR have been detected as newly erratic. Negative values mean that baseline is faster, positive comparison. Results that do not exhibit more than a ±5.00% change in their mean optimization goal are discarded. An experiment is erratic if its coefficient of variation is greater than 0.1. The abbreviated table will be omitted if no interesting change is observed. Changes in experiment optimization goals with confidence ≥ 90.00% and |Δ mean %| ≥ 5.00%:
Fine details of change detection per experiment.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good start! Just a small recommendation to do one pull request per-script. This will help them get reviewed and merged in more quickly.
Ah I see I'll do one more script here that I am almost finished with and then do a PR by PR basis |
Signed-off-by: jonathanpv <jonpad512@gmail.com>
@jszwedko how can I test the |
Regression Detector ResultsRun ID: 5ad8e58d-6694-40e1-89f6-c8d486db93e3 ExplanationA regression test is an integrated performance test for The table below, if present, lists those experiments that have experienced a statistically significant change in mean optimization goal performance between baseline and comparison SHAs with 90.00% confidence OR have been detected as newly erratic. Negative values mean that baseline is faster, positive comparison. Results that do not exhibit more than a ±5.00% change in their mean optimization goal are discarded. An experiment is erratic if its coefficient of variation is greater than 0.1. The abbreviated table will be omitted if no interesting change is observed. No interesting changes in experiment optimization goals with confidence ≥ 90.00% and |Δ mean %| ≥ 5.00%. Fine details of change detection per experiment.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A small nit, we tend to group imports in this style - but it's not enforced (the option hasn't landed in stable
).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a great start. Make sure to also replace the use of these scripts in, for example, the Makefile
with running cargo vdev …
.
check-spelling should be happy if you merge w/ master again. |
Regression Detector ResultsRun ID: 092eade9-d6b5-4e27-9355-bba8bc32eb3c ExplanationA regression test is an integrated performance test for The table below, if present, lists those experiments that have experienced a statistically significant change in mean optimization goal performance between baseline and comparison SHAs with 90.00% confidence OR have been detected as newly erratic. Negative values mean that baseline is faster, positive comparison. Results that do not exhibit more than a ±5.00% change in their mean optimization goal are discarded. An experiment is erratic if its coefficient of variation is greater than 0.1. The abbreviated table will be omitted if no interesting change is observed. No interesting changes in experiment optimization goals with confidence ≥ 90.00% and |Δ mean %| ≥ 5.00%. Fine details of change detection per experiment.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a couple of requests for fixes below, and it looks like this needs to be merged with master to pull in spelling checking fixes. Otherwise these should be good to go.
Regression Detector ResultsRun ID: 110bbab3-e1cd-467c-904d-fefb03192c07 ExplanationA regression test is an integrated performance test for The table below, if present, lists those experiments that have experienced a statistically significant change in mean optimization goal performance between baseline and comparison SHAs with 90.00% confidence OR have been detected as newly erratic. Negative values mean that baseline is faster, positive comparison. Results that do not exhibit more than a ±5.00% change in their mean optimization goal are discarded. An experiment is erratic if its coefficient of variation is greater than 0.1. The abbreviated table will be omitted if no interesting change is observed. No interesting changes in experiment optimization goals with confidence ≥ 90.00% and |Δ mean %| ≥ 5.00%. Fine details of change detection per experiment.
|
rustfmt nightly options aren't enforced yet and was discussed it's not super important to have this resolved for now
Regression Detector ResultsRun ID: 00116545-f3b4-4e64-8c01-1fb24d98705f ExplanationA regression test is an integrated performance test for The table below, if present, lists those experiments that have experienced a statistically significant change in mean optimization goal performance between baseline and comparison SHAs with 90.00% confidence OR have been detected as newly erratic. Negative values mean that baseline is faster, positive comparison. Results that do not exhibit more than a ±5.00% change in their mean optimization goal are discarded. An experiment is erratic if its coefficient of variation is greater than 0.1. The abbreviated table will be omitted if no interesting change is observed. No interesting changes in experiment optimization goals with confidence ≥ 90.00% and |Δ mean %| ≥ 5.00%. Fine details of change detection per experiment.
|
Problem
It is requested to rewrite some of the shell scripts into Rust so that it is supported with vdev
Solution
I rewrote
test-vrl.sh
,install-git-hooks.sh
,release-github.sh
andcheck-component-docs.sh
in Rust and put them in the vdev command hierarchy I saw fit.This involved adding a
mod <name of command>
, implementing the Cli command interface, translating the shell code into Rust, deleting the shell script, adding helper git functions, modifying the workflow to point tovdev cargo commands
.Testing
I require help verifying the following commands work:
cargo vdev test-vrl
This works and tests vrl if ran through the repository directory.
cargo vdev meta install-git-hooks
I verified this works by running it and then looking at the .git directory for the shell script in question to be copied into there.
cargo vdev release github
@jszwedko I request help with testing the release function
cargo vdev check component-docs
@jszwedko @bruceg I believe the only way to test this one will be through the CI as running generate component docs results in an error in the local machine, and upon reading the source it seems this is only to be ran in the CI, making it difficult for me to test all the cases of component-docs is supposed to hit.
Related Issues
#16383
Checklist