Add settings for redis and fix settings handling in the provisioning script #531

Merged
mpepin merged 0 commits from Kerl/settings2 into master 2017-04-26 11:19:51 +02:00
mpepin commented 2017-04-24 23:21:17 +02:00 (Migrated from git.eleves.ens.fr)

New settings:

  • REDIS_PASSWD
  • REDIS_HOST
  • REDIS_DB: the database's id
New settings: - `REDIS_PASSWD` - `REDIS_HOST` - `REDIS_DB`: the database's id
delobell commented 2017-04-25 11:12:43 +02:00 (Migrated from git.eleves.ens.fr)

Yolo mais pourquoi pas

Yolo mais pourquoi pas
delobell commented 2017-04-25 11:12:51 +02:00 (Migrated from git.eleves.ens.fr)

resolved all discussions

resolved all discussions
delobell commented 2017-04-25 11:15:14 +02:00 (Migrated from git.eleves.ens.fr)

Si on place REDIS_DB dans secret pourquoi pas aussi y mettre le port ?

Si on place REDIS_DB dans secret pourquoi pas aussi y mettre le port ?
delobell commented 2017-04-25 11:24:34 +02:00 (Migrated from git.eleves.ens.fr)

Ah mais en fait non, parce que la ligne est ajoutée à chaque fois que le provisioning est lancé. Ça marche probablement tout de même mais c'est pas joli.
Mais: https://redis.io/commands/config-rewrite

Ah mais en fait non, parce que la ligne est ajoutée à chaque fois que le provisioning est lancé. Ça marche probablement tout de même mais c'est pas joli. Mais: https://redis.io/commands/config-rewrite
delobell commented 2017-04-25 12:17:48 +02:00 (Migrated from git.eleves.ens.fr)

echo -e "AUTH dummy\nCONFIG SET requirepass dummy\nAUTH dummy\nCONFIG REWRITE" | redis-cli > /dev/null
Pour couvrir tous les cas de figure, dégager les messages "no password set" ou "ok". Et autant mettre le dummy dans une variable au-dessus puis utiliser cette variable.
Si t'as mieux je t'en prie.

``echo -e "AUTH dummy\nCONFIG SET requirepass dummy\nAUTH dummy\nCONFIG REWRITE" | redis-cli > /dev/null`` Pour couvrir tous les cas de figure, dégager les messages "no password set" ou "ok". Et autant mettre le dummy dans une variable au-dessus puis utiliser cette variable. Si t'as mieux je t'en prie.
delobell commented 2017-04-25 12:20:20 +02:00 (Migrated from git.eleves.ens.fr)

Et tu peux virer le service restart en-dessous avec ça

Et tu peux virer le service restart en-dessous avec ça
delobell commented 2017-04-25 12:22:23 +02:00 (Migrated from git.eleves.ens.fr)

Tu me diras le premier auth est inutile...

Tu me diras le premier auth est inutile...
mpepin commented 2017-04-25 21:08:45 +02:00 (Migrated from git.eleves.ens.fr)

Normalement requirepass dummy n'est ajouté qu'une fois au provisioning de la VM. Mais je t'accorde que ce n'est pas beau. Je vais utiliser redis-cli comme tu le suggère, c'est la bonne méthode.

Normalement `requirepass dummy` n'est ajouté qu'une fois au provisioning de la VM. Mais je t'accorde que ce n'est pas beau. Je vais utiliser `redis-cli` comme tu le suggère, c'est la bonne méthode.
mpepin commented 2017-04-25 21:09:16 +02:00 (Migrated from git.eleves.ens.fr)

Bonne idée pour le port

Bonne idée pour le port
mpepin commented 2017-04-25 21:30:06 +02:00 (Migrated from git.eleves.ens.fr)

added 2 commits

  • dbff7740 - Add REDIS_PORT to the settings and secrets
  • fb4258f8 - Set the redis passwd properly in bootstrap.sh

Compare with previous version

added 2 commits * dbff7740 - Add REDIS_PORT to the settings and secrets * fb4258f8 - Set the redis passwd properly in bootstrap.sh [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/227/diffs?diff_id=450&start_sha=52d9979d7c5cbdd13d3eea8eae8bdc9335102ed9)
delobell commented 2017-04-26 11:19:55 +02:00 (Migrated from git.eleves.ens.fr)

merged

merged
delobell commented 2017-04-26 11:20:00 +02:00 (Migrated from git.eleves.ens.fr)

mentioned in commit 4ed5128829

mentioned in commit 4ed51288299e87b2f92e473e858061624934cc8f
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#531
No description provided.