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

Anbox: Use ARM64 images #14197

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

AleksaSvitlica
Copy link

This updates the Anbox quickstart to use ARM64 images, now that Pro ARM64 images are available in the marketplace. Anbox runs better on ARM so this is a better default for users.

PR Checklist

Check these items before submitting a PR...

Contribution Guide

Best Practice Guide

  • - Please check this box once you've submitted the PR if you've read through the Contribution Guide and best practices checklist.

Changelog

  • Update the Anbox quickstart defaults to arm64 images

@AleksaSvitlica
Copy link
Author

@microsoft-github-policy-service agree company="Canonical"

@AleksaSvitlica
Copy link
Author

The CI failure is due to the image terms needing to be accepted for the new SKU pro-22_04-lts-arm64 of the 0001-com-ubuntu-pro-jammy offer.

AleksaSvitlica and others added 2 commits December 6, 2024 11:23
This updates the Anbox quickstart to use ARM64 images, now that Pro
ARM64 images are available in the marketplace. Anbox runs better on
ARM so this is a better default for users.
The plan field only needs to be populated when using an AzureVM
image. Populating the field when using a CoreVM will result in an
error. This extra logic ensures the plan will only be populated when
using AzureVM and no Pro token is provided.

This change has been tested with
ubuntu-24_04-lts:ubuntu-pro-arm64 (CoreVM) and
0001-com-ubuntu-pro-jammy:pro-22_04-lts-arm64 (AzureVM).

Co-authored-by: Daniel Draper <[email protected]>
@AleksaSvitlica AleksaSvitlica force-pushed the feat/anbox-quickstart-use-arm64-image branch from 4ed78a6 to b8df026 Compare December 6, 2024 16:23
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