Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Nydusify: fix some bug about the subcommand mount of nydusify #1328

Merged
merged 1 commit into from
Jun 19, 2023

Conversation

lihuahua123
Copy link
Contributor

@lihuahua123 lihuahua123 commented Jun 18, 2023

Relevant Issue (if applicable)

this is the solution of issue #1306

Details

we fix the bug, the nydusify mount subcommand will not need the --backend-type and --backend-config options when the blobs of target image are all stored in the registry,

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I have added tests to cover my changes.

    - The `nydusify mount` subcomend don't require `--backend-type` and `--backend-config` options when the backend is registry.
    - The methord to resolve it is we can get the `--backend-type` and `--backend-config` options  from the docker configuration.
    - Also, we have refractor the code of checker module in order to reuse the code

Signed-off-by: lihuahua123 <771725652@qq.com>
@lihuahua123 lihuahua123 requested a review from a team as a code owner June 18, 2023 02:48
@lihuahua123 lihuahua123 requested review from jiangliu, changweige and adamqqqplay and removed request for a team June 18, 2023 02:48
@anolis-bot
Copy link
Collaborator

@lihuahua123 , a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/79034

@anolis-bot
Copy link
Collaborator

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

Test CaseTest Result
build rust golang image✅ SUCCESS
compile nydusd✅ SUCCESS
compile ctr remote✅ SUCCESS
compile nydus snapshotter✅ SUCCESS
run container with rafs✅ SUCCESS
run container with zran✅ SUCCESS
run container with rafs and compile linux✅ SUCCESS

Congratulations, your test job passed!

@anolis-bot
Copy link
Collaborator

@lihuahua123 , 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/79067

@anolis-bot
Copy link
Collaborator

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

Test CaseTest Result
build rust golang image✅ SUCCESS
compile nydusd✅ SUCCESS
compile ctr remote✅ SUCCESS
compile nydus snapshotter✅ SUCCESS
run container with rafs✅ SUCCESS
run container with zran✅ SUCCESS
run container with rafs and compile linux✅ SUCCESS

Congratulations, your test job passed!

Copy link
Member

@adamqqqplay adamqqqplay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jiangliu jiangliu requested a review from imeoer June 19, 2023 03:24
@imeoer imeoer merged commit a508ddd into dragonflyoss:stable/v2.2 Jun 19, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants