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

Move containerd archives from code to yaml #2403

Merged
merged 1 commit into from
Sep 1, 2024

Conversation

afbjorklund
Copy link
Member

@afbjorklund afbjorklund commented Jun 8, 2024

This makes it more similar to update compared to the images, and allows for integrating with other archives in the future*.

@afbjorklund
Copy link
Member Author

For instance, for newer versions of podman:

https://github.com/afbjorklund/podman-full

@afbjorklund
Copy link
Member Author

afbjorklund commented Jun 16, 2024

Possibly the version should be removed from the yaml, and be made a part of the filename instead ?

  • containerd-1.7.6.yaml
  • containerd-2.0.0-beta.5.yaml

Similar to the change made for the "image.yaml".

The Version field is currently not used for anything.

@afbjorklund afbjorklund force-pushed the containerd-yaml branch 2 times, most recently from 8590963 to 48db7d1 Compare June 17, 2024 14:38
@AkihiroSuda AkihiroSuda requested a review from a team July 2, 2024 06:15
@AkihiroSuda AkihiroSuda added this to the v1.0 (tentative) milestone Jul 17, 2024
AkihiroSuda
AkihiroSuda previously approved these changes Jul 17, 2024
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
Copy link
Member

CI is failing, could you try rebasing?

AkihiroSuda
AkihiroSuda previously approved these changes Jul 17, 2024
@AkihiroSuda AkihiroSuda modified the milestones: v0.22.1, v1.0 Jul 25, 2024
@AkihiroSuda
Copy link
Member

Sorry, needs rebase again

This makes it more similar to update compared to the images,
and allows for integrating with other archives in the future.

Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
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 6be4f29 into lima-vm:master Sep 1, 2024
27 checks passed
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