-
Notifications
You must be signed in to change notification settings - Fork 95
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
Add crate for stress testing the shims #734
Conversation
This would unblock #615 |
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.
Reviewed the first half of the code. Will continue some time later
Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
643c578
to
80b360a
Compare
Signed-off-by: Jorge Prendes <[email protected]>
Thanks for the feedback @Mossaka , updated :-) |
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.
LGTM! Thanks for working on this
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.
sorry for the last review, was head down on a few different this this week and had some power outages.
Is there a reason we are coping in the protos and generating them when we already have a dep on rust-extensions which has these protos build for us?
https://github.com/containerd/rust-extensions/blob/main/crates/shim-protos/src/events.rs
This PR adds the work presented during the last community meeting for stress testing the shims.
I intentionally left out running this in CI, as this is already a good amount of code.
We can add stress testing to CI on a different PR.