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

feat: Use remote OCI image references as rootfs element #1630

Merged

Conversation

nderjung
Copy link
Member

@nderjung nderjung commented May 6, 2024

Prerequisite checklist

  • Read the contribution guidelines regarding submitting new changes to the project;
  • Tested your changes against relevant architectures and platforms;
  • Ran make fmt on your commit series before opening this PR;
  • Updated relevant documentation.

Description of changes

This PR makes it possible to specify a remote OCI image as a rootfs. This prevents having to use BuildKit in order to retrieve the rootfs, which has become a common operation.

@nderjung nderjung force-pushed the nderjung/feat/initrd/ociimage branch 5 times, most recently from b5d9e9d to 1fb48ed Compare May 6, 2024 10:08
@nderjung
Copy link
Member Author

nderjung commented May 6, 2024

Pipeline is failing due to a different cause, fixed in #1631.

@craciunoiuc craciunoiuc self-requested a review May 7, 2024 07:46
@nderjung nderjung force-pushed the nderjung/feat/initrd/ociimage branch 2 times, most recently from de42a67 to 4ce7f11 Compare May 7, 2024 08:11
Copy link
Member

@craciunoiuc craciunoiuc left a comment

Choose a reason for hiding this comment

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

only one comment after going through the code

initrd/ociimage.go Outdated Show resolved Hide resolved
@nderjung nderjung force-pushed the nderjung/feat/initrd/ociimage branch 2 times, most recently from 4351e08 to 620a048 Compare May 7, 2024 10:34
initrd/ociimage.go Outdated Show resolved Hide resolved
@nderjung nderjung force-pushed the nderjung/feat/initrd/ociimage branch 8 times, most recently from 17e2ce5 to d68458d Compare May 11, 2024 09:06
@nderjung nderjung requested a review from craciunoiuc May 11, 2024 09:10
@craciunoiuc
Copy link
Member

@nderjung you'll have to rebase again :D

This commit makes it possible to specify a remote OCI image as a
rootfs.  This prevents having to use BuildKit in order to retrieve
the rootfs, which has become a common operation.

Signed-off-by: Alexander Jung <[email protected]>
@nderjung nderjung force-pushed the nderjung/feat/initrd/ociimage branch from d68458d to 5cb57ad Compare May 13, 2024 07:48
Copy link
Member

@craciunoiuc craciunoiuc left a comment

Choose a reason for hiding this comment

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

All good here. Thanks!

Reviewed-by: Cezar Craciunoiu [email protected]
Approved-by: Cezar Craciunoiu [email protected]

@craciunoiuc craciunoiuc merged commit 6479a1d into unikraft:staging May 13, 2024
13 checks passed
nderjung added a commit to nderjung/kraftkit that referenced this pull request May 13, 2024
The PR unikraft#1630 accidentally overwrote the changes from unikraft#1646.

This commit reverts this mistake.

Signed-off-by: Alexander Jung <[email protected]>
@nderjung nderjung mentioned this pull request May 13, 2024
4 tasks
nderjung added a commit to nderjung/kraftkit that referenced this pull request May 13, 2024
…#1646.

This commit reverts this mistake.

Signed-off-by: Alexander Jung <[email protected]>
nderjung added a commit to nderjung/kraftkit that referenced this pull request May 13, 2024
The PR unikraft#1630 accidentally overwrote the changes from unikraft#1646.

This commit reverts this mistake.

Signed-off-by: Alexander Jung <[email protected]>
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.

2 participants