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

Add support for azlinux3 #223

Merged
merged 3 commits into from
May 14, 2024
Merged

Add support for azlinux3 #223

merged 3 commits into from
May 14, 2024

Conversation

cpuguy83
Copy link
Member

@cpuguy83 cpuguy83 commented Apr 30, 2024

Most of the changes here is shuffling bits around to allow shared implementations between mariner2 and azlinux3.
Also includes some simplifications around creating an image config.

Closes #210

@cpuguy83 cpuguy83 changed the title Prepare for adding azlinux3 support Add support for azlinux3 Apr 30, 2024
@cpuguy83 cpuguy83 force-pushed the azlinux3 branch 3 times, most recently from 243641f to e0dda1b Compare April 30, 2024 22:33
@cpuguy83 cpuguy83 marked this pull request as ready for review April 30, 2024 22:34
@cpuguy83 cpuguy83 requested a review from a team as a code owner April 30, 2024 22:34
@cpuguy83 cpuguy83 added this to the v0.4.0 milestone May 6, 2024
@cpuguy83 cpuguy83 force-pushed the azlinux3 branch 2 times, most recently from 1ba981a to 89cfcf4 Compare May 9, 2024 23:41
This moves some things around to make the mariner2 implementation
re-usable for azlinux3.
This changes some things in the mariner2 target so we can share as much
as makes sense for azlinux3.

Note: mariner was renamed to azure linux, hence mariner2 -> azlinux3.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
This requires callers to pass in a default image config instead of
expecting the function to resolve a config from an image ref.
This makes things a bit easier to integrate with and not making
assumptions that the base config is going to come from an image ref.

In the future I plan to make the default base config to by built-in
rather than always pulling from an image ref.
This will save a round trip with the registry and makes ure we aren't
relying on an image in the registry that we wouldn't otherwise need to.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
Copy link
Member

@sozercan sozercan left a comment

Choose a reason for hiding this comment

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

Sorry about the delay. LGTM

@cpuguy83 cpuguy83 merged commit 53bcda7 into Azure:main May 14, 2024
9 checks passed
@cpuguy83 cpuguy83 deleted the azlinux3 branch May 14, 2024 19:58
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.

[REQ] Add Azure Linux v3 support
2 participants