Project

General

Profile

Tâche #8365

Distribution EOLE - Scénario #8826: Gestion des conteneurs par les outils Creole

Les variables "container_name_<conteneur>" ne sont pas générées en 2.4

Added by Joël Cuissinat almost 7 years ago. Updated over 6 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Start date:
Due date:
06/20/2014
% Done:

100%

Estimated time:
1.00 h
Spent time:
Remaining (hours):

Description

On doit pouvoir s'en passer mais, en l'état, au moins un template en utilise :

Erreur: Utilisation d'une variable non existante dans le template de /etc/hosts : u'container_name_dns'

creolelint.patch View (804 Bytes) Joël Cuissinat, 07/30/2014 04:33 PM


Related issues

Related to creole - Tâche #8684: get_container_info ne fonctionne pas avec les groupes de conteneurs Fermé
Blocked by ERA - Tâche #8917: Utilisation de variables potentiellement inexistantes dans le template "active_tags" Fermé 09/16/2014

Associated revisions

Revision f5e9356d (diff)
Added by Joël Cuissinat almost 7 years ago

Génération des variables container_name_<conteneur>

Fixes #8365 @30m

Revision 611820a7 (diff)
Added by Joël Cuissinat over 6 years ago

Variables "container_name_*" en mode non conteneur

  • creole/var_loader.py (gen_containers_creole) : traitements spécifiques pour
    le conteneur "all" et le mode "non conteneur"

Fixes: #8365 @30m

Revision 4d97e527 (diff)
Added by Joël Cuissinat over 6 years ago

Suppression des faux positifs CreoleLint

Ref: #8365 @20m [application du patch creolelint.patch]

History

#1 Updated by Joël Cuissinat almost 7 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100

#2 Updated by Joël Cuissinat almost 7 years ago

Pour tester : CreoleGet container_name_mail doit renvoyer root en mode non conteneur et reseau sur AmonEcole.

#3 Updated by Joël Cuissinat almost 7 years ago

  • Status changed from Résolu to Réouvert
  • Target version changed from Eole 2.4.0.1 to 231
  • % Done changed from 100 to 60
  • Parent task set to #8326

Hum, visiblement la correction ne correspond pas à ce que j'ai écrit :

root@horus:~# CreoleGet container_name_mail
mail

En plus, je m'interroge sur l'utilité de ces variables sachant qu'il ne doit y avoir qu'un template qui les utilise ...

#4 Updated by Joël Cuissinat over 6 years ago

Si ces variables sont conservées, voila un patch pour supprimer les faux positifs générés dans CreoleLint :

Libellé de variable non valide dans un dictionnaire (valid_var_label:NAME:warning)
 \-- %%container_name_dhcp : dhcp => DHCP
 \-- %%container_name_ftp : ftp => FTP
 \-- %%container_name_mysql : mysql => MySQL
 \-- %%container_name_internet : internet => Internet

#5 Updated by Joël Cuissinat over 6 years ago

  • Parent task changed from #8326 to #8770

#6 Updated by Joël Cuissinat over 6 years ago

  • Parent task changed from #8770 to #8826

#7 Updated by Daniel Dehennin over 6 years ago

  • Status changed from Réouvert to Nouveau

#8 Updated by Daniel Dehennin over 6 years ago

  • Remaining (hours) set to 0.42

#9 Updated by Joël Cuissinat over 6 years ago

  • Status changed from Nouveau to En cours

#10 Updated by Daniel Dehennin over 6 years ago

Depuis la résolution de #8684 :

root@amonecole:~# CreoleGet container_name_proxy
internet

#11 Updated by Daniel Dehennin over 6 years ago

Le calcule de container_name_<container>[1] se base sur le nom du groupe sans se soucier du real_container[2]

[1] source:creole/var_loader.py@676b0c7#L513
[2] source:creole/var_loader.py@676b0c7#L250

#12 Updated by Joël Cuissinat over 6 years ago

  • Status changed from En cours to Résolu
  • % Done changed from 60 to 100

#13 Updated by Joël Cuissinat over 6 years ago

  • Remaining (hours) deleted (0.42)

#14 Updated by Benjamin Bohard over 6 years ago

  • Status changed from Résolu to Fermé

Paquet testé : 2.4.1-11
Sur amonecole :

for cont in mail mysql dns reseau internet partage all; do echo $cont : $(CreoleGet container_name_$cont); done
mail : reseau
mysql : bdd
dns : internet
reseau : reseau
internet : internet
partage : partage
all : all

Sur eolebase :

CreoleGet container_name_mail
root

Also available in: Atom PDF