NCAR / container-dtc-nwp

End-to-end NWP systems in containers.
https://dtcenter.org/community-code/numerical-weather-prediction-nwp-containers
28 stars 13 forks source link

Feature 93 aws port #99

Closed TatianaBurek closed 1 year ago

TatianaBurek commented 1 year ago

Docker file did not get built because of epel-release error. This commit should fix it

Pull Request Checklist

fossell commented 1 year ago

@TatianaBurek - What Dockerfile didn't build? I was testing this morning building from scratch from the main branch (which has previous mods merged already) and everything seemed to work fine. Though I did have to change the metviewer version in docker-compose.yml. Can you provide some more details?

TatianaBurek commented 1 year ago

I got email form the DockerHub with the message that the image was not built and the link to the log: https://hub.docker.com/repository/registry-1.docker.io/dtcenter/nwp-container-metviewer/builds/76fe2bef-d692-40d3-9728-665b42ae687e

fossell commented 1 year ago

@TatianaBurek - We can't view the log file from Dockerhub unfortunately, probably because we're not admins and don't have permissions. So I'm not sure what is going on with the error. Is there a reason that Dockerhub produced a failure but the Github actions test seems to be ok and I was able to build from scratch on my Mac and AWS and run cases successfully?

TatianaBurek commented 1 year ago

Here is the error from the DockerHub:

error: failed to solve: executor failed running [/bin/sh -c yum -y update && yum -y install epel-release && yum -y install 'dnf-command(config-manager)' && yum-config-manager --enable PowerTools]: exit code: 1
Build failed using Buildkit

I remember fixing it in Dockerfile_for_Singularity by enabling PowerTools before installing epel-release but I forgot to make the same change in Dockerfile.

fossell commented 1 year ago

@TatianaBurek - Thanks for the info! I'm testing this now. Not sure why I didn't get errors before, but we need the Dockerhub built properly so we'll need this fix regardless. If everything completes successfully I'll go ahead and merge this to main and then wait to see if our main Dockerhub builds are successful.