-
Notifications
You must be signed in to change notification settings - Fork 209
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
rafs: only enable digest validate based on configuration #1155
rafs: only enable digest validate based on configuration #1155
Conversation
@imeoer , a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/57313 |
@imeoer , 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/57314 |
@imeoer , 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/57315 |
ae457a9
to
d27d3e7
Compare
d27d3e7
to
61f32e8
Compare
@imeoer , 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/57319 |
@imeoer , 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/57320 |
@imeoer , 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/57321 |
@imeoer , The CI test is completed, please check result:
Congratulations, your test job passed! |
We found that when using the "nydus-image check --bootstrap /path/to/bootstrap" command, it takes about 15s to check a 35MB bootstrap file (rafs v5) due to the default digest validation. This is very slow and disabling it can reduce the time to 3s. We should only allow this option to be configurable at runtime. Signed-off-by: Yan Song <[email protected]>
61f32e8
to
ac2d786
Compare
@imeoer , 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/57684 |
@imeoer , The CI test is completed, please check result:
Congratulations, your test job passed! |
We found that when using the
nydus-image check --bootstrap /path/to/bootstrap
command, it takes about 15s to check a 35MB bootstrap file (rafs v5) due to the
default digest validation. This is very slow, disabling it can reduce the time to 3s.
We should only allow this option to be configurable at runtime.