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

Adds support for downloading arm64 binaries #10

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jonathanberube-zylo
Copy link

Summary | Résumé

This PR updates the shell script to add automatic support for macOS operating systems and ARM64 architectures. It also makes a minor adjustment to the test suite so that it will pass regardless of the version of terraform is running, which incidentally resolves #8.

Test instructions | Instructions pour tester la modification

This PR can be tested on the new operating systems and architectures fairly easily when running on a Mac running on arm64. The only necessary command to run is the test_script.sh script.

In order to test on a Linux operating system, I used Docker and included the following files:

Dockerfile

FROM ubuntu

RUN apt update && apt install -y wget unzip

WORKDIR /app
COPY . /app
RUN ./test_script.sh

.dockerignore

Dockerfile
test_dir

Then ran the following steps to build / test:

docker build --tag terraform-tools-setup .
docker build --platform=linux/amd64 --tag terraform-tools-setup .

Alternately the source code could be mounted into a virtual machine or physical hardware with the appropriate CPU architecture and Operating Systems.

* Updates get_tools to check CPU architecture
* Includes arm64 CPU support
* Includes darwin OS support
* Refactors test to only review terraform version
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.

Update Terraform version
1 participant