-
Notifications
You must be signed in to change notification settings - Fork 208
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
action: add read-amount and read-count for benchmark #1243
action: add read-amount and read-count for benchmark #1243
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1243 +/- ##
=======================================
Coverage 45.01% 45.01%
=======================================
Files 126 126
Lines 37382 37382
Branches 37382 37382
=======================================
Hits 16828 16828
Misses 19667 19667
Partials 887 887 |
ffb2f48
to
48abd18
Compare
cargo-deny has fixed by #1240. |
@Desiki-high , the title has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/69428 |
@Desiki-high , The CI test is completed, please check result:
Congratulations, your test job passed! |
We should add the read-amount and read-count for nydus benchmark to compare nydus and zran or different batchsize. Signed-off-by: Desiki-high <[email protected]>
48abd18
to
2551c4b
Compare
@jiangliu , the code has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/69509 |
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!
We should add the
read-amount
andread-count
for nydus benchmark to compare nydus, zran with different batchsize, config and so on.