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

server/etcdmain: add build support for Apple M1 #13545

Merged
merged 1 commit into from
Dec 21, 2021

Conversation

dbussink
Copy link
Contributor

@dbussink dbussink commented Dec 17, 2021

This has been additionally verified by running the tests locally as a basic smoke test. GitHub Actions doesn't provide MacOS M1 (arm64) yet, so there's no good way to automate testing.

Ran TMPDIR=/tmp make test locally. The TMPDIR bit is needed so there's no really long path used that breaks Unix socket setup in one of the tests.

The level of support would be tier 3 in https://etcd.io/docs/v3.5/op-guide/supported-platform/ which is the same as the existing Darwin amd64 support.

This has been additionally verified by running the tests locally as a
basic smoke test. GitHub Actions doesn't provide MacOS M1 (arm64) yet,
so there's no good way to automate testing.

Ran `TMPDIR=/tmp make test` locally. The `TMPDIR` bit is needed so
there's no really long path used that breaks Unix socket setup in one of
the tests.
@ptabor ptabor merged commit 1e4a345 into etcd-io:main Dec 21, 2021
@dbussink dbussink deleted the build-apple-m1 branch December 22, 2021 08:49
@dbussink
Copy link
Contributor Author

@ptabor @hexfusion I was looking at updating the docs page, but it looks like there's no directory set up yet for 3.6 which is I imagine the first version that will have this support?

@spzala
Copy link
Member

spzala commented May 10, 2022

@dbussink we have https://etcd.io/docs/v3.6/op-guide/supported-platform/ now. I believe this is the doc page that you were looking to update? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants