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

Support running on ARM #1088

Merged
merged 1 commit into from
Nov 15, 2023
Merged

Support running on ARM #1088

merged 1 commit into from
Nov 15, 2023

Conversation

huwshimi
Copy link
Contributor

Description

Support running inside a multipass on ARM.

Engineering checklist

Check only items that apply

  • Documentation updated
  • Covered by unit tests
  • Covered by integration tests

Test instructions

Note: this PR will need to land so that you can get past the make pull/candid step.

  • Launch a multipass container on an ARM host.
  • Check out this branch.
  • Follow the setup instructions.
  • The docker compose --profile dev up step should complete successfully (you should be able to run make get-local-auth and get a response).

Notes for code reviewers

@@ -1,7 +1,8 @@
#!/bin/sh

# Grab JQ for ease of use.
wget -O jq https://github.com/jqlang/jq/releases/download/jq-1.7/jq-linux-amd64
ARCH=$(arch | sed s/aarch64/arm64/ | sed s/x86_64/amd64/)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This method of getting the arch is a lot less nice than dpkg --print-architecture but the Vault Docker image uses Alpine which doesn't appear to have an easy way to get the base arch.

Copy link
Contributor

@kian99 kian99 left a comment

Choose a reason for hiding this comment

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

Thanks for this!

@kian99 kian99 merged commit 39d9af4 into canonical:v3 Nov 15, 2023
2 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