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

Add ability to bring up gameboard on AWS. #41

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

Conversation

ezeteze
Copy link

@ezeteze ezeteze commented Jan 15, 2022

  • Add docker image.
  • Add github action to build docker image.
  • Add kubespray to bring up k8s on aws.
  • Add gameboard k8s manifests.

Check k8s/README.md for instructions.

edwinschaap and others added 24 commits October 9, 2019 16:13
- Separated database from Dockerfile
- added docker-compose example
- added .env support, adjusted common.php to read environment variables
- adjusted the sql so it could me imported when mysql initialises
- Bump PHP version to 7.4
- Add missing packages (default-mysql-client, nullmailer, cron)
- Add back apache hardening configs
- Removed UNIQUE constraint from contracts.title (MySQL has maximum of
  3072 bytes per key, VARCHAR(1024) in utf8mb4 has 4096)
@ezeteze ezeteze marked this pull request as ready for review January 15, 2022 23:27
@stamparm
Copy link
Collaborator

@ezeteze lots of changes in one go :)

  1. am I wrong, or this looks like exclusively being done for AWS? i mean, have you tested a standalone Docker run with these changes?
  2. i don't see a CREATE DATABASE for scoreboard in File changes. am I missing smth?

@ezeteze
Copy link
Author

ezeteze commented Feb 20, 2022

Hi @stamparm although this is mainly done for AWS there is also a docker-compose that can be used to bring up the application.
Regarding the create database it's not needed since it's done by the helm chart for k8s

database: "scoreboard"
and by the .env variable for docker-compose
database: "scoreboard"

Still i added the on main.sql to create it if not exists in the case someone tries to bring this up through the setup.sh.

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.

4 participants