-
Notifications
You must be signed in to change notification settings - Fork 214
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
Improve nydusify build subcommand #721
Conversation
Fix a minor in Makefile for musl static build. Signed-off-by: Jiang Liu <[email protected]>
@jiangliu , a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/21220 |
@jiangliu , The CI test is completed, please check result:
Congratulations, your test job passed! |
72acb28
to
dd51401
Compare
@jiangliu , 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/21221 |
dd51401
to
8f607ec
Compare
@jiangliu , 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/21222 |
@jiangliu , The CI test is completed, please check result:
Congratulations, your test job passed! |
8f607ec
to
6cc206d
Compare
@jiangliu , 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/21225 |
@jiangliu , The CI test is completed, please check result:
Congratulations, your test job passed! |
6cc206d
to
a18d85c
Compare
@jiangliu , 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/21226 |
@jiangliu , The CI test is completed, please check result:
Congratulations, your test job passed! |
@jiangliu , The CI test is completed, please check result:
Congratulations, your test job passed! |
Fix several possible resource leakages. Signed-off-by: Jiang Liu <[email protected]>
a18d85c
to
cb69660
Compare
@jiangliu , 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/21238 |
@jiangliu , The CI test is completed, please check result:
Congratulations, your test job passed! |
Usage: "Pack a directory to nydus bootstrap and blob", | ||
Name: "build", | ||
Aliases: []string{"pack"}, | ||
Usage: "Build a Nydus image from a source directory", |
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.
It does not actually generate the image manifest, so maybe we can't say build a nydus image
. :)
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.
How abot "Build a Nydus filesystem from a source directory"?
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.
Looks good.
Flags: []cli.Flag{ | ||
&cli.StringFlag{ | ||
Name: "source-dir", | ||
Aliases: []string{"target-dir"}, // for compatibility | ||
Required: true, | ||
Usage: "The source directory of build target", | ||
Usage: "Source directory to build image from", |
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.
Ditto.
&cli.BoolFlag{ | ||
Name: "backend-push", | ||
Value: false, | ||
Usage: "Push Nydus blob to storage backend", | ||
Usage: "Push generated Nydus image to storage backend", |
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.
Ditto.
Improve help messages and fix a crash bug related to oss. Signed-off-by: Jiang Liu <[email protected]>
cb69660
to
a9a436f
Compare
@jiangliu , 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/21459 |
@jiangliu , The CI test is completed, please check result:
Congratulations, your test job passed! |
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, cc @hhhhsdxxxx PTAL.
LGTM |
Improve nydusify build subcommand and fix some resource leakages.