Project

General

Profile

Development #31166

python3: s'assurer que authentic2.exponential_retry_timeout.ExponentialRetryTimeout.seconds_to_wait retourne un entier

Added by Paul Marillonnet 3 months ago. Updated 2 months ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
06 Mar 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Et là, je ne me souviens plus pourquoi ça ne tourne pas en python3.
Si le relecteur me le demande, je chercherai/relancerai mes tests en local pour en retrouver la raison.

0001-python3-make-the-exp-retry-timeout-seconds_to_wait-r.patch View (1.19 KB) Paul Marillonnet, 06 Mar 2019 08:12 PM

0001-python3-make-the-exp-retry-timeout-seconds_to_wait-r.patch View (876 Bytes) Paul Marillonnet, 25 Mar 2019 06:28 PM

0001-python3-make-the-exp-retry-timeout-seconds_to_wait-r.patch View (876 Bytes) Paul Marillonnet, 02 Apr 2019 06:31 PM


Related issues

Related to Authentic 2 - Development #28276: Fonctionner avec Python3 pour Django1.11 En cours 23 Nov 2018

Associated revisions

Revision 47566924 (diff)
Added by Paul Marillonnet 2 months ago

python3: make the exp retry timeout 'seconds_to_wait' return an int (#31166)

History

#1 Updated by Paul Marillonnet 3 months ago

#2 Updated by Paul Marillonnet 3 months ago

#3 Updated by Benjamin Dauvergne 3 months ago

  • Assignee set to Paul Marillonnet
  • Status changed from Solution proposée to En cours

Parce que None est comparable avec int en python2 et pas en python3, ça marchait ou pas par hasard (en tout cas ça ne traçait pas), remplace juste return par return 0.

#4 Updated by Paul Marillonnet 3 months ago

#5 Updated by Benjamin Dauvergne 3 months ago

Rebaser et renommer la branche dont le nom est trop long ça fait planter le serveur LDAP temporaire.

#6 Updated by Benjamin Dauvergne 3 months ago

  • Status changed from Solution proposée to En cours

#7 Updated by Paul Marillonnet 3 months ago

Benjamin Dauvergne a écrit :

Rebaser et renommer la branche dont le nom est trop long ça fait planter le serveur LDAP temporaire.

D'ac, c'est fait et c'est en attente de build.

#8 Updated by Paul Marillonnet 3 months ago

Bizarre, je ne pensais pas tomber sur ce genre d'erreur depuis l'utilisation de pg_virtualenv introduite dans #31437 :

setting in postgresql.conf.
Error: Port conflict: another instance is already running on /tmp with port 5433
Dropping cluster 9.6/regress ...

#9 Updated by Paul Marillonnet 3 months ago

  • Status changed from En cours to Solution proposée

Relancé le build et c'est bon.
Je n'ai pas le temps de creuser l'affaire maintenant pour ce premier build en échec (peut⁻être une option manquante à l'initialisation du pg_virtualenv ?), je reviendrai dessus plus tard, dans un autre ticket sans doute.

#10 Updated by Benjamin Dauvergne 3 months ago

Paul Marillonnet a écrit :

Relancé le build et c'est bon.
Je n'ai pas le temps de creuser l'affaire maintenant pour ce premier build en échec (peut⁻être une option manquante à l'initialisation du pg_virtualenv ?), je reviendrai dessus plus tard, dans un autre ticket sans doute.

À mon avis il n'y pas grand chose à chercher, il est bien possible que la façon dont pg_virtualenv recherche un port libre ne soit pas adapté à une forte concurrence (et j'ai regardé, il itère depuis 5432, essaye de bind, si ça marche il ferme la socket et prend le port, 100% de chance de faire des collisions régulièrement).

#11 Updated by Paul Marillonnet 3 months ago

Et donc le patch est bien celui-ci.

#12 Updated by Benjamin Dauvergne 3 months ago

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

Ack.

#13 Updated by Paul Marillonnet 2 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 47566924f9f9a2155b231be69a166db34dd72c4f
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Thu Jan 31 14:20:43 2019 +0100

    python3: make the exp retry timeout 'seconds_to_wait' return an int (#31166)

#14 Updated by Frédéric Péters 2 months ago

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

Also available in: Atom PDF