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.

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

Add support to overlaybd convert #1517

Merged
merged 1 commit into from
Nov 24, 2022

Conversation

HileQAQ
Copy link
Contributor

@HileQAQ HileQAQ commented Nov 17, 2022

This PR supports using nerdctl image convert to convert docker v2 or OCI images to OverlayBD images.

Before the conversion, users should have the overlaybd-snapshotter binary installed, which build from accelerated-container-image. They can run the command like nerdctl image convert --overlaybd --oci <source_image> <target_image> to convert a docker v2 or OCI image to a OverlayBD image whose tag is <target_image>.

After the conversion, they can use the nerdctl image push command to push the converted OverlayBD image to their registry and then use the nerdctl run --snapshotter overlaybd to run the image.

Signed-off-by: Haoqi Miao [email protected]

@HileQAQ HileQAQ force-pushed the convert branch 15 times, most recently from 20777a2 to ce06425 Compare November 18, 2022 03:54
@HileQAQ HileQAQ marked this pull request as ready for review November 20, 2022 05:32
@BigVan
Copy link
Member

BigVan commented Nov 21, 2022

@AkihiroSuda @fuweid
we add some code to support overlaybd convert in nerdctl. Do you have time to review it?

return errors.New("option --nydus conflicts with --zstdchunked")
}
if !oci {
logrus.Warnln("option --nydus should be used in conjunction with '--oci', forcibly enabling on oci mediatype for nydus conversion")
Copy link
Member

Choose a reason for hiding this comment

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

Where has this gone?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

L188-205 to deal with convert conflict, and L260-262 to show this warning.

Copy link
Member

Choose a reason for hiding this comment

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

Thanks, but "forcibly enabling on oci mediatype for nydus conversion" seems no longer printed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, but "forcibly enabling on oci mediatype for nydus conversion" seems no longer printed

Sorry, it was fixed.

@AkihiroSuda AkihiroSuda added this to the v1.1.0 milestone Nov 21, 2022
if !oci {
logrus.Warnf("option --%s should be used in conjunction with --oci", convertType)
logrus.Warnf("option --%s should be used in conjunction with --oci, forcibly enabling on oci mediatype for %s conversion", convertType, convertType)
Copy link
Member

Choose a reason for hiding this comment

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

"forcibly enabling on oci mediatype" is untrue for estargz.
True for Nydus. Not sure for overlaybd.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, it was fixed in here, mediatype with/without --oci has been confirmed.

Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda AkihiroSuda merged commit 05a7580 into containerd:main Nov 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants