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 Dockerfiles and images for Spack v0.18.0 #30216

Merged
merged 7 commits into from
Apr 22, 2022

Conversation

alalazo
Copy link
Member

@alalazo alalazo commented Apr 21, 2022

fixes #30211
fixes #21131
fixes #18005

closes #14357
closes #23322
closes #23323
closes #19991

This PR updates the list of images we build nightly, deprecating Ubuntu 16.04 and CentOS 8 and adding Ubuntu 20.04, Ubuntu 22.04 and CentOS Stream. It also removes a lot of duplication by generating the Dockerfiles during the CI workflow and uploading them as artifacts for later inspection or reuse.

Modifications:

  • Added Ubuntu 22.04 to the list of supported images
  • Added CentOS Stream to the list of supported images
  • Removed Ubuntu 16.04 from the list of supported images
  • Removed CentOS 8 from the list of supported images
  • Dockerfiles are generated in CI using spack containerize and target the current GITHUB_REF
  • All the Dockerfiles are uploaded as artifacts

@alalazo alalazo force-pushed the features/update_container_images branch 2 times, most recently from 13c1b77 to a6a3e69 Compare April 21, 2022 12:55
@alalazo alalazo force-pushed the features/update_container_images branch from a6a3e69 to df58eba Compare April 21, 2022 12:58
@alalazo alalazo added this to In progress in Spack v0.18.0 release via automation Apr 21, 2022
@alalazo alalazo self-assigned this Apr 21, 2022
@alalazo alalazo changed the title Update Dockerfiles and images for Spack v1.18.0 Update Dockerfiles and images for Spack v0.18.0 Apr 21, 2022
@alalazo alalazo force-pushed the features/update_container_images branch from a8f4315 to 0e31746 Compare April 21, 2022 15:22
@alalazo alalazo marked this pull request as ready for review April 21, 2022 16:04
@alalazo alalazo requested review from opadron and alecbcs April 21, 2022 16:04
Spack v0.18.0 release automation moved this from In progress to Reviewer approved Apr 21, 2022
Copy link
Member

@alecbcs alecbcs left a comment

Choose a reason for hiding this comment

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

Looks good to me! The reuse of the same Dockerfile for Ubuntu 18.04, 20.04, 22.04 threw me for a bit of a loop but using a template like this makes sense when all are so similar.

@alalazo alalazo merged commit f961a11 into spack:develop Apr 22, 2022
Spack v0.18.0 release automation moved this from Reviewer approved to Done Apr 22, 2022
@alalazo alalazo deleted the features/update_container_images branch April 22, 2022 06:51
@vsoch
Copy link
Member

vsoch commented Apr 22, 2022

Amazing timing! Ubuntu 22.04 LTS came out (like today? or this week?) and I was just building with spack 18.04 and was like hmmm this is very old do we have a newer one yet?

... and we do! 🥳

trws pushed a commit to trws/spack that referenced this pull request Apr 29, 2022
This PR updates the list of images we build nightly, deprecating 
Ubuntu 16.04 and CentOS 8 and adding Ubuntu 20.04, Ubuntu 22.04
and CentOS Stream. It also removes a lot of duplication by generating
the Dockerfiles during the CI workflow and uploading them as artifacts
for later inspection or reuse.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
4 participants