Projet

Général

Profil

Development #18525

utiliser un ini pour les options de uwsgi

Ajouté par Thomas Noël il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
08 septembre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Actuellement on a un ExecStart=/usr/bin/uwsgi --option --option ...

C'est pas très pratique quand on veut modifier localement les paramètres de lancement, par exemple le nombre de workers.

Je serais en faveur de passer par un /etc/wcs/uwsgi.ini qui permettrait de profiter des variables magiques (processes=%k), placeholders, @(uwsgi_local.ini) etc. (cf http://uwsgi-docs.readthedocs.io/en/latest/Configuration.html et http://uwsgi-docs.readthedocs.io/en/latest/ConfigLogic.html)

Autre possibilité : laisser les options dans le ExecStart mais les terminer par « --ini /tmp/uwsgi_options.ini », vide par défaut, qui permette de redéfinir des choses si besoin


Fichiers

Révisions associées

Révision 4cdbfa58 (diff)
Ajouté par Thomas Noël il y a plus de 6 ans

debian: use a uwsgi.ini file (#18525)

Révision ba12f5f5 (diff)
Ajouté par Thomas Noël il y a plus de 6 ans

debian: fix service, use ini instead of unknown init option (#18525)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 6 ans

Ok mais mécanisme à définir de manière générale; pas juste w.c.s.

#2

Mis à jour par Thomas Noël il y a plus de 6 ans

Yep, bien sûr. J'ai mis ça sur wcs parce que les autres sont encore en gunicorn.

#3

Mis à jour par Thomas Noël il y a plus de 6 ans

j'ajoute dans le ini:
  • un procname-prefix-spaced = wcs pour que les processus aient des noms lisibles (sinon on voit juste /usr/bin/uwsgi --init /etc/wcs/uwsgi.ini
  • la possibilité d'un /etc/wcs/uwsgi-local.ini pour faire les manips locales à la config, et donc normalement ne jamais toucher à uwsgi.ini
#4

Mis à jour par Thomas Noël il y a plus de 6 ans

  • Statut changé de Nouveau à En cours

Ce que donne le auto-procname :

wcs      14434 74.6  1.4 165752 60032 ?        Ss   14:30   0:02 wcs uWSGI master
wcs      14451  0.0  1.2 165752 53400 ?        S    14:30   0:00  \_ wcs uWSGI worker 1
wcs      14452  0.0  1.2 165752 53400 ?        S    14:30   0:00  \_ wcs uWSGI worker 2

#5

Mis à jour par Frédéric Péters il y a plus de 6 ans

J'adore endif =. Mais ok.

#6

Mis à jour par Thomas Noël il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 4cdbfa58eb631e64e7e1c60c6dc8a160ea5da4d2
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Fri Sep 29 14:07:31 2017 +0200

    debian: use a uwsgi.ini file (#18525)

#7

Mis à jour par Thomas Noël il y a plus de 6 ans

Et c'était --ini et pas --init ... corrigé par :

commit ba12f5f58cf2a7444ab83f65c11a17eb3b7504dd
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Sun Oct 8 16:19:56 2017 +0200

    debian: fix service, use ini instead of unknown init option (#18525)

diff --git a/debian/wcs.service b/debian/wcs.service
index 2dd29b61..4d33e967 100644
--- a/debian/wcs.service
+++ b/debian/wcs.service
@@ -9,7 +9,7 @@ User=%p
 Group=%p
 ExecStartPre=/usr/bin/wcs-manage migrate
 ExecStartPre=/usr/bin/wcs-manage collectstatic
-ExecStart=/usr/bin/uwsgi --init /etc/%p/uwsgi.ini
+ExecStart=/usr/bin/uwsgi --ini /etc/%p/uwsgi.ini
 ExecReload=/bin/kill -HUP $MAINPID
 KillSignal=SIGQUIT
 PrivateTmp=true

#8

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF