Produits Entr'ouvert » Passerelle

0001 : ok
0002 : il y a plusieurs trucs qui semblent sans rapport avec les dates
0003 : un ersatz de 0002 ?
0004: ok
0005: ok
0006: je suis plutôt contre, je pense qu'on va les oublier et qu'on se réveillera quand ça plantera, le couteau sous la gorge.

Emmanuel Cazenave a écrit :

0001 : ok
0002 : il y a plusieurs trucs qui semblent sans rapport avec les dates

Yep j'ai oublié de nettoyé les modifications qui permettent de voir les erreurs de warning (le code est super sale mais plus utilisé donc bon...).

0003 : un ersatz de 0002 ?

Si on veut mais c'est dans les tests et pas dans le code passerelle donc j'ai séparé.

0004: ok
0005: ok
0006: je suis plutôt contre, je pense qu'on va les oublier et qu'on se réveillera quand ça plantera, le couteau sous la gorge.

On va détailler un peu :

# to delete when migration to new style MIDDLEWARE is done
   ignore:Old-style:django.utils.deprecation.RemovedInDjango20Warning:django

Impossible que ça plante en prod avant de planter en recette (en gros dès qu'on passera en django 2 ça va planter).

# to delete when using python3, it's a warning inside ElementTree
   ignore:.*tree.iter().*:PendingDeprecationWarning

Impossible que ça plante du tout, c'est même pas dans notre code, ça se résoudra tout seul en passant en python3.

# to delete when migrating the login view to CBV
   ignore:.*class-based.*LoginView():django.utils.deprecation.RemovedInDjango21Warning

Idem que 1, quand login() disparaîtra on ne pourra pas le rater.

# pyproj warning about deprecation of '+init=authority:code which comes from using Proj(init='EPSG:4326') instead of Proj('EPSG:4326')
# I tried the new syntax but it broke some opengis tests
   ignore:.*init.*authority.*code.*syntax is deprecated:DeprecationWarning:pyproj

Là comme je n'ai aucune idée du souci ni pourquoi je n'arrive pas à simplement le résoudre avec la nouvelle syntaxe je veux bien le laisser visible, en mettant juste comme action "module" pour qu'il n'apparaisse qu'une fois.

# wcs root directory must be renamed to fix this one
   ignore:Not importing directory.*/wcs':ImportWarning

Là on s'en fout c'est uniquement dans les tests.

Navré je suis vraiment pas fan. "Ça plantera en recette avant la prod", oui forcément, c'est quand même ce que j'appelle le couteau sous la gorge.

Et plus généralement pourquoi se mettre a cacher des warning ici et pas dans d'autres projets, ça crée une situation particulière dont on oubliera l'existence, je suis pour l'uniformisation, je trouve meilleure et plus simple la situation générale qui est de rien cacher et de traiter vraiment le problème pour les faire disparaître.

(tu trouvera peut-être moins intégriste que moi pour valider).

Emmanuel Cazenave a écrit :

Navré je suis vraiment pas fan. "Ça plantera en recette avant la prod", oui forcément, c'est quand même ce que j'appelle le couteau sous la gorge.

ignore:Old-style:django.utils.deprecation.RemovedInDjango20Warning:django

Bon pour celui là je vais faire la migration donc, en fait n'y a rien de grave ici c'est juste changé le nom d'un setting vu qu'il n'y a aucun middleware dans passerelle.

ignore:.*class-based.*LoginView():django.utils.deprecation.RemovedInDjango21Warning

Je vais le rendre visible une fois (ce qui ne changera absolument au fait que ça va planter en recette avant la prod (vraiment ta remarque n'a pas de sens, on ne regardait déjà pas les warning avant, ça n'y changera rien de voir le warning un jour puis le jour suivant quand on passera la recette en django 2.1 que ça pète). Ça s'est passé à chaque fois comme ça et pourtant on avait les warnings.

Et plus généralement pourquoi se mettre a cacher des warning ici et pas dans d'autres projets, ça crée une situation particulière dont on oubliera l'existence, je suis pour l'uniformisation, je trouve meilleure et plus simple la situation générale qui est de rien cacher et de traiter vraiment le problème pour les faire disparaître.

Je cache les warnings dans a2 aussi, je ne vous demande pas votre avis.

Benjamin Dauvergne a écrit :

Je vais le rendre visible une fois (ce qui ne changera absolument au fait que ça va planter en recette avant la prod (vraiment ta remarque n'a pas de sens, on ne regardait déjà pas les warning avant, ça n'y changera rien de voir le warning un jour puis le jour suivant quand on passera la recette en django 2.1 que ça pète). Ça s'est passé à chaque fois comme ça et pourtant on avait les warnings.

Lauréline a récemment fait un ticket sur je sais plus quel brique pour en corriger, je m'en suis servi comme guide pour migrer docbow à django 1.11 .

Je cache les warnings dans a2 aussi, je ne vous demande pas votre avis.

Oui justement ici tu demande l'avis de quelqu'un mais comme mon avis n'a pas de sens, j'arrête de polluer ce ticket et ne m'en occupe plus.

Pourquoi tu vires le LocalMiddleware ?

Ça a cassé Hobo.