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

Adds Optional SQLite Docker #141

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

therealkevinard
Copy link

@therealkevinard therealkevinard commented Oct 9, 2020

This PR allows opt-in SQLite support with different build/run commands.

usage:

  • build checkup:sqlite3 with make docker_sqlite
  • start stack with docker-compose -f docker-compose.sqlite3.yml

The d/c stack includes a separate worker, allowing its own entrypoint directive. eg: checkup every 10s, independent of the frontend.
A sqlite config.js is bind-mounted from ./statuspage/js/config_sqlite3.js

The built docker image is larger: 155MB vs the fs-only image's 33.4 MB.

This is to address #140

@titpetric
Copy link
Contributor

Hey, thanks for the contribution. Could you please just squash the commits so I have a better overview of the proposed change(s) over the multiple commits.

@therealkevinard
Copy link
Author

Of course! 👍

adds optional dockerfile and docker-compose with sqlite
support/config. adds make target for alternate build.

addresses sourcegraph#140
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.

2 participants