Crée un setup proche de celui en production #383

Merged
mpepin merged 0 commits from Kerl/production_settings into master 2016-09-23 22:20:26 +02:00
mpepin commented 2016-09-12 16:58:13 +02:00 (Migrated from git.eleves.ens.fr)

On fait tourner GestioCOF avec daphne derrière un reverse-proxy Apache
sur la VM Vagrant tout comme sur le serveur de production. On peut tester
en local GestioCOF en “conditions réelles”.

Le serveur lancé avec python manage.py runserver 0.0.0.0:8000 est toujours
accessible à la même url localhost:8000.

Le (nouveau) serveur apache est accessible à localhost:8080.

Pour appliquer les changements dans le code au serveur type prod, il faut
relancer le worker : sudo supervisorctl restart worker. Alors que le serveur
de dev se relance tout seul.

NB important : ce patch supprime le mot de passe sur le serveur redis en dev,
pour faire marcher ce nouveau setup avec un version précédente de la VM, il faut
lancer sudo redis-cli config set requirepass ""

Fixes #66
Fixes #9

On fait tourner GestioCOF avec daphne derrière un reverse-proxy Apache sur la VM Vagrant tout comme sur le serveur de production. On peut tester en local GestioCOF en “conditions réelles”. Le serveur lancé avec `python manage.py runserver 0.0.0.0:8000` est toujours accessible à la même url `localhost:8000`. Le (nouveau) serveur apache est accessible à `localhost:8080`. Pour appliquer les changements dans le code au serveur type prod, il faut relancer le worker : `sudo supervisorctl restart worker`. Alors que le serveur de dev se relance tout seul. NB important : ce patch supprime le mot de passe sur le serveur redis en dev, pour faire marcher ce nouveau setup avec un version précédente de la VM, il faut lancer `sudo redis-cli config set requirepass ""` Fixes #66 Fixes #9
bclement commented 2016-09-12 17:01:22 +02:00 (Migrated from git.eleves.ens.fr)

Ajouter un lien vers la section "Lancer le serveur de dév" peut-être ?

Ajouter un lien vers la section "Lancer le serveur de dév" peut-être ?
bclement commented 2016-09-12 17:02:28 +02:00 (Migrated from git.eleves.ens.fr)

mot de passe redis le plus simple -> mot de passe redis. Le plus simple

mot de passe redis le plus simple -> mot de passe redis. Le plus simple
bclement commented 2016-09-12 17:03:43 +02:00 (Migrated from git.eleves.ens.fr)

Rien à voir avec le changement, mais pourquoi ce fichier s'appelle-t-il asgi.py ?

Rien à voir avec le changement, mais pourquoi ce fichier s'appelle-t-il `asgi.py` ?
mpepin commented 2016-09-12 17:05:19 +02:00 (Migrated from git.eleves.ens.fr)

Added 1 commit:

Added 1 commit: * 7e48210a - Correction de port
bclement commented 2016-09-12 17:13:33 +02:00 (Migrated from git.eleves.ens.fr)

Ça serait bien de réutiliser les variables définies dans le bootstrap.sh. {DBUSER}, {DBNAME}, {DBPASSWD} et un coup de sed, système de templating du pauvre ?

Ça serait bien de réutiliser les variables définies dans le bootstrap.sh. `{DBUSER}`, `{DBNAME}`, `{DBPASSWD}` et un coup de sed, système de templating du pauvre ?
bclement commented 2016-09-12 17:14:10 +02:00 (Migrated from git.eleves.ens.fr)

Pourquoi pas dans le requirements.txt?

Pourquoi pas dans le `requirements.txt`?
bclement commented 2016-09-12 17:17:03 +02:00 (Migrated from git.eleves.ens.fr)

LGTM à part les remarques ci-dessus, mais non testé et je n'ai pas vraiment toutes les technos en tête, donc à prendre avec des pincettes - surtout que gestioCOF a l'air de pas mal avoir changé depuis la dernière fois que j'ai regardé le code.

LGTM à part les remarques ci-dessus, mais non testé et je n'ai pas vraiment toutes les technos en tête, donc à prendre avec des pincettes - surtout que gestioCOF a l'air de pas mal avoir changé depuis la dernière fois que j'ai regardé le code.
mpepin commented 2016-09-12 17:59:12 +02:00 (Migrated from git.eleves.ens.fr)

J'ai suivi https://channels.readthedocs.io/en/latest/deploying.html, ça a l'air standard. Je n'ai pas l'acronyme par contre.

J'ai suivi https://channels.readthedocs.io/en/latest/deploying.html, ça a l'air standard. Je n'ai pas l'acronyme par contre.
mpepin commented 2016-09-12 18:16:42 +02:00 (Migrated from git.eleves.ens.fr)

moui bonne idée

moui bonne idée
mpepin commented 2016-09-12 18:18:24 +02:00 (Migrated from git.eleves.ens.fr)

Parce qu'on s'en sert comme un outil de mise en prod, ça ne sert à rien pour dev, c'est pas une lib qu'on utilise au même titre que le reste. Mais si tu penses que c'est mieux là, je l'y mets.

Parce qu'on s'en sert comme un outil de mise en prod, ça ne sert à rien pour dev, c'est pas une lib qu'on utilise au même titre que le reste. Mais si tu penses que c'est mieux là, je l'y mets.
mpepin commented 2016-09-12 19:34:25 +02:00 (Migrated from git.eleves.ens.fr)

Added 1 commit:

Added 1 commit: * 14733c07 - Petits changements
delobell commented 2016-09-14 02:44:29 +02:00 (Migrated from git.eleves.ens.fr)

Seems legit

Seems legit
mpepin commented 2016-09-23 22:20:23 +02:00 (Migrated from git.eleves.ens.fr)

mentioned in commit cf430b5d3b

mentioned in commit cf430b5d3bf35ad00c754d756beabe7d4ca8ebb7
mpepin commented 2016-09-23 22:20:26 +02:00 (Migrated from git.eleves.ens.fr)

Status changed to merged

Status changed to merged
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
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: DGNum/gestioCOF#383
No description provided.