Skip to content
This repository was archived by the owner on Aug 1, 2024. It is now read-only.

Conversation

@NIXKnight
Copy link
Contributor

@NIXKnight NIXKnight commented Sep 7, 2022

The new MySQL 5.7 images uses Docker Buildx to build container image for multiple platforms. Currently the image is built for amd64 and arm64 CPU architectures.

@johnnagro
Copy link
Contributor

@NIXKnight this seems to work on M1s just fine. Can we rebase this PR and mark it for review so we can get someone from ArchBOM to approve it? With this merged, we no longer need (and can close) the apple-silicone branch. Thank you.

@johnnagro johnnagro mentioned this pull request Sep 13, 2022
1 task
@NIXKnight NIXKnight marked this pull request as ready for review September 14, 2022 11:01
@NIXKnight
Copy link
Contributor Author

@johnnagro I have re-based the branch. The PR is ready for review.

@kdmccormick
Copy link
Contributor

👀 Watching along because I'm interested in whether we could do the same thing in Tutor

Copy link
Contributor

@rgraber rgraber left a comment

Choose a reason for hiding this comment

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

LGTM! Hooray!

@johnnagro johnnagro merged commit 7e6b9df into master Sep 15, 2022
@johnnagro johnnagro deleted the NIXKnight/PSRE-1800 branch September 15, 2022 15:30
@kdmccormick
Copy link
Contributor

@johnnagro is the build script for these images internal or is it something I could peek at?

@NIXKnight
Copy link
Contributor Author

NIXKnight commented Sep 15, 2022

@johnnagro is the build script for these images internal or is it something I could peek at?

@kdmccormick We are planning to update the configuration repo with a new MySQL Dockerfile and a GitHub Actions workflow that builds the image.

@kdmccormick
Copy link
Contributor

Nice, thanks Saad.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants