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.

Sign up for GitHub

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

Jump to bottom

Dockerfile.base-spack: update to openmpi 4.1.5 and latest ifort #198

Merged
merged 3 commits into from
Mar 25, 2025
Merged

Dockerfile.base-spack: update to openmpi 4.1.5 and latest ifort #198

merged 3 commits into from
Mar 25, 2025

Conversation

harshula
Copy link
Collaborator

  • Also delete files in sourcecache to reduce the image size.

@harshula harshula self-assigned this Mar 18, 2025
@harshula harshula marked this pull request as draft March 18, 2025 04:39
@harshula
Copy link
Collaborator Author

Test build still running ...

harshula added 2 commits March 25, 2025 15:17
@harshula
Dockerfile.base-spack: copy command sequence from Spack upstream
9746948 
* Spack upstream has share/spack/templates/container/leap-15.dockerfile
@harshula
Dockerfile.base-spack: reduced ifort image from 8.63GB to 6.47GB
b9daa89 
* Delete files in sourcecache
* Reduce layers. Thanks to Tommy for telling me about layers.
@harshula harshula requested a review from CodeGat March 25, 2025 04:55
@harshula harshula marked this pull request as ready for review March 25, 2025 04:56
@harshula
Copy link
Collaborator Author

harshula commented Mar 25, 2025 •
edited
Loading

Successful builds:

spack-dev-suse    v0.22.5-intel-20250318    61f4a964ad55   6 days ago     6.66GB
spack-dev-rocky   v0.22.5-oneapi-20250318   1f9595d0b5a5   6 days ago     4.63GB
spack-dev-rocky   v0.22.5-intel-20250318    59e8b6ee4dee   6 days ago     6.46GB

CodeGat
CodeGat approved these changes Mar 25, 2025
View reviewed changes
Copy link
Member

@CodeGat CodeGat left a comment

Choose a reason for hiding this comment

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

Looks like some good space-saving. Thanks!

@harshula harshula merged commit 461a98f into main Mar 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CodeGat CodeGat CodeGat approved these changes

@aidanheerdegen aidanheerdegen Awaiting requested review from aidanheerdegen

Assignees

@harshula harshula

Labels
None yet
Projects
ACCESS-NRI Infrastructure
Status: Done
Model Release
Status: Done ✅
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@harshula @CodeGat