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

chore: update bootc extension to 1.6.0 #204

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

cdrage
Copy link
Contributor

@cdrage cdrage commented Dec 19, 2024

chore: update bootc extension to 1.6.0

Updates bootc extension to 1.6.0

Signed-off-by: Charlie Drage [email protected]

@cdrage cdrage requested review from benoitf and a team as code owners December 19, 2024 16:14
@cdrage cdrage requested review from jeffmaury and axel7083 and removed request for a team December 19, 2024 16:14
@cdrage
Copy link
Contributor Author

cdrage commented Dec 19, 2024

DO NOT MERGE UNTIL QE APPROVAL

@cdrage
Copy link
Contributor Author

cdrage commented Dec 19, 2024

Biggest difference is the switch to /podman-desktop instead of /containers for the ociURI:

ghcr.io/podman-desktop/podman-desktop-extension-bootc:1.6.0

@benoitf I'm assuming the switch will not break anything since it's still a valid ociURL.

@benoitf
Copy link
Contributor

benoitf commented Dec 19, 2024

yes it'll still be ok

@@ -468,6 +468,27 @@
"categories": ["Containers"],
"keywords": ["bootc", "bootable", "OS", "disk"],
"versions": [
{
"version": "1.6.0",
"podmanDesktopVersion": "^1.12.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

should we bump the required version there to a more recent one ? like 1.14 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not really required to be honest since we don't rely on any 1.14+ features, but yeah... may be good to get people to upgrade.

Updates bootc extension to 1.6.0

Signed-off-by: Charlie Drage <[email protected]>
Copy link
Contributor

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

approving, needs to be approved by QE before merging

@cdrage
Copy link
Contributor Author

cdrage commented Dec 20, 2024

@benoitf Approved by QE from Slack, merging!

@cdrage cdrage merged commit ddc79bb into podman-desktop:main Dec 20, 2024
5 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.

2 participants