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.

Sign up for GitHub

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

Jump to bottom

add basic nydusify conversion case #566

Merged
merged 2 commits into from
Jul 6, 2022
Merged

add basic nydusify conversion case #566

merged 2 commits into from
Jul 6, 2022

Conversation

changweige
Copy link
Contributor

The newly added case can measure conversion time and trim unnecessary cases with private testing image reference

@yqleng1987
Copy link
Contributor

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

@yqleng1987
Copy link
Contributor

@changweige , 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

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

@changweige
nydus-test: remove unnecssary cases from testing nydusify
1d0845d 
Some container images can't be accessed for public. So
they are not necessary.

Signed-off-by: Changewei Ge <gechangwei@bytedance.com>
@changweige
nydus-test: add a case for nydusify basic conversion
8402360 
Just perform v5 and v6 image conversion by nydusify with
minial features on

Signed-off-by: Changewei Ge <gechangwei@bytedance.com>
@yqleng1987
Copy link
Contributor

@changweige , 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

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

@yqleng1987
Copy link
Contributor

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

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

@jiangliu jiangliu merged commit 65ebb7f into dragonflyoss:master Jul 6, 2022
bergwolf added a commit to bergwolf/nydus that referenced this pull request Dec 9, 2022
@bergwolf
nydus-test: restore chunk dict test
c8e6a86 
This partly reverts dragonflyoss#566 to get back the chunk dict test.

Signed-off-by: Peng Tao <bergwolf@hyper.sh>
@bergwolf bergwolf mentioned this pull request Dec 9, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jiangliu jiangliu jiangliu approved these changes

Assignees
No one assigned
Labels
anolis_test_pass
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@changweige @yqleng1987 @hsiangkao @jiangliu