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

Update envtest to support build on M1 machine #366

Closed
wants to merge 1 commit into from

Conversation

simon-mo
Copy link
Collaborator

@simon-mo simon-mo commented Jul 8, 2022

Why are these changes needed?

  • I needed to update the ENVTEST_K8S_VERSION flag to make it possible to run make on m1 mac.
  • I also need to replace go get with go install to ensure Makefile working. (local go version go1.18.3 darwin/arm64)

Related issue number

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

@simon-mo
Copy link
Collaborator Author

simon-mo commented Jul 8, 2022

hmmm looks like that version doesn't quite work. @brucez-anyscale how did you get the make running on m1?

@brucez-anyscale
Copy link
Contributor

hmmm looks like that version doesn't quite work. @brucez-anyscale how did you get the make running on m1?

I think the go version is still 1.17 for kuberay. I do not know special things for local m1 build.
Only the unit test will not work on m1, but build should work.

echo "Downloading $(2)" ;\
GOBIN=$(PROJECT_DIR)/bin go get $(2) ;\
echo "Downloading $(2) to $(PROJECT_DIR)/bin" ;\
GOBIN=$(PROJECT_DIR)/bin go install $(2) ;\
Copy link
Contributor

Choose a reason for hiding this comment

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

this is go 1.17 and go 1.18 difference.

@simon-mo
Copy link
Collaborator Author

ahhh i see the issue now; i'll downgrade to go 1.17

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.

4 participants