Skip to content
This repository was archived by the owner on Jul 14, 2025. It is now read-only.

Add support for pulling Spin OCI images#180

Merged
Mossaka merged 3 commits intodeislabs:mainfrom
jsturtevant:spin-oci
Nov 13, 2023
Merged

Add support for pulling Spin OCI images#180
Mossaka merged 3 commits intodeislabs:mainfrom
jsturtevant:spin-oci

Conversation

@jsturtevant
Copy link
Copy Markdown
Contributor

Replaces #164

Signed-off-by: James Sturtevant <jstur@microsoft.com>
Signed-off-by: James Sturtevant <jstur@microsoft.com>
Copy link
Copy Markdown
Member

@Mossaka Mossaka left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Comment thread containerd-shim-spin/src/engine.rs
Copy link
Copy Markdown
Member

@radu-matei radu-matei left a comment

Choose a reason for hiding this comment

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

Tested this in a new k3d cluster with a Spin app published to a registry using spin registry push and everything works great!

Thank you so much for the work here!

Comment thread containerd-shim-spin/quickstart.md Outdated
Signed-off-by: James Sturtevant <jstur@microsoft.com>
@Mossaka
Copy link
Copy Markdown
Member

Mossaka commented Nov 13, 2023

This can be a follow-up task. Could you please add at least one integration test for testing spin OCI images?

@Mossaka Mossaka merged commit b2d695c into deislabs:main Nov 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants