Distribution EOLE » Commun » creole

Après diagnostic il faut utiliser get_group_infos et non get_container_infos.

Le problème c'est que certaines applications utilisent l'un et d'autre l'autre.

C'est totalement problématique. Voici un exemple :

root@amonecole:~# cat test
%%current_container['interfaces']
root@amonecole:~# CreoleCat -s test -o test.fichier -c fichier
root@amonecole:~# CreoleCat -s test -o test.partage -c partage
root@amonecole:~# md5sum test.partage test.fichier 
34890f87ae91d14f077dd439370eabbf  test.partage
6c0da116cda3116bb262153b2aeb253f  test.fichier

CreoleCat n'est pas fiable !

Voici un code fiable :

root@amonecole:~# cat test.py
from creole.client import CreoleClient, NotFoundError
import sys

cont = sys.argv[1]

c = CreoleClient()

try:
    print c.get_group_infos(cont)['interfaces']
except NotFoundError:
    print c.get_container_infos(cont)['interfaces']

Il faut modifier tous les appels à get_group_infos et get_container_infos par ce code.

Outre le fait que le code est peu compréhensible, il génère inutilement de temps de chargement inutilement lent. Ne voudrait-il mieux pas fusionner les deux méthodes par une seule fiable ?

Avec un conteneur, il génère 31 requêtes sur le serveur et prend 1s50.
Avec un groupe de conteneur, il génère 13 requêtes sur le serveur et prend 1s30.

Si je comprends bien, il faudrait :

1. Retourner les informations du groupe de conteneurs lorsque l’on demande les informations d’un des conteneurs de ce groupe ;
2. Retourner les informations du conteneurs s’il ne fait pas parti du groupe.

Dans ce cas, nous pouvons modifier CreoleClient.get_container_infos() (source:creole/client.py@4183ad6#L608) :

    def get_container_infos(self, container):
        """Get all components of a container or its group

        :param container: container name
        :type container: `str`
        :return: components of the container or its group
        :rtype: `dict`

        """ 
        container_info = self.get_container(container)
        if container != container_info['group']:
            container_info = self.get_group_infos(container_info['group'])
        else:
            container_info.update(self.get_containers_components([container]))
        return container_info