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

Build Arm64 dependencies image #43

Merged
merged 1 commit into from
Jun 22, 2024

Conversation

gassmoeller
Copy link
Member

@tjhei @luca-heltai FYI

This PR does:

  • update the README.md to recent deal.II versions
  • Proposes to add a github actions workflow that:
    • is only manually executed by clicking a button here https://github.com/dealii/docker-files/actions/new
    • builds the dependencies docker image on github actions machines
    • builds both a amd64 and arm64 docker image
    • pushes those docker images both to the github container registry and the docker hub

This change would allow us as a next step to:

  • build a deal.II image for both amd64 and arm64
  • simplifies rebuilding the images for new deal.II releases

You can see how the final images would look like here: https://github.com/gassmoeller/docker-files/pkgs/container/docker-files (I tested the workflow in my account).

The docker hub upload will not work unless we set up secrets to log into docker hub. If we agree that is a good thing I can do that or someone else can as well. The github container registry upload should word out of the box.

Comments and suggestions are welcome. As discussed with @tjhei I think having a deal.II arm64 image would be worthwhile (because it runs well on the Macs ARM architecture).

@luca-heltai
Copy link
Member

Very nice!

@luca-heltai luca-heltai merged commit cbc943e into dealii:master Jun 22, 2024
@gassmoeller gassmoeller deleted the update_dependencies_arm64 branch June 22, 2024 21:39
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.

2 participants