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

runners: Use ubuntu version as an argument #30

Merged
merged 3 commits into from
Aug 20, 2024
Merged

Conversation

chantra
Copy link
Owner

@chantra chantra commented Aug 20, 2024

Instead of hardcoding version 20.04, use the argument UBUNTU_VERSION and default to "focal" (e.g 20.04). All depedencies accept both 20.04 and focal as a tag to the same container.

Instead of hardcoding version 20.04, use the argument UBUNTU_VERSION and default
to "focal" (e.g 20.04). All depedencies accept both 20.04 and focal as a tag to
the same container.

Signed-off-by: Manu Bretelle <[email protected]>
@chantra chantra force-pushed the use_version_as_arg branch 8 times, most recently from 97d8004 to 4ef63b6 Compare August 20, 2024 07:02
We can now pass a custom UBUNTU_VERSION value to build a specific version of the container.
The name of the container needs to be changed to account for the version name.

Signed-off-by: Manu Bretelle <[email protected]>
@chantra chantra force-pushed the use_version_as_arg branch 2 times, most recently from 5b8968f to 845e5fa Compare August 20, 2024 16:44
The tooling currently expect a tag of {branch}-{arch}.
Previous diff changed this to {branch}-{ubuntu_version}-{arch}.
This change also add the old tag format when building for focal, this way we
won't have to change the tooling while transitioning.

Signed-off-by: Manu Bretelle <[email protected]>
@chantra chantra force-pushed the use_version_as_arg branch from 845e5fa to 153cd9c Compare August 20, 2024 16:51
@chantra chantra merged commit 3685609 into main Aug 20, 2024
6 checks passed
@chantra chantra deleted the use_version_as_arg branch August 20, 2024 19:13
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.

1 participant