Gladys sur Unraid

jgcb00 · Mars 4, 2022, 9:02

Ok c’est ça que je comprend pas bien :

ça correspond au valeur par defaut ? ou à ce qui est actuellement utilisé sur le docker ?
Par ce que si c’est ce qui est actuellement en cours sur le docker c’est facile à récupérer. Donc je suis persuader que je comprend pas bien le problème vu que la solution m’a l’air simple

VonOx · Mars 4, 2022, 9:06

Bah la c’est simple mais si ton path c’est /ici/toto y’a aucun moyen de deviner que c’est le mount sur l’hôte.

Ça pourrait le faire mais à voir ce que les autres en pensent.

J’essaye d’anticiper les problèmes

J’ai éditer mon précédent message

jgcb00 · Mars 4, 2022, 9:16

Donc si j’applique à mon cas tu obtiens :

[
'/var/run/docker.sock:/var/run/docker.sock',
'/run/udev:/run/udev',
'/dev:/dev',
'/mnt/user/appdata/Gladys:/var/lib/gladysassistant'
]

Sauf qu’il y a en permanence la même chose à droite le /var/lib/gladysassistant/ il change jamais donc c’est ça qui permet de l’identifier non ?

Edit après avoir vu ton edit : oui je suis parfaitement d’accord avec toi, mais toi tu semblais avoir trouver une méthode pour le faire automatiquement correct ou pas ?

VonOx · Mars 4, 2022, 9:20

En théorie oui ça ne change pas.

Edit: oui l’idée c’est de lister les mount, de chercher /var/lib/gladysassistant et de prendre la valeur à gauche

pierre-gilles · Mars 7, 2022, 8:51

Le /var/lib/gladysassistant il vient du SQLITE_FILE_PATH, donc je ne dirais pas non plus qu’il bouge jamais, mais oui il est connu dans le code

Du coup pour récapituler, deux options:

Option 1: Extraire le dossier depuis les volumes montés, en trouvant le bon dossier grâce au SQLITE_FILE_PATH
Option 2: Variable d’environnement, genre GLADYS_FOLDER_PATH_ON_HOST, ce qui est peut-être plus explicite et plus prédictible. Surtout que dans le cas où on lance Gladys via Syno ou Unraid, on peut très bien ajouter cette variable.

guim31 · Mars 7, 2022, 9:09

Moi qui utilises UNRAID depuis assez longtemps, je pense que l’option 2 est la meilleure. D’autant qu’il est facile dans le template pour Unraid de commenter les champs à remplir pour guider l ’ utilisteur si besoin

pierre-gilles · Mars 7, 2022, 4:11

Pour rebondir sur tout ça, je suis allé voir le code pour essayer de voir si je pouvais le faire, mais en fait tout ça est déjà codé du coup je comprend plus rien

L’intégration Zigbee2mqtt appelle bien la fonction basePath:

github.com

GladysAssistant/Gladys/blob/master/server/services/zigbee2mqtt/lib/installMqttContainer.js#L31

      
        
            
            
if (dockerContainers.length === 0) {
              let containerMqtt;
              try {
                logger.info('MQTT broker is being installed as Docker container...');
                logger.info(`Pulling ${containerDescriptor.Image} image...`);
                await this.gladys.system.pull(containerDescriptor.Image);
            
            
    // Prepare broker env
                logger.info(`Preparing broker environment...`);
                const { basePathOnContainer, basePathOnHost } = await this.basePath();
                const brokerEnv = await exec(`sh ./services/zigbee2mqtt/docker/gladys-z2m-mqtt-env.sh ${basePathOnContainer}`);
                logger.trace(brokerEnv);
                containerDescriptor.HostConfig.Binds.push(`${basePathOnHost}/zigbee2mqtt/mqtt:/mosquitto/config`);
            
            
    logger.info(`Creating container...`);
                containerMqtt = await this.gladys.system.createContainer(containerDescriptor);
                logger.trace(containerMqtt);
                this.mqttExist = true;
              } catch (e) {
                logger.error('MQTT broker failed to install as Docker container:', e);

 const { basePathOnContainer, basePathOnHost } = await this.basePath();

Qui elle même retourne bien le path côté host:

github.com

GladysAssistant/Gladys/blob/master/server/services/zigbee2mqtt/lib/basePath.js#L23

      
        
                const base = process.env.SQLITE_FILE_PATH;
                basePathOnContainer = base.substring(0, base.lastIndexOf('/'));
              }
              // Find mount linked to this path to fetch host path
              const gladysMounts = await this.gladys.system.getContainerMounts(os.hostname());
              if (gladysMounts) {
                const baseMount = gladysMounts.find((mount) => {
                  return mount.Destination === basePathOnContainer;
                });
                if (baseMount) {
                  return { basePathOnContainer, basePathOnHost: baseMount.Source };
                }
              }
              return { basePathOnContainer, basePathOnHost: '/var/lib/gladysassistant' };
            }
            
            
module.exports = {
              basePath,
            };

Donc je comprend pas pourquoi ça marche pas, le code est là !

@cicoub13 dis moi si ça te dit quelque chose !

VonOx · Mars 7, 2022, 5:29

En lisant le code j’ai l’impression que c’est inversé, qu’on récupère le path côté container, je me trompe ?

github.com

GladysAssistant/Gladys/blob/master/server/services/zigbee2mqtt/lib/installMqttContainer.js#L32

      
        
            if (dockerContainers.length === 0) {
              let containerMqtt;
              try {
                logger.info('MQTT broker is being installed as Docker container...');
                logger.info(`Pulling ${containerDescriptor.Image} image...`);
                await this.gladys.system.pull(containerDescriptor.Image);
            
            
    // Prepare broker env
                logger.info(`Preparing broker environment...`);
                const { basePathOnContainer, basePathOnHost } = await this.basePath();
                const brokerEnv = await exec(`sh ./services/zigbee2mqtt/docker/gladys-z2m-mqtt-env.sh ${basePathOnContainer}`);
                logger.trace(brokerEnv);
                containerDescriptor.HostConfig.Binds.push(`${basePathOnHost}/zigbee2mqtt/mqtt:/mosquitto/config`);
            
            
    logger.info(`Creating container...`);
                containerMqtt = await this.gladys.system.createContainer(containerDescriptor);
                logger.trace(containerMqtt);
                this.mqttExist = true;
              } catch (e) {
                logger.error('MQTT broker failed to install as Docker container:', e);
                this.mqttExist = false;

gladys-z2m-mqtt-env.sh ${basePathOnContainer}

pierre-gilles · Mars 7, 2022, 5:32

C’est normal, cette partie se déroule dans le container.

L’endroit où on définit le volume, c’est ici:

github.com

GladysAssistant/Gladys/blob/master/server/services/zigbee2mqtt/lib/installMqttContainer.js#L34

      
        
            try {
              logger.info('MQTT broker is being installed as Docker container...');
              logger.info(`Pulling ${containerDescriptor.Image} image...`);
              await this.gladys.system.pull(containerDescriptor.Image);
            
            
  // Prepare broker env
              logger.info(`Preparing broker environment...`);
              const { basePathOnContainer, basePathOnHost } = await this.basePath();
              const brokerEnv = await exec(`sh ./services/zigbee2mqtt/docker/gladys-z2m-mqtt-env.sh ${basePathOnContainer}`);
              logger.trace(brokerEnv);
              containerDescriptor.HostConfig.Binds.push(`${basePathOnHost}/zigbee2mqtt/mqtt:/mosquitto/config`);
            
            
  logger.info(`Creating container...`);
              containerMqtt = await this.gladys.system.createContainer(containerDescriptor);
              logger.trace(containerMqtt);
              this.mqttExist = true;
            } catch (e) {
              logger.error('MQTT broker failed to install as Docker container:', e);
              this.mqttExist = false;
              this.gladys.event.emit(EVENTS.WEBSOCKET.SEND_ALL, {
                type: WEBSOCKET_MESSAGE_TYPES.ZIGBEE2MQTT.STATUS_CHANGE,

pierre-gilles · Mars 7, 2022, 5:34

Par contre, il y a quelque chose qui ne veut pas, cette ligne mutate l’objet json global, donc ça fait push à chaque fois que cette fonction est appelée, et ça touche à la variable globale, ça va pas ça

pierre-gilles · Mars 7, 2022, 5:36

Idem ici ( https://github.com/GladysAssistant/Gladys/blob/master/server/services/zigbee2mqtt/lib/installMqttContainer.js#L34 ), le push mutate l’objet global.

VonOx · Mars 7, 2022, 5:37

C’est ça qui pose problème tu penses ?

pierre-gilles · Mars 7, 2022, 5:39

Je pense pas que ce soit le bug ici, mais en tout cas c’est sûr que ce problème cause des bugs

pierre-gilles · Mars 7, 2022, 5:57

Je me suis lancé une petite VM pour tester tout ça.

J’ai lancé un container avec les paramètres suivants :

docker run -d \
// retiré pour lisibilité
-v /var/lib/gladysassistant_other_folder:/var/lib/gladysassistant \
gladysassistant/gladys:v4

J’ai rajouté des logs, et j’ai lancé un container Zigbee2mqtt, et ça donne ça :

2022-03-07T18:55:44+0100 <info> installMqttContainer.js:25 (Zigbee2mqttManager.installMqttContainer) MQTT broker is being installed as Docker container...
2022-03-07T18:55:44+0100 <info> installMqttContainer.js:26 (Zigbee2mqttManager.installMqttContainer) Pulling eclipse-mosquitto:2 image...
2022-03-07T18:55:47+0100 <info> installMqttContainer.js:30 (Zigbee2mqttManager.installMqttContainer) Preparing broker environment...
{
  basePathOnContainer: '/var/lib/gladysassistant',
  basePathOnHost: '/var/lib/gladysassistant'
}
[ '/var/lib/gladysassistant/zigbee2mqtt/mqtt:/mosquitto/config' ]
2022-03-07T18:55:47+0100 <info> installMqttContainer.js:37 (Zigbee2mqttManager.installMqttContainer) Creating container...
2022-03-07T18:55:47+0100 <info> installMqttContainer.js:51 (Zigbee2mqttManager.installMqttContainer) MQTT broker is restarting...

Le basePathOnHost n’est pas bon, il y a un bug dans la fonction basePath

J’enquête…

pierre-gilles · Mars 7, 2022, 6:02

J’avance plus loin, après avoir ajouté des logs, l’appel:

const gladysMounts = await this.gladys.system.getContainerMounts(os.hostname());

Renvoie un tableau vide… Ainsi, gladysMounts est vide et le default est appliqué…

Je continue mon enquête

pierre-gilles · Mars 7, 2022, 6:08

Je pense que je comprend, la fonction attend un containerId :

github.com

GladysAssistant/Gladys/blob/master/server/lib/system/system.getContainerMounts.js#L10

      
        
            const { PlatformNotCompatible } = require('../../utils/coreErrors');
            
            
/**
             * @description Return list of mounts for this container.
             * @param {string} containerId - Id of the container.
             * @returns {Promise} Resolve with list of mounts.
             * @example
             * const binds = await getContainerMounts('e24ae1745d91');
             */
            async function getContainerMounts(containerId) {
              if (!this.dockerode) {
                throw new PlatformNotCompatible('SYSTEM_NOT_RUNNING_DOCKER');
              }
            
            
  const containers = await this.dockerode.listContainers({
                filters: { id: [containerId] },
              });
              const [container] = containers;
              if (!container) {
                return [];

Et derrière la fonction envoie le hostname :

github.com

GladysAssistant/Gladys/blob/master/server/services/zigbee2mqtt/lib/basePath.js#L17

      
        
            * basePath();
            */
            async function basePath() {
             let basePathOnContainer = '/var/lib/gladysassistant';
             // Fetch container path mount
             if (process.env.SQLITE_FILE_PATH) {
               const base = process.env.SQLITE_FILE_PATH;
               basePathOnContainer = base.substring(0, base.lastIndexOf('/'));
             }
             // Find mount linked to this path to fetch host path
             const gladysMounts = await this.gladys.system.getContainerMounts(os.hostname());
             if (gladysMounts) {
               const baseMount = gladysMounts.find((mount) => {
                 return mount.Destination === basePathOnContainer;
               });
               if (baseMount) {
                 return { basePathOnContainer, basePathOnHost: baseMount.Source };
               }
             }
             return { basePathOnContainer, basePathOnHost: '/var/lib/gladysassistant' };
            }

Il faudrait envoyer l’id du container actuelle

pierre-gilles · Mars 7, 2022, 6:10

J’ai créé une issue Github avec les détails :

github.com/GladysAssistant/Gladys

Zigbee2mqtt : basePath function doesn't work, it doesn't return the path of the folder on the host

opened 06:10PM - 07 Mar 22 UTC

Pierre-Gilles

bug zigbee2mqtt

The function here needs the containerId => https://github.com/GladysAssistant/Gl…adys/blob/master/server/lib/system/system.getContainerMounts.js#L10 But instead, sends the os.hostname() => https://github.com/GladysAssistant/Gladys/blob/master/server/services/zigbee2mqtt/lib/basePath.js#L17 https://community.gladysassistant.com/t/gladys-sur-unraid/6959/37?u=pierre-gilles

Et :

jgcb00 · Mars 8, 2022, 9:24

Merci beaucoup pour ces avancées,

Avec la publication de la 4.8 ça va être l’occasion de tester la mise à jour sur unraid parce que je ne fais pas tourner de watchtower…
Par conséquent, j’attend de voir si il me propose de mettre à jour l’image docker avec une nouvelle

pierre-gilles · Mars 8, 2022, 9:25

Tiens nous au courant! Par contre, ce dont j’ai parlé par rapport à Unraid (les fixs lié au Zigbee2mqtt) ne sont pas dans cette 4.8, ce sera dispo dans la prochaine maj J’ai travaillé dessus ce matin c’était trop tard pour partir dans la 4.8.

jgcb00 · Mars 8, 2022, 9:29

Oui, je m’en doutais il y a pas de problème, l’instance fonctionne de mon côté, on est pas pressé plus le temps passe et plus gladys est compétitif donc autant laissé un peu de temps.

Merci beaucoup en tout cas, très sympa la mise à jour !