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

Dockerfile to run in docker #44

Merged
merged 1 commit into from
Feb 25, 2020
Merged

Conversation

ShaneMcC
Copy link
Contributor

@ShaneMcC ShaneMcC commented Feb 24, 2020

This should fix issue #6

Realistically, users are going to want to use docker-compose or so to actually run this and make it work and have mysql dbs etc, and it would be nice to put an example somewhere.

@greboid
Copy link
Contributor

greboid commented Feb 24, 2020

This doesn't have an alias for the resources, so won't be quite right yet

@tsumaru720
Copy link
Owner

Not issue 2 :)

@ShaneMcC ShaneMcC force-pushed the master branch 2 times, most recently from 5547ba6 to 645a5ff Compare February 24, 2020 01:26
@ShaneMcC
Copy link
Contributor Author

Includes AliasMatch stuff now, and also appropriate config to make /conspectus/themes web-accessible.

@ShaneMcC
Copy link
Contributor Author

Some quick testing instructions to help you:

# Run a throw-away database container (when you ctrl-c this it will vanish along with all the data)
docker run --name conspectus_db -e MYSQL_USER=assets -e MYSQL_PASSWORD=asdfsdf -e MYSQL_DATABASE=asset_tracker -e MYSQL_RANDOM_ROOT_PASSWORD=true --rm mysql:latest
# Watch and wait for that to be ready: [Server] /usr/sbin/mysqld: ready for connections

# Build the container
docker build --rm -t conspectus .

# Run the container
docker run --link conspectus_db -e SQL_HOSTNAME=conspectus_db --name conspectus_web -p 800:80 --rm conspectus

# Test at http://localhost:800/

@tsumaru720 tsumaru720 merged commit 22f6786 into tsumaru720:master Feb 25, 2020
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.

3 participants