search

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

Fix METviewer image creation via Docker build #96

Closed JohnHalleyGotway closed 1 year ago

JohnHalleyGotway commented 1 year ago

While working on development of GitHub actions for #48, I found that running docker build for METviewer no longer works. This task is to update the METviewer Dockerfile(s) to get it building again. Note that there are 3 Dockerfile variants in this directory. Please fix all 3.

The command to build METviewer listed as option 2 on this page now fails. Please see this build log for more details.

cd ${PROJ_DIR}/container-dtc-nwp/components
cd metviewer/METviewer 
docker build -t dtcenter/nwp-container-metviewer:${PROJ_VERSION} .

And here's the error at the end:

The command '/bin/sh -c yum -y update  && yum -y install epel-release  && yum -y install 'dnf-command(config-manager)'  && yum-config-manager --enable PowerTools' returned a non-zero code: 1

See docker_build_metviewer_20220809.txt

TatianaBurek commented 1 year ago

I successfully built the Docket image using feature_93-aws_port branch

fossell commented 1 year ago

@JohnHalleyGotway - It looks like GH Actions built the new image successfully too from PR #95 (although there was still an error doing "docker exec -i metviewer /scripts/common/metv_load_all.ksh mv_sandy). But does the success build satisfy this Issue?

fossell commented 1 year ago

Fixed with #93