Produits Entr'ouvert » Authentic 2

Je suis en train de regarder. A priori juste un changement tout à fait minime dans les vues d’api.

Valentin Deniaud a écrit :

0002, j'aurais bien juste viré le check de version, notamment le le test est rendu caduc par 0001 et le code n'est plus bien couvert.

Oui ok très bien. Pendant un moment on cherchait à ne pas volontairement casser le code pour des versions de dépendance obsolètes s’il existe un autre moyen de procéder, mais c’est vrai que ce sera plus simple ici en virant cette vérification de version, les gens qui utilisent une version obsolète de DRF n’ont qu’à pas appeler ce endpoint :)

L'archéologie donne #49447 vieux d'un an, ouvert à cause d'une trace en local, découlant de la situation décrite par #49447#note-2 qui n'est plus vraie aujourd'hui, dans mon devinst j'ai :
[...]
donc on a le droit d'augmenter la version minimale de DRF pour authentic à >=3.9 dans le setup.py (0004), ça ne cassera pas l'installation des autres briques.

Ok, parfait, merci pour le travail d’exhumation :)

0005, ça ne sera pas testé par Jenkins, perso j'aurais juste monté la version de l'environnement de test sans en ajouter un nouveau (et en profiter pour simplement virer ce suffixe DRF et faire un pas vers un nom d'environnement qui matche celui des autres briques)

Je dis ça comme ça mais il vaut sûrement mieux attendre l'avis de Benj avant d'appliquer ces remarques.

Sans contrainte sur la version de DRF ça teste actuellement en 3.12.4 sur Jenkins. Ce n’est pas la version de DRF disponible dans buster mais on peut estimer que ce n’est pas un problème pour autant. D’ailleurs est-ce qu’on a une date fixée pour la migration à bullseye ?

Le patches à jour, maintenant au nombre de quatre, visibles ici.

Ok pour tout, sauf virer les cibles drf39/drf312, ne le faire qu'une fois qu'on sera en bullseye et là effectivement on peut juste imposer drf312 de base. Mais bon ça reviendra quand testing sera en drf31x et pas nous enfin, ça n'a rien d'exceptionnel de tester son soft au dessus de diverses versions de ses dépendances, c'est une bonne pratique, après c'est bien de le faire sans ralentir les build des branches wip/* en général, la nuit par exemple.

Benjamin Dauvergne a écrit :

Ok pour tout, sauf virer les cibles drf39/drf312, ne le faire qu'une fois qu'on sera en bullseye et là effectivement on peut juste imposer drf312 de base. Mais bon ça reviendra quand testing sera en drf31x et pas nous enfin, ça n'a rien d'exceptionnel de tester son soft au dessus de diverses versions de ses dépendances, c'est une bonne pratique, après c'est bien de le faire sans ralentir les build des branches wip/* en général, la nuit par exemple.

Tu veux que le job jenkins principal teste les deux environnements drf3.9 et 3.12 ?
Je n’ai pas d’autres idées que de faire un if (env.JOB_NAME 'authentic' && env.GIT_BRANCH 'origin/main') et de relancer un second tox dans cet environnement juste sur le fichier tests/test_api, est-ce que ça irait ?

Paul Marillonnet a écrit :

Benjamin Dauvergne a écrit :

Ok pour tout, sauf virer les cibles drf39/drf312, ne le faire qu'une fois qu'on sera en bullseye et là effectivement on peut juste imposer drf312 de base. Mais bon ça reviendra quand testing sera en drf31x et pas nous enfin, ça n'a rien d'exceptionnel de tester son soft au dessus de diverses versions de ses dépendances, c'est une bonne pratique, après c'est bien de le faire sans ralentir les build des branches wip/* en général, la nuit par exemple.

Tu veux que la job jenkins principal teste les deux environnements drf3.9 et 3.12 ?
Je n’ai pas d’autres idées que de faire un if (env.JOB_NAME 'authentic' && env.GIT_BRANCH 'origin/main') et de relancer un second tox dans cet environnement juste sur le fichier tests/test_api, est-ce que ça irait ?

Quelque chose comme ça, peut-être.

Ouch, oui bien vu la typo dans le debian/control.
Voilà, tout pris en compte.

Fire.

commit 31fa1dbf4612933733c850823d264df43d2f7c91
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Thu Dec 16 11:03:26 2021 +0100

    jenkins: run drf3.12 env on main branch nightly build (#59601)

commit aec2a4fa6fafa8cdc004f5ebf88b8df8da0b6ada
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Mon Dec 13 18:09:43 2021 +0100

    tox: update main env for more recent restframework testing (#59601)

commit e8b901df260c9a2fcc0ab9684de6dd806da762bf
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Mon Dec 13 18:07:27 2021 +0100

    setup: update upper & lower bounds on restframework version (#59601)

commit 3e85e0d68ecd558cc650688cec98c0d37d7860e9
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Mon Dec 13 18:05:54 2021 +0100

    debian: allow for djangorestframework bullseye package (#59601)

commit 58f58ca337dde15c126fcba7d1d18133947b6b88
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Mon Dec 13 18:35:26 2021 +0100

    api: drop deprecated drf version check (#59601)