Tâche #23842
Scénario #23861: Finaliser la correction des scripts eole-schedule
manage_schedule ne permet pas d’activer certaines tâches
Description
Il y a une incohérence dans la gestion de l’option "mode" entre tiramisu et manage_schedule. Cette option ne doit plus pouvoir être surchargée à l’activation de la tâche.
Faire évoluer manage_schedule et pyeole.schedule en conséquence.
Révisions associées
Ne plus prendre en compte l’option mode
Ref #23842
Rendre plus explicite l’aide associée aux options de manage_schedule.
Ref #23842
Correction du message d’aide pour l’option obsolète mode.
Ref #23842
Adaptations pyeole liées à la suppression du "mode"
Ref: #23842
Historique
#1 Mis à jour par Benjamin Bohard il y a presque 6 ans
- Statut changé de Nouveau à Résolu
- Assigné à mis à Benjamin Bohard
- % réalisé changé de 0 à 100
- Temps estimé mis à 0.25 h
- Restant à faire (heures) mis à 0.25
#2 Mis à jour par Joël Cuissinat il y a presque 6 ans
- Statut changé de Résolu à En cours
#3 Mis à jour par Joël Cuissinat il y a presque 6 ans
La doc est également à mettre à jour : http://eole.ac-dijon.fr/documentations/2.6/partielles/beta/Eolebase/co/09a-gestionSchedule.html
Si on reprend les exemples de la doc sans l'option -m, on a une erreur :
root@scribe:~# manage_schedule -a once -s majauto 'NoneType' object has no attribute 'startswith'
On a obtient également une erreur dans l'EAD si on désactive puis ré-active la mise à jour hebdomadaire :
Le backend a renvoyé une erreur lors de l'exécution de la requête :
Traceback: <class 'tiramisu.error.PropertiesOptionError'>: ne peut modifier la valeur de l'option "mode" cette option n'est pas modifiable
#4 Mis à jour par Joël Cuissinat il y a presque 6 ans
- Tâche parente changé de #23541 à #23742
- Restant à faire (heures) changé de 0.25 à 1.0
Visiblement, il manque des commit et un nouveau paquet python-pyeole
J'ai déjà poussé un début de correction pour 2.6.1 : #23854 à merger ?
Du coup on a commencé de déboguer avec Manu mais ce boulot a sans doute déjà été fait ???
diff --git a/pyeole/schedule.py b/pyeole/schedule.py
index 35f3f03..4503a68 100644
--- a/pyeole/schedule.py
+++ b/pyeole/schedule.py
@@ -40,15 +40,21 @@ def _list_schedules(schedules):
def add_schedule(day, mode, script):
orig = join(SCHEDULE_SCRIPT_DIR , script)
+ if not isfile(orig):
+ raise Exception('unable to find schedule script {0}'.format(orig))
+ if mode is None:
+ mode = client.get('schedule.{}.mode'.format(script))
dest = join(SCHEDULE_DIR, day, mode, script)
if isfile(dest):
raise Exception('{0} already exists'.format(dest))
- if not isfile(orig):
- raise Exception('unable to find schedule script {0}'.format(orig))
- else:
- symlink(orig, dest)
+ symlink(orig, dest)
def del_schedule(day, mode, script):
+ orig = join(SCHEDULE_SCRIPT_DIR , script)
+ if not isfile(orig):
+ raise Exception('unable to find schedule script {0}'.format(orig))
+ if mode is None:
+ mode = client.get('schedule.{}.mode'.format(script))
dest = join(SCHEDULE_DIR, day, mode)
old_filename = join(dest, script)
if not islink(old_filename):
#5 Mis à jour par Joël Cuissinat il y a presque 6 ans
- Tâche parente changé de #23742 à #23861
#6 Mis à jour par Joël Cuissinat il y a presque 6 ans
- Assigné à changé de Benjamin Bohard à Joël Cuissinat
#7 Mis à jour par Joël Cuissinat il y a presque 6 ans
- Restant à faire (heures) changé de 1.0 à 0.5
#8 Mis à jour par Joël Cuissinat il y a presque 6 ans
- Temps estimé changé de 0.25 h à 3.00 h
#9 Mis à jour par Scrum Master il y a presque 6 ans
- Statut changé de En cours à Résolu
#10 Mis à jour par Fabrice Barconnière il y a presque 6 ans
- Restant à faire (heures) changé de 0.5 à 0.0
Test sur etb1.amon 2.6.2, la ligne de commande et l'activation/désactivation dans l'EAD ne posent plus de problème.
#11 Mis à jour par Fabrice Barconnière il y a presque 6 ans
Doc OK :
À partir d'EOLE 2.6.1, l'option -m (ou --mode) est obsolète. Le mode (pre ou post) utilisé pour un script est forcément celui qui a été déclaré dans le dictionnaire extra. Les commandes actuelles sont toujours fonctionnelles mais sont susceptibles d'afficher des avertissements.
#12 Mis à jour par Scrum Master il y a presque 6 ans
- Statut changé de Résolu à Fermé