-
Notifications
You must be signed in to change notification settings - Fork 230
Fix anolis test #1052
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
Fix anolis test #1052
Conversation
Signed-off-by: Changwei Ge <[email protected]>
@changweige , a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/47762 |
docs/nydus-fscache.md
Outdated
@@ -89,13 +89,10 @@ make | |||
mkdir -p /var/lib/containerd-nydus | |||
|
|||
./bin/containerd-nydus-grpc \ | |||
--config-path /etc/nydus/nydusd-config.fscache.json \ | |||
--nydusd-config /etc/nydus/nydusd-config.fscache.json \ | |||
--daemon-mode shared \ |
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.
Is it necessary?
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.
You are right, we can also remove the flag as --fs-driver fscache
enforces it.
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.
fixed
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.
others LGTM!
Signed-off-by: Changwei Ge <[email protected]>
f1dd763
to
5e7b3da
Compare
@changweige , 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/47776 |
@changweige , The CI test is completed, please check result:
Sorry, your test job failed. Please get the details in the link. |
@changweige , The CI test is completed, please check result:
Sorry, your test job failed. Please get the details in the link. |
Update smoke tests on anolis and docs about how to start nydus-snapshotter service.
In addition, simplify how to start nydus-snapshotter by removing cli flags with the same value as default value