Projet

Général

Profil

Tâche #13729

ERA - Scénario #13707: Script 40-icmp_static_rules non créé

Il n'est pas possible de passer une variable en disabled via le mécanisme de redefine

Ajouté par Emmanuel GARETTE il y a plus de 8 ans. Mis à jour il y a plus de 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Début:
23/10/2015
Echéance:
% réalisé:

100%

Temps estimé:
1.00 h
Temps passé:
Restant à faire (heures):
0.0

Description

L'attribut disabled n'est pas surchargé lors de la redéfinition.

Dans le fichier le fichier /usr/lib/python2.7/dist-packages/creole/var_loader.py, remplacer :

        if vdata['disabled'] is True:
            attrs['disabled'] = True

par :

        if vdata['disabled'] is True or (newdata is not None and newdata['disabled'] is True):
            attrs['disabled'] = True

Révisions associées

Révision 5cc5578d (diff)
Ajouté par Emmanuel GARETTE il y a plus de 8 ans

il est maintenant possible de redéfinir la variable disabled (ref #13729 @1h)

Révision fe130468 (diff)
Ajouté par Emmanuel GARETTE il y a plus de 8 ans

ajout de l'attribut 'remove_condition' pour les variables (ref #13729 @2h)

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

Test unitaire pour redefine et remove_condition

  • tests/dicos/redefine/*.xml : ajout de variables et de contraintes spécifiques
  • tests/test_genconfig.py : ajout des tests associés dans "test_redefine()"

Ref: #13729 @30m

Historique

#1 Mis à jour par Scrum Master il y a plus de 8 ans

  • Statut changé de Nouveau à En cours

#2 Mis à jour par Scrum Master il y a plus de 8 ans

  • Assigné à mis à Emmanuel GARETTE

#3 Mis à jour par Emmanuel GARETTE il y a plus de 8 ans

Sur un AmonEcole etb3, il n'y a pas de différence avec et sans le changement :

root@amonecole:~# cat pouet.py 
from creole.loader import creole_loader; c=creole_loader(load_extra=True);
m = c.make_dict()
keys = m.keys()
keys.sort()
for k in keys:
    print k, m[k]

root@amonecole:~# python pouet.py > avant.txt

Je fais la modification puis :

root@amonecole:~# python pouet.py > apres.txt
root@amonecole:~# diff -u avant.txt apres.txt 
--- avant.txt    2015-12-02 16:14:27.196103818 +0100
+++ apres.txt    2015-12-02 16:18:25.352149845 +0100
@@ -9278,7 +9278,7 @@
 creole.onduleur.nut_ups_name.nut_ups_productid [None]
 creole.onduleur.nut_ups_name.nut_ups_serial [None]
 creole.onduleur.nut_ups_name.nut_ups_upstype [None]
-creole.onduleur.nut_ups_password 50e98059494a752fdea4a48aea36be2631afbacf4ae4aea9a6521ce2
+creole.onduleur.nut_ups_password 36d4d2ffc8bac27b7f3fd855c9630d08edfec62ca83ad627c455f129
 creole.openldap.ldap_cachesize 1000
 creole.openldap.ldap_loglevel 0
 creole.openldap.ldap_replication non

Seul le mot de passe NUT est changé (pas lié à ma modification).

#4 Mis à jour par Emmanuel GARETTE il y a plus de 8 ans

  • % réalisé changé de 0 à 100
  • Restant à faire (heures) changé de 1.0 à 0.25

#5 Mis à jour par Scrum Master il y a plus de 8 ans

  • Statut changé de En cours à Résolu

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

  • Projet changé de ERA à creole

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

  • Statut changé de Résolu à Fermé
  • Restant à faire (heures) changé de 0.25 à 0.0

Validé par la création des tests unitaires.

Formats disponibles : Atom PDF