-
Notifications
You must be signed in to change notification settings - Fork 203
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
Display detailed error in FsCacheHandler constructor when cachefiles device not ready #596
Display detailed error in FsCacheHandler constructor when cachefiles device not ready #596
Conversation
@benjaminhuanghuang , a new test job has been submitted. Please wait in patience. |
@benjaminhuanghuang , The CI test is completed, please check result:
Sorry, your test job failed. Please get the details in the link. |
@benjaminhuanghuang , your pull request has been updated. A new test job will be submitted. Please wait in patience. |
@benjaminhuanghuang , the test job has been submitted. Please wait in patience. |
6b524f8
to
cc122f3
Compare
@benjaminhuanghuang , your pull request has been updated, but a test job is already running. A new test job wil be submitted after the current job is finished. |
@benjaminhuanghuang , your test job is running now. Please wait in patience. |
@benjaminhuanghuang , The CI test is completed, please check result:
Sorry, the current test job result is not valid, because your pull request has been updated, and a new test job will be submitted. |
src/bin/nydusd/fs_cache.rs
Outdated
.open(path) | ||
.map_err(|e| { | ||
error!( | ||
"Failed to open cachefiles device {}. {}. Please try `modprobe cachefiles`", |
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.
"Failed to open cachefiles device {}:" is better. :)
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.
Yeah, I think modprobe cachefiles
is no needed to be seen here, there might be other reasons.
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.
@hsiangkao You are right. Cachefiles device does not work in my env (Ubuntu 22.04, 5.19.0-rc3+, Virtualbox) after cachefiles enabled. Do you know what are the reasons cause this issue ?
/retest |
@hsiangkao , the test job has been submitted. Please wait in patience. |
@hsiangkao , The CI test is completed, please check result:
Congratulations, your test job passed! |
cc122f3
to
294af0f
Compare
@benjaminhuanghuang , 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/10858 |
…device not ready. Files: fs_cache.rs Signed-off-by: Benjamin Huang <benjaminhuanghuang@gmail.com>
@benjaminhuanghuang , The CI test is completed, please check result:
Congratulations, your test job passed! |
294af0f
to
d226879
Compare
@benjaminhuanghuang , 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/10859 |
@benjaminhuanghuang , 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/10860 |
@benjaminhuanghuang , 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/10861 |
@benjaminhuanghuang , The CI test is completed, please check result:
Congratulations, your test job passed! |
@benjaminhuanghuang , The CI test is completed, please check result:
Congratulations, your test job passed! |
@benjaminhuanghuang , The CI test is completed, please check result:
Congratulations, your test job passed! |
Display detailed error when construct FsCacheHandler with a nonexistent path
Files: fs_cache.rs
Signed-off-by: Benjamin Huang benjaminhuanghuang@gmail.com