Skip to content

Conversation

@WardF
Copy link

@WardF WardF commented Jul 27, 2021

This PR contains the following changes:

  • The awips_install.sh script is no longer fetched via wget. It is instead stored locally, under the new scripts/ directory, and has been renamed awips_install_local.sh.
  • The -e flag has been added to the install script so that errors cause it to exit immediately.
  • The -x flag has been added to the install script so that the output is verbose when the script runs.
  • rsync and less have been added to the Dockerfile so that they are installed during the docker build process. less is useful in debugging, rsync is invoked by the install script.
  • The arguments to yum clean have been reversed, in order to address a script failure.

Even with these changes, I'm still working to get the docker image to build, but this is s a step in the correct direction. Currently, the install script fails when trying to installs various awips2- packages, awips2-edex, awips2-psql, etc. But this pull request makes debugging easier, and moves past some initial errors and towards these more interesting ones.

With this PR, docker build will succeed. I have been using the following syntax:

$ docker build -t tmp . --no-cache
$ docker run --rm -it tmp bash

Once I have the shell prompt in the running container, I can run $ cd to change to /root/ directory, where the awips_install_local.sh script lives. At this point the script can be run manually with ./awips_install_local.sh, and the subsequent errors can be observed and, hopefully, debugged.

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.

1 participant