Daemon Python ou daemon PHP pour effectuer une requête toutes les X secondes?

Spine · Septembre 16, 2022, 3:13

Bonjour,

J’ai passé le daemon du #plugin-jeeorangetv de Python 2 à Python 3 pour le rendre compatible Bullseye. Mais avant de pousser un PR, je me demande si ce daemon est vraiment nécessaire.
Sauf erreur de ma part, il n’y a pas vraiment de communication entre le plugin et le daemon, celui-ci est utilisé uniquement pour descendre en dessous du cron minute et effectuer une requête curl toutes les X secondes. Il n’y a donc pas de connexion permanente avec le décodeur.
L’appel dans le daemon :

github.com

totoff974/JeeOrangeTv/blob/master/resources/JeeOrangeTvd/JeeOrangeTvd.py#L65

      
        
                    logging.error('Error on read socket : '+str(e))
            
            

            
# ----------------------------------------------------------------------------
            def send_message():
            
            
    action = {}
                action['refresh'] = "maj"
            
            
    try:
                    globals.JEEDOM_COM.add_changes('devices::'+action['refresh'],action)
                except Exception, e:
                    pass
            
            
    time.sleep(_freq_actu)
                return
            
            
def listen():
                logging.debug("|---> Start listening...")
                jeedom_socket.open()
                logging.debug("    |---> Start deamon")

L’appel de la fonction PHP :

github.com

totoff974/JeeOrangeTv/blob/master/core/php/JeeOrangeTv.inc.php#L34

      
        
                echo 'OK';
                die();
            }
            $result = json_decode(file_get_contents("php://input"), true);
            if (!is_array($result)) {
                die();
            }
            
            
if (isset($result['devices'])) {
                foreach ($result['devices'] as $key => $datas) {
                    JeeOrangeTv::MaJ_JSON();
                }
            }

La fonction PHP :

github.com

totoff974/JeeOrangeTv/blob/master/core/class/JeeOrangeTv.class.php#L153

      
        
              }
             */
            
            
/*
             * Fonction exécutée automatiquement tous les jours par Jeedom
              public static function cronDayly() {
            
            
  }
             */
            
            
public function MaJ_JSON() {
                foreach (eqLogic::byType('JeeOrangeTv') as $JeeOrangeTv) {
                    log::add('JeeOrangeTv', 'debug', '|---> Décodeur : ' . $JeeOrangeTv->getName());
                    log::add('JeeOrangeTv', 'debug', '    |---> mise à jour du JSON');
                    $JeeOrangeTv->ActionInfo($JeeOrangeTv->getConfiguration('box_ip'));
                }
            }
            
            
public function autoMaJCommande() {
            }

Qui appelle cette fonction :

github.com

totoff974/JeeOrangeTv/blob/master/core/class/JeeOrangeTv.class.php#L211

      
        
                $retour_action = shell_exec($cmd_html);
                // log
                log::add('JeeOrangeTv', 'debug', '    |---> Décodeur : ' . JeeOrangeTv::getName());
                log::add('JeeOrangeTv', 'debug', '        |---> IP : '. $box_ip);
                log::add('JeeOrangeTv', 'debug', '        |---> cmd operation=09');
                log::add('JeeOrangeTv', 'debug', '            |---> EPG ID : ' . $epg_id . ' envoyé');
                return;
            }
            
            
// recupere les infos du decodeur selon frequence
            public function ActionInfo($box_ip) {
                $localisation = JeeOrangeTv::getConfiguration('template');
            
            
    // etat du decodeur
                $cmd_retour = 'curl -s "http://'.$box_ip.':8080/remoteControl/cmd?operation=10"';
                // execution de la commande
                $retour_action = shell_exec($cmd_retour);
            
            
    // lecture du json depuis le décodeur
                $retour = json_decode($retour_action, true);
                log::add('JeeOrangeTv', 'debug', '        |---> cmd operation=10');

Du coup, je me demande si un daemon Python est vraiment adapté pour faire cela, est-ce qu’un deamon PHP comme dans le #plugin-sonos3 ne serait pas plus simple ? Je ne cherche pas forcément la simplicité mais la méthode qui va consommer le moins de ressources possibles.

pifou · Septembre 16, 2022, 10:30

Bonjour,
Pas sur qu’il soit utile, pour moi l’utilité d’un démon c’est dans le cas où tu veux « écouter » un service extérieur pour réagir instantanément
Ici on dirait que c’est juste une commande jeedom qui déclenche une action curl pour interroger le demon du plugin ? c’est utile de descendre à la seconde, le cron minute ne suffirait pas ?

Salvialf · Septembre 17, 2022, 7:35

Salut @Spine,

A priori oui, si le démon actuel ne fait qu’appeler une fonction de refresh il sera plus simple et plus lisible de passer par un démon php à mon avis.

Spine · Septembre 17, 2022, 1:58

Bonjour,

Merci pour vos retours.

Le curl envoie juste une requête au décodeur qui retourne un JSON pour mettre à jour les commandes info.

Pas pour moi, mais si je passe le retour d’etat de 5 secondes à 1 minute, les utilisateurs risquent de ne pas être content.

Super, merci pour la confirmation, je viens de mettre en place un daemon PHP, ça fonctionne très bien, c’est beaucoup plus simple et en plus ça permet de supprimer les dépendances.

Je teste quelques jours sur ma prod. et je pousse un PR.

Mips · Septembre 17, 2022, 2:33

Salut,
Je suis du même avis mais un détail à avoir en tête avec les démons php: ils vivent bien dans un thread php séparé donc les changements de config par exemple ne seront pas visible par le démon jusqu’à son redémarrage (car la class config « cache » les infos en mémoires par défaut, comportement que l’on peut modifier en forcant le fetch en db; mais il y aura d’autres cas).

Comme les démons python tu vas me dire? oui, mais c’est moins « intuitif » avec le démon php qui en fait est du code dans notre class eqLogic et donc on a vite oublié qu’il a son prorpre thread et on se retrouve à debug pendant 10min en se demandant comment cela se fait que les données ne sont pas à jour alors qu’il suffisait de redémarrer le démon (oui, c’est du vécu )

Spine · Septembre 17, 2022, 3:56

Merci pour ces précisions, même si ça me semble assez logique, ça m’évitera peut-être de me faire avoir une prochaine fois.

system · Septembre 18, 2022, 3:56

Ce sujet a été automatiquement fermé après 24 heures suivant le dernier commentaire. Aucune réponse n’est permise dorénavant.