halo-battle/game
1
0
Fork 1
Code Issues Pull requests Projects Releases Activity

Ajout d'un readme et d'un docker-compose #2

Merged
nemunaire merged 3 commits from task/docker-compose into master 2020-11-21 23:17:43 +00:00
Conversation 1 Commits 3 Files changed 4 +89
Nigel commented 2020-11-21 14:12:07 +00:00
Collaborator
Copy link

Suite de https://gitea.nemunai.re/halo-battle/game/pulls/1, c'est mieux quand il y a le docker-compose.yml dans la PR :)

Suite de https://gitea.nemunai.re/halo-battle/game/pulls/1, c'est mieux quand il y a le docker-compose.yml dans la PR :)
nemunaire requested changes 2020-11-21 15:12:50 +00:00
docker-compose.yml Outdated
@ -0,0 +7,4 @@
services:
mariadb:
restart: always
image: mariadb:10.5.8-focal
nemunaire commented 2020-11-21 15:09:32 +00:00
Owner
Copy link

idem image nginx, on a pas envie de gérer les versions spécifiques :

image: mariadb:10-focal
idem image nginx, on a pas envie de gérer les versions spécifiques : ```yaml image: mariadb:10-focal ```
docker-compose.yml
@ -0,0 +11,4 @@
networks:
- halonet
volumes:
- .data:/var/lib/mysql
nemunaire commented 2020-11-21 15:06:54 +00:00
Owner
Copy link

Ça nous intéresse pas de récupérer les datas de MySQL dans un dossier. Tu vois une utilité ?

  - /var/lib/mysql

serait suffisant pour moi

Ça nous intéresse pas de récupérer les datas de MySQL dans un dossier. Tu vois une utilité ? ```yaml - /var/lib/mysql ``` serait suffisant pour moi
Nigel commented 2020-11-21 18:52:48 +00:00
Author
Collaborator
Copy link

J'utilise beaucoup docker et suis donc habitué à régulièrement faire sur ma machine docker rm $(docker -a -q) suivi d'un docker rmi $(docker images) histoire de pas cramer mon disque. Ce .data/ me protège de ce genre de manœuvre.

De plus, ça permet de facilement tester les changements de versions de mariadb, sans avoir besoin de faire un mysql_dump.

Bref, c'est pas idéale, mais je me dit que c'est pas génant ? Surtout que je l'ajoute dans le .gitignore et le .dockerignore. Your call :)

J'utilise beaucoup docker et suis donc habitué à régulièrement faire sur ma machine `docker rm $(docker -a -q)` suivi d'un `docker rmi $(docker images)` histoire de pas cramer mon disque. Ce `.data/` me protège de ce genre de manœuvre. De plus, ça permet de facilement tester les changements de versions de mariadb, sans avoir besoin de faire un `mysql_dump`. Bref, c'est pas idéale, mais je me dit que c'est pas génant ? Surtout que je l'ajoute dans le `.gitignore` et le `.dockerignore`. Your call :)
nemunaire commented 2020-11-21 23:16:14 +00:00
Owner
Copy link

Ça me va.

Ça me va.
nemunaire marked this conversation as resolved
docker-compose.yml Outdated
@ -0,0 +41,4 @@
- DEV=true
nginx:
image: nginx:1.19.4-alpine
nemunaire commented 2020-11-21 15:05:23 +00:00
Owner
Copy link
image: nginx:1-alpine

Cela permettra d'éviter d'avoir à changer le docker-compose à chaque nouvelle version de nginx, 1-alpine devrait toujours marcher, connaissant nginx.

```yaml image: nginx:1-alpine ``` Cela permettra d'éviter d'avoir à changer le docker-compose à chaque nouvelle version de nginx, `1-alpine` devrait toujours marcher, connaissant `nginx`.
nginx.conf Outdated
@ -0,0 +4,4 @@
root /usr/src/hb-main/htdocs;
location / {
try_files $uri $uri?$query_string /index.php?$query_string;
nemunaire commented 2020-11-21 15:02:45 +00:00
Owner
Copy link

Plutôt :

    index index.php;
    try_files $uri $uri/ /index.php;

Il n'y a pas besoin des $query_string pour nous.

Plutôt : ```conf index index.php; try_files $uri $uri/ /index.php; ``` Il n'y a pas besoin des `$query_string` pour nous.
nginx.conf
@ -0,0 +10,4 @@
location ~* \.php$ {
include fastcgi_params;
fastcgi_param SCRIPT_FILENAME $document_root$fastcgi_script_name;
fastcgi_param SCRIPT_NAME $fastcgi_script_name;
nemunaire commented 2020-11-21 15:01:27 +00:00
Owner
Copy link

Il faudrait ajouter :

        fastcgi_param HTTP_HOST $server_name;
        fastcgi_param REMOTE_ADDR $remote_addr;
Il faudrait ajouter : ```conf fastcgi_param HTTP_HOST $server_name; fastcgi_param REMOTE_ADDR $remote_addr; ```
Nigel marked this conversation as resolved
nemunaire merged commit 38851de02b into master 2020-11-21 23:17:43 +00:00
nemunaire deleted branch task/docker-compose 2020-11-21 23:19:19 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
nemunaire
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
halo-battle/game!2
No description provided.