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

DEMO: Add docker image files #58

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

steven1003
Copy link

This includes a Dockerfile to build a Groundr Docker image, the script file ran inside of the Docker image, and instructions on how to build and test the Docker image.

Copy link

@kev-the-dev kev-the-dev left a comment

Choose a reason for hiding this comment

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

Thanks! I'll see if I can build it later

@steven1003
Copy link
Author

Forgot to mention this, but when build the image before merging this PR, you will need to change the url of the script file to the one in my repo.

Copy link

@kev-the-dev kev-the-dev left a comment

Choose a reason for hiding this comment

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

Docker built fine, but could not run server because jsonschema is not installed, see comments.


To be able to use the demo, do the following to have a working docker image. This command will be ran in the demo directory to build the Groundr Docker image using the provided Dockerfile.

docker build scripts -t groundr

Choose a reason for hiding this comment

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

Should probably be docker build demo/scripts -t groundr so user can run from the repo's root

&& apt-get -y clean \
&& rm -rf /var/lib/apt/lists/* \
&& rm -f /var/cache/apt/*.bin \
&& pip3 install flask \

Choose a reason for hiding this comment

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

Gotta add pip3 install jsonschema

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.

None yet

2 participants