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

Nigel · 2020-11-21T14:12:07Z

Nigel commented

2020-11-21 14:12:07 +00:00

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

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

Ç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

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

Ç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

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

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

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.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: halo-battle/game#2