-
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
nydus-test: Replace storage backend with BACKEND_PROXY and adapts github action #570
Conversation
If backend-proxy is not specifield, do not try to mkdir since it will raise exception Signed-off-by: Changewei Ge <gechangwei@bytedance.com>
Signed-off-by: Changewei Ge <gechangwei@bytedance.com>
@changweige , a new test job has been submitted. Please wait in patience. |
@changweige , The CI test is completed, please check result:
Congratulations, your test job passed! |
.github/workflows/it.yml
Outdated
@@ -79,7 +81,9 @@ jobs: | |||
"registry": { | |||
"registry_url": "localhost:5000", | |||
"registry_namespace": "", | |||
"registry_auth": "Y2h1YW5sYW5nLmdjdzoxcWF6MndzeA==" | |||
"registry_auth": "Y2h1YW5sYW5nLmdjdzoxcWF6MndzeA==", |
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 this sensitive?
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.
We can trim it though it should point to any public registry.
Signed-off-by: Changewei Ge <gechangwei@bytedance.com>
7f325f8
to
3297b3a
Compare
@changweige , your pull request has been updated. A new test job will be submitted. Please wait in patience. |
@changweige , your test job has passed, and no need to test again. |
test_api is the only case suite being tested. Let's also replace its storage backend type.
In addition, fill backend proxy configurations into github action.