Projet

Général

Profil

Development #15514

ajouter un paramètre timeout à la commande cook

Ajouté par Frédéric Péters il y a environ 7 ans. Mis à jour il y a presque 7 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Jean-Baptiste Jaillet
Catégorie:
-
Version cible:
-
Début:
20 mars 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour dans certaines configurations (genre déploiement ozwillo) ne pas utiliser la durée par défaut.


Fichiers


Demandes liées

Lié à Hobo - Development #14935: Création d'une app ozwillo dans hobo/contribFermé09 février 2017

Actions

Révisions associées

Révision 78971ab9 (diff)
Ajouté par Jean-Baptiste Jaillet il y a environ 7 ans

environnement: add timeout option to cook command (#15514)

Historique

#1

Mis à jour par Jean-Baptiste Jaillet il y a environ 7 ans

j'ai ajouté l'option timeout à cook.

#2

Mis à jour par Frédéric Péters il y a environ 7 ans

kwargs['timeout'] est ici une chaine de caractères, le résultat c'est que time.time() - t0 > timeout ne sera jamais vrai, que l'attente sera désormais éternelle.

timeout étant systématiquement passé, ne pas inclure timeout=120 dans la signature de la fonction, histoire de ne pas laisser croire que la valeur par défaut s'y définit.

#3

Mis à jour par Jean-Baptiste Jaillet il y a environ 7 ans

voilà j'ai réctifié. (s'il est acké est ce qu'il serait possible de le poussé ou c'est préférable d'attendre le 27/03 ?)

#4

Mis à jour par Frédéric Péters il y a environ 7 ans

  • J+11, du lundi au jeudi, semaine de recette, tests des nouveautés et vérifications de non-régression.

Non ça ne peut pas être poussé dans ce cycle.

#5

Mis à jour par Jean-Baptiste Jaillet il y a environ 7 ans

#6

Mis à jour par Frédéric Péters il y a environ 7 ans

Ajouter type='int' à make_option pour marquer que c'est un entier qui est attendu. Ça permettra aussi d'éviter le cast int() au moment de l'utilisation de la variable.

#8

Mis à jour par Frédéric Péters il y a environ 7 ans

Ok.

#9

Mis à jour par Jean-Baptiste Jaillet il y a environ 7 ans

  • Statut changé de En cours à Résolu (à déployer)

Poussé

#10

Mis à jour par Jean-Baptiste Jaillet il y a presque 7 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF