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

Fix flavor is not the core one for the provider images #451

Merged
merged 1 commit into from
Jul 21, 2023

Conversation

simonfr
Copy link
Contributor

@simonfr simonfr commented Jul 17, 2023

Pass flavor to OSRELEASE to not use default one defined at https://github.com/kairos-io/kairos/blob/master/Earthfile#L4.

Issue : kairos-io/kairos#1607

@simonfr simonfr force-pushed the patch-kairos-flavor-for-provider branch 2 times, most recently from d5c15b5 to 2b06088 Compare July 17, 2023 11:37
@simonfr simonfr marked this pull request as ready for review July 17, 2023 11:40
Copy link
Member

@mudler mudler left a comment

Choose a reason for hiding this comment

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

Ooops, Good catch, thank you @simonfr !

@mudler mudler enabled auto-merge (squash) July 17, 2023 13:21
@simonfr
Copy link
Contributor Author

simonfr commented Jul 17, 2023

ARM with buildx check KO seems to not be related to this MR. See for example, previous commit : https://github.com/kairos-io/provider-kairos/actions/runs/5573877300

Maybe the reason is : earthly/earthly#2957

@simonfr simonfr force-pushed the patch-kairos-flavor-for-provider branch from 2b06088 to 983fa8d Compare July 21, 2023 10:40
@Itxaka
Copy link
Member

Itxaka commented Jul 21, 2023

No need to wait for CI, thanks @simonfr for the patch!

@Itxaka Itxaka disabled auto-merge July 21, 2023 11:29
@Itxaka Itxaka merged commit 360f1f1 into kairos-io:main Jul 21, 2023
@simonfr
Copy link
Contributor Author

simonfr commented Jul 21, 2023

Thank you @Itxaka. When do you plan to release the next version ?

@Itxaka
Copy link
Member

Itxaka commented Jul 21, 2023

Thank you @Itxaka. When do you plan to release the next version ?

umm, not sure, we could do a minor bump of the provider-kairos only to bundle this fix..

@Itxaka
Copy link
Member

Itxaka commented Jul 21, 2023

Thank you @Itxaka. When do you plan to release the next version ?

How about today? :P

https://github.com/kairos-io/provider-kairos/releases/tag/v2.3.1

@mauromorales
Copy link
Member

Fixes kairos-io/kairos#1622

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.

4 participants