-
Notifications
You must be signed in to change notification settings - Fork 109
benches: add a new wasi-demo-{app, oci} benchmark #764
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
Conversation
2efd4a2
to
9b729b4
Compare
benches/containerd-shim-benchmarks/benches/wasi-demo-app-benchmarks.rs
Outdated
Show resolved
Hide resolved
benches/containerd-shim-benchmarks/benches/wasi-demo-app-benchmarks.rs
Outdated
Show resolved
Hide resolved
benches/containerd-shim-benchmarks/benches/wasi-demo-app-benchmarks.rs
Outdated
Show resolved
Hide resolved
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, the two comments about re-usability are minor and could be done when needed
benches/containerd-shim-benchmarks/benches/wasi-demo-app-benchmarks.rs
Outdated
Show resolved
Hide resolved
benches/containerd-shim-benchmarks/benches/wasi-demo-app-benchmarks.rs
Outdated
Show resolved
Hide resolved
This commit adds a new benchmark for running the shims end-to-end on wasi-demo-apps to print 'hello'. Signed-off-by: Jiaxiao (mossaka) Zhou <[email protected]>
It seems like that the 'sudo make install' uses a different TARGET_DIR than 'make build' Signed-off-by: Jiaxiao (mossaka) Zhou <[email protected]>
Signed-off-by: Jiaxiao (mossaka) Zhou <[email protected]>
this adds wasi-demo-app OCI image to the benches Signed-off-by: Jiaxiao (mossaka) Zhou <[email protected]>
Signed-off-by: Jiaxiao (mossaka) Zhou <[email protected]>
Signed-off-by: Jiaxiao (mossaka) Zhou <[email protected]>
this commit extract the TestCase out to possibly run any OCI images. It also extract the verify_output out as a closure so that the bench group can customize how to verify the output. Signed-off-by: Jiaxiao (mossaka) Zhou <[email protected]>
f02f9ec
to
292a656
Compare
Signed-off-by: Jiaxiao (mossaka) Zhou <[email protected]>
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
It doesn't seem like GitHub has recognized this as an approval, but I will see it as one! |
This commit adds a new benchmark for running the shims end-to-end on wasi-demo-{app, oci} to print 'hello'.