Projet

Général

Profil

Bug #72162

build Jenkins cassé, getlasso3.sh en erreur

Ajouté par Paul Marillonnet il y a plus d'un an. Mis à jour il y a plus d'un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
08 décembre 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

py3-buster: commands[0]> ./getlasso3.sh
py3-buster: failed with ./getlasso3.sh (resolves to ./getlasso3.sh) is not allowed, use allowlist_externals to allow it
py3-buster: FAIL ✖ in 22.32 seconds

dans le dernier build nocturne : https://jenkins.entrouvert.org/job/authentic/3204/console
Les jobs wip aussi sont cassés.

Fichiers


Demandes liées

Lié à Publik - Support #72165: limiter la version de tox lorsque celui-ci provient de pypiFermé08 décembre 2022

Actions

Révisions associées

Révision 29893d53 (diff)
Ajouté par Paul Marillonnet il y a plus d'un an

ci: limit tox version used while running tests (#72162)

Historique

#1

Mis à jour par Paul Marillonnet il y a plus d'un an

En fait les branches wip ne prennent plus les environnements déclarés dans la section [tox:jenkins] du tox.ini, elles prennent les environnements globaux :

[tox]
toxworkdir = {env:TMPDIR:/tmp}/tox-{env:USER}/authentic/
envlist =
  py3
  py3-buster
  py3-bullseye
  py3-stable-backports
  code-style

J’ai pas compris ce qui avait changé.

#2

Mis à jour par Paul Marillonnet il y a plus d'un an

  • Statut changé de Nouveau à En cours

Ah, c’est le passage à la version 4 de tox qui ne se passe pas bien.

#3

Mis à jour par Paul Marillonnet il y a plus d'un an

#4

Mis à jour par Frédéric Péters il y a plus d'un an

  • Statut changé de Solution proposée à Solution validée

C'est sans doute pertinent d'appliquer ça à tous les modules, tu t'en charges ?

#5

Mis à jour par Paul Marillonnet il y a plus d'un an

Frédéric Péters a écrit :

C'est sans doute pertinent d'appliquer ça à tous les modules, tu t'en charges ?

Après quelques coups de grep il me semble que seuls les plugins d’authentic et wcs-olap sont concernés, j’ai créé #72165.

#6

Mis à jour par Paul Marillonnet il y a plus d'un an

  • Statut changé de Solution validée à Résolu (à déployer)
commit 29893d530ad613e1ab4e5c7d6a08ff890493bd58
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Thu Dec 8 10:14:25 2022 +0100

    ci: limit tox version used while running tests (#72162)
#7

Mis à jour par Paul Marillonnet il y a plus d'un an

  • Lié à Support #72165: limiter la version de tox lorsque celui-ci provient de pypi ajouté
#8

Mis à jour par Benjamin Dauvergne il y a plus d'un an

Ils ont viré ce truc spécifique mais il y a mieux, il y a une notion de label qui permet de lister des environnements à exécuter, https://tox.wiki/en/latest/config.html#labels c'est vers ça qu'il faudrait migrer.

#9

Mis à jour par Frédéric Péters il y a plus d'un an

Mais en local ça reste le tox de la distribution qu'on utilise, il me semble (chez moi oui, et je ne vois pas de références à une installation d'un autre tox dans publik-devinst). On attendrait alors que tox 4 soit disponible dans debian testing avant de faire cette migration ?

#10

Mis à jour par Paul Marillonnet il y a plus d'un an

Frédéric Péters a écrit :

([…] je ne vois pas de références à une installation d'un autre tox dans publik-devinst)

(non rien de tel dans devinst)

#11

Mis à jour par Benjamin Dauvergne il y a plus d'un an

Frédéric Péters a écrit :

Mais en local ça reste le tox de la distribution qu'on utilise, il me semble (chez moi oui, et je ne vois pas de références à une installation d'un autre tox dans publik-devinst). On attendrait alors que tox 4 soit disponible dans debian testing avant de faire cette migration ?

D'un rapide test, avec le patch suivant :

diff --git a/tox.ini b/tox.ini
index 4af84c24..d3945582 100644
--- a/tox.ini
+++ b/tox.ini
@@ -6,19 +6,16 @@
 [tox]
 toxworkdir = {env:TMPDIR:/tmp}/tox-{env:USER}/authentic/
 envlist =
+  update-locales
   py3
   py3-buster
   py3-bullseye
   py3-stable-backports
-  code-style
-
-[tox:jenkins]
-envlist =
-  update-locales
-  py3-bullseye
   check-migrations
   code-style
   pylint
+labels =
+  jenkins = update-locales,py3-bullseye,check-migrations,code-style,pylint

 [testenv]
 setenv =

On a un tox.ini qui fonctionne en version <4 localement avec maintenant toutes les cibles possibles dans toxenv dans le bon ordre (l'ordre dans labels est ignoré, c'est l'ordre de envlist qui prévaut toujours), et sur jenkins en forçant tox>=4 on peut faire tox run -m jenkins et avoir le comportement précédent.

Et on peut créer maintenant en plus un label "jenkins-night" pour les builds complets de nuit, ça simplifierait le jenkins.sh qui pour ce job est obligé de lister explicitement les cibles.

Ça obliger à mettre des trucs dans envlist que je n'exécutais pas moi localement, mais j'ai l'impression que toute le monde préfère si par défaut ça exécute tout et toutes les cibles sont visibles d'un coup d'oeuil. Si l'idée va à tout le monde je peux ouvrir un ticket en ce sens.

#12

Mis à jour par Transition automatique il y a plus d'un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#13

Mis à jour par Transition automatique il y a environ un an

Automatic expiration

Formats disponibles : Atom PDF