Skip to content
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

Merged

Conversation

benjaminhuanghuang
Copy link
Contributor

Display detailed error when construct FsCacheHandler with a nonexistent path

Files: fs_cache.rs

Signed-off-by: Benjamin Huang benjaminhuanghuang@gmail.com

@yqleng1987
Copy link
Contributor

@benjaminhuanghuang , a new test job has been submitted. Please wait in patience.

@yqleng1987
Copy link
Contributor

@benjaminhuanghuang , The CI test is completed, please check result:

Test CaseTest Result
merge-target-branch✅SUCCESS
build-docker-image✅SUCCESS
compile-nydus❌FAIL

Sorry, your test job failed. Please get the details in the link.

src/bin/nydusd/fs_cache.rs Outdated Show resolved Hide resolved
@yqleng1987
Copy link
Contributor

@benjaminhuanghuang , your pull request has been updated. A new test job will be submitted. Please wait in patience.

@yqleng1987
Copy link
Contributor

@benjaminhuanghuang , the test job has been submitted. Please wait in patience.

@yqleng1987
Copy link
Contributor

@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.

@yqleng1987
Copy link
Contributor

@benjaminhuanghuang , your test job is running now. Please wait in patience.

@yqleng1987
Copy link
Contributor

@benjaminhuanghuang , The CI test is completed, please check result:

Test CaseTest Result
merge-target-branch✅SUCCESS
build-docker-image✅SUCCESS
compile-nydus✅SUCCESS
compile-ctr-remote✅SUCCESS
compile-nydus-snapshotter✅SUCCESS
start-nydus-snapshotter-config-containerd✅SUCCESS
run-container-with-nydus-image✅SUCCESS

Sorry, the current test job result is not valid, because your pull request has been updated, and a new test job will be submitted.

.open(path)
.map_err(|e| {
error!(
"Failed to open cachefiles device {}. {}. Please try `modprobe cachefiles`",
Copy link
Collaborator

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. :)

Copy link
Contributor

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.

Copy link
Contributor Author

@benjaminhuanghuang benjaminhuanghuang Jul 15, 2022

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 ?

@hsiangkao
Copy link
Contributor

/retest

@yqleng1987
Copy link
Contributor

@hsiangkao , the test job has been submitted. Please wait in patience.

@yqleng1987
Copy link
Contributor

@hsiangkao , The CI test is completed, please check result:

Test CaseTest Result
merge-target-branch✅SUCCESS
build-docker-image✅SUCCESS
compile-nydus✅SUCCESS
compile-ctr-remote✅SUCCESS
compile-nydus-snapshotter✅SUCCESS
start-nydus-snapshotter-config-containerd✅SUCCESS
run-container-with-nydus-image✅SUCCESS

Congratulations, your test job passed!

@yqleng1987
Copy link
Contributor

@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>
@yqleng1987
Copy link
Contributor

@benjaminhuanghuang , The CI test is completed, please check result:

Test CaseTest Result
merge-target-branch✅SUCCESS
build-docker-image✅SUCCESS
compile-nydus✅SUCCESS
compile-ctr-remote✅SUCCESS
compile-nydus-snapshotter✅SUCCESS
start-nydus-snapshotter-config-containerd✅SUCCESS
run-container-with-nydus-image✅SUCCESS

Congratulations, your test job passed!

@yqleng1987
Copy link
Contributor

@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 benjaminhuanghuang changed the title Display detailed error when construct FsCacheHandler with a nonexistent path Display detailed error in FsCacheHandler constuctor when cachefiles device not ready Jul 15, 2022
@yqleng1987
Copy link
Contributor

@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 benjaminhuanghuang changed the title Display detailed error in FsCacheHandler constuctor when cachefiles device not ready Display detailed error in FsCacheHandler constructor when cachefiles device not ready Jul 15, 2022
@yqleng1987
Copy link
Contributor

@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

@yqleng1987
Copy link
Contributor

@benjaminhuanghuang , The CI test is completed, please check result:

Test CaseTest Result
merge-target-branch✅SUCCESS
build-docker-image✅SUCCESS
compile-nydus✅SUCCESS
compile-ctr-remote✅SUCCESS
compile-nydus-snapshotter✅SUCCESS
start-nydus-snapshotter-config-containerd✅SUCCESS
run-container-with-nydus-image✅SUCCESS

Congratulations, your test job passed!

@yqleng1987
Copy link
Contributor

@benjaminhuanghuang , The CI test is completed, please check result:

Test CaseTest Result
merge-target-branch✅SUCCESS
build-docker-image✅SUCCESS
compile-nydus✅SUCCESS
compile-ctr-remote✅SUCCESS
compile-nydus-snapshotter✅SUCCESS
start-nydus-snapshotter-config-containerd✅SUCCESS
run-container-with-nydus-image✅SUCCESS

Congratulations, your test job passed!

@yqleng1987
Copy link
Contributor

@benjaminhuanghuang , The CI test is completed, please check result:

Test CaseTest Result
merge-target-branch✅SUCCESS
build-docker-image✅SUCCESS
compile-nydus✅SUCCESS
compile-ctr-remote✅SUCCESS
compile-nydus-snapshotter✅SUCCESS
start-nydus-snapshotter-config-containerd✅SUCCESS
run-container-with-nydus-image✅SUCCESS

Congratulations, your test job passed!

@liubogithub liubogithub merged commit ed95277 into dragonflyoss:master Jul 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants