Project

General

Profile

Development #59312

agent authentic : comportement provisionning HTTP par défaut

Added by Paul Marillonnet about 2 months ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
03 Dec 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Pour permettre #58466.


Files


Related issues

Related to Publik - Development #58466: activation provisionning HTTP par défautFermé08 Nov 2021

Actions

Associated revisions

Revision 339ab638 (diff)
Added by Paul Marillonnet about 2 months ago

agent/authentic: use http provisionning by default (#59312)

History

#1

Updated by Paul Marillonnet about 2 months ago

#2

Updated by Paul Marillonnet about 2 months ago

#3

Updated by Paul Marillonnet about 2 months ago

  • Tracker changed from Bug to Development
#4

Updated by Frédéric Péters about 2 months ago

J'aurais plutôt préféré le paramétrage par défaut modifié dans hobo, façon

-        if getattr(settings, 'HOBO_HTTP_PROVISIONNING', False):
+        if getattr(settings, 'HOBO_HTTP_PROVISIONNING', True):
#5

Updated by Paul Marillonnet about 2 months ago

  • Subject changed from HOBO_HTTP_PROVISIONNING = True dans la config debian multitenant to agent authentic : comportement provisionning HTTP par défaut
  • Project changed from Authentic 2 to Hobo
#7

Updated by Thomas Noël about 2 months ago

Peut-être un peu adapter les tests, parce que :

$ git grep HOBO_HTTP_PROVISIONNING tests_authentic/
tests_authentic/test_provisionning.py:    settings.HOBO_HTTP_PROVISIONNING = True
tests_authentic/test_provisionning.py:    settings.HOBO_HTTP_PROVISIONNING = True

#8

Updated by Paul Marillonnet about 2 months ago

  • Status changed from Solution proposée to En cours

Thomas Noël a écrit :

Peut-être un peu adapter les tests, parce que :
[...]

Oui je viens de voir ça, je fais les adaptations.

#11

Updated by Thomas Noël about 2 months ago

  • Status changed from Solution proposée to Solution validée

Ainsi modifiés les tests ne montrent pas que HOBO_HTTP_PROVISIONNING est vrai par défaut... mais je suis un peu tatasse.

Tu pousses ainsi, ou tu ré-ordonnes un peu test_provision_using_http (pour mettre le False à la fin genre)... comme tu veux !

#12

Updated by Paul Marillonnet about 2 months ago

Thomas Noël a écrit :

Ainsi modifiés les tests ne montrent pas que HOBO_HTTP_PROVISIONNING est vrai par défaut... mais je suis un peu tatasse.

Tu pousses ainsi, ou tu ré-ordonnes un peu test_provision_using_http (pour mettre le False à la fin genre)... comme tu veux !

J’avoue que j’aime bien le test qui commence par tester en mode legacy pour ensuite basculer sur le provisionning http et n’en plus revenir :)
J’ai mis des commentaires dans le test pour qu’on comprenne l’esprit de la chose.

#13

Updated by Thomas Noël about 2 months ago

Paul Marillonnet a écrit :

J’avoue que j’aime bien le test qui commence par tester en mode legacy pour ensuite basculer sur le provisionning http et n’en plus revenir :)

:-) Ce que je voulais dire aussi c'est que ça ne permet pas de vérifier que c'est True par défaut, mais let's go.

#14

Updated by Paul Marillonnet about 1 month ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 339ab638558342a4d74f6b96d89ce22edff6b66e
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Fri Dec 3 12:27:23 2021 +0100

    agent/authentic: use http provisionning by default (#59312)
#15

Updated by Paul Marillonnet about 1 month ago

Si jamais ça doit apparaître dans les notes de mise à jour en section dev, quelqe chose comme :

Basculement du provisionning en HTTP par défaut. Le provisionning “legacy” par file de messages (via rabbitmq) doit maintenant faire l’objet d’une configuration explicite à cet effet.

#16

Updated by Frédéric Péters about 1 month ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF