Development #58404
multiplication des messages de log sync-ldap-users au fur et à mesure des tenants
Début:
05 novembre 2021
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
* Running command on tenant connexion-... 2021-11-05 09:47:18 Synchronising users from realm "ldap" 2021-11-05 09:47:18 Binding to server ldaps://XXX:636 (anonymously) 2021-11-05 09:47:20 Search for uid=* returned 23 users. * Running command on tenant connexion-... 2021-11-05 09:47:20 Synchronising users from realm "ldap" 2021-11-05 09:47:20 Synchronising users from realm "ldap" 2021-11-05 09:47:20 Binding to server ldaps://XXX:636 (anonymously) 2021-11-05 09:47:20 Binding to server ldaps://XXX:636 (anonymously) 2021-11-05 09:47:21 Search for uid=* returned 23 users. 2021-11-05 09:47:21 Search for uid=* returned 23 users. * Running command on tenant connexion-... 2021-11-05 09:47:21 Synchronising users from realm "ldap" 2021-11-05 09:47:21 Synchronising users from realm "ldap" 2021-11-05 09:47:21 Synchronising users from realm "ldap" 2021-11-05 09:47:21 Binding to server ldaps://XXX:636 (anonymously) 2021-11-05 09:47:21 Binding to server ldaps://XXX:636 (anonymously) 2021-11-05 09:47:21 Binding to server ldaps://XXX:636 (anonymously) 2021-11-05 09:47:22 Search for uid=* returned 23 users. 2021-11-05 09:47:22 Search for uid=* returned 23 users. 2021-11-05 09:47:22 Search for uid=* returned 23 users. * Running command on tenant connexion-... 2021-11-05 09:47:22 Synchronising users from realm "ldap" 2021-11-05 09:47:22 Synchronising users from realm "ldap" 2021-11-05 09:47:22 Synchronising users from realm "ldap" 2021-11-05 09:47:22 Synchronising users from realm "ldap" etc.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-commands-prevent-duplicated-log-messages-in-sync-lda.patch 0001-commands-prevent-duplicated-log-messages-in-sync-lda.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
C'est imbitable, ça suppose de savoir que la commande est appelée via tenant_command --all-tenants, on ne sait pas vraiment si on est en console ou pas; comme dans hobo1 on pourrait juste faire ('TERM' in os.environ marche mieux que isatty) :
import os
@contextmanager
def log_ldap_to_console(verbosity):
if 'TERM' not in os.environ:
yield
else:
handler = logging.handlers.StreamHandler...
if verbosity == 1:
handler.setLevel(...)
elif verbosity == ...
...
logger = logging.getLogger('...ldap_backend')
try:
logger.propagate = False
logger.addHandler(handler)
yield
finally:
logger.propagate = True
logger.removeHandler(handler)
...
with log_ldap_to_console(verbosity):
....
[1]: https://git.entrouvert.org/hobo.git/tree/hobo/multitenant/management/commands/__init__.py#n206
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution proposée à En cours
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-commands-prevent-duplicated-log-messages-in-sync-lda.patch 0001-commands-prevent-duplicated-log-messages-in-sync-lda.patch ajouté
- Statut changé de En cours à Solution proposée
Du coup caplog est tout à fait dépassé par ce qu'il se passe, il faut en revenir à mocker logging à la main dans les tests.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-commands-prevent-duplicated-log-messages-in-sync-lda.patch 0001-commands-prevent-duplicated-log-messages-in-sync-lda.patch ajouté
C'était rouge et ça ne l'est plus (je ne reçois plus les mails « build failed » pour mes branches wip sur authentic, juste le mail « back to normal »).
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 30099cc66d4a3b15130cb13669a000bc214c650d
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Tue Nov 9 10:26:24 2021 +0100
commands: prevent duplicated log messages in sync-ldap-users (#58404)Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
commands: prevent duplicated log messages in sync-ldap-users (#58404)