-
Notifications
You must be signed in to change notification settings - Fork 5
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
Changed Tutorial 3 config files to support arm architecture #3
Conversation
@sayanchowdhury - Please reivew. |
Thanks a lot for finding and solving this issue. Instead of having an additional file for M1/M2 laptops, I would suggest to update the |
Sure @tormath1, let me send the changes. FYI - I do not have AMD64 machine to test these changes. |
No worries, I'll try on my machine. |
@tormath1 - Thanks a lot! Made the changes, please check. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I tested on my setup, it works as usual. 👌
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file modification can now be dropped.
@psrvere thanks for your contribution, this will benefit to M1/M2 users! |
Existing config files throw error for M1/M2 laptops as template provider is not supported for arm. This PR adds two alternate files
compute.tf.arm
andprovider.tf.arm
which have correct config for M1/M2 laptops. Read me file has also been modified with Instructions.