Tâche #11403: Plantage CreoleLint sur Amon et AmonEcole 2.5 - creole - Ensemble Ouvert Libre Évolutif

Tâche #11403

Distribution EOLE - Scénario #12022: Résoudre les problèmes CreoleLint constatés sur amon et amonEcole 2.5

Plantage CreoleLint sur Amon et AmonEcole 2.5

Ajouté par Joël Cuissinat il y a presque 9 ans. Mis à jour il y a plus de 8 ans.

Statut:

Fermé

Priorité:

Normal

Assigné à:

Joël Cuissinat

Version cible:

Distribution EOLE - Sprint_2015_29-35 - Équipe MENESR

Début:

22/07/2015

Echéance:

% réalisé:

100%

Temps estimé:

4.00 h

Temps passé:

2.67 h

Restant à faire (heures):

0.0

Description

Erreur originale :

root@amonecole:~# CreoleLint -t /usr/share/eole/creole/distrib/rsyslog-ca.tmpl 

Template Non valide (valid_parse_tmpl:ERROR:error)
 \-- /usr/share/eole/creole/distrib/rsyslog-ca.tmpl
('/usr/share/eole/creole/distrib/rsyslog-ca.tmpl', ':', AttributeError("'NoneType' object has no attribute 'append'",))

Avec les modifications suivantes (à valider) :

--- /usr/lib/python2.7/dist-packages/creole/lint/parsetemplate.py    2015-04-21 11:49:33.000000000 +0200
+++ parsetemplate.py    2015-04-24 16:15:34.853093660 +0200
@@ -318,7 +318,7 @@
         config.read_only()

     def template(self):
-        def get_value(opt_, path_, second_pass):
+        def get_value(opt_, path_, second_pass2):
             try:
                 return getattr(config.creole, path_)
             except PropertiesOptionError, err:
@@ -338,8 +338,10 @@
                 config.read_only()
                 try:
                     return getattr(config.creole, path_)
-                except ConfigError:
-                    second_pass.append((opt_, path_))
+                except ConfigError, err:
+                    if second_pass2 is None:
+                        raise err
+                    second_pass2.append((opt_, path_))
         try:
             is_gen_file = getattr(config, self.file_path)
         except PropertiesOptionError, err:

On obtient :

root@amonecole:~# CreoleLint -t /usr/share/eole/creole/distrib/rsyslog-ca.tmpl 

Template Non valide (valid_parse_tmpl:ERROR:error)
 \-- /usr/share/eole/creole/distrib/rsyslog-ca.tmpl
('/usr/share/eole/creole/distrib/rsyslog-ca.tmpl', ':', ValueError(('erreur de templating', ConfigError("impossible d'effectuer le calcul, l'option wpad_proxyport_eth0 a les propri\xc3\xa9t\xc3\xa9s : ['disabled'] pour : wpad_proxyport_alias_eth0",)),))

Révisions associées

Révision 09d3526a
Ajouté par Joël Cuissinat il y a plus de 8 ans

Merge remote-tracking branch 'lab/1.0'

Ref: #11403

Révision 8bd9db00 (diff)
Ajouté par Joël Cuissinat il y a plus de 8 ans

Correction du traitement d'une exception particulière par CreoleLint

creole/lint/parsetemplate.py : application du patch proposé

Ref: #11403 @20m

Historique

#1 Mis à jour par Joël Cuissinat il y a presque 9 ans

Sujet changé de Plantage CreoleLint sur AmonEcole 2.5 à Plantage CreoleLint sur Amon et AmonEcole 2.5

#2 Mis à jour par Joël Cuissinat il y a presque 9 ans

Tracker changé de Anomalie à Tâche
Temps estimé mis à 4.00 h
Tâche parente mis à #12022
Restant à faire (heures) mis à 4.0

#3 Mis à jour par Joël Cuissinat il y a plus de 8 ans

Assigné à mis à Emmanuel GARETTE
% réalisé changé de 0 à 30

Il y a bien un problème de lancement d'un calcul d'une esclave désactivée utilisant des variables désactivées...

Proposition de correction :

Dans /usr/lib/python2.7/dist-packages/tiramisu/value.py, ligne 546 dans la fonction _valid_slave, gérer l'exception ConfigError sur la ligne suivante :

                    value.append(values._getcallback_value(self.opt,
                                                           index=index))

Ce qui donnerait :

                    try:
                        value.append(values._getcallback_value(self.opt,
                                                               index=index))
                    except ConfigError:
                        pass