Patch pour plugins adressing

benoit_glpi · 2008-05-28 14:48:54

Hello,

Le plugins addressing de GPLI ignorait royalement les IPs des équipements réseau que l'on pouvais trouver dans :
# Central >
# Inventaire >
# Réseaux

Ce que dans mon cas posait problème car sur une même classe d'adresse on peut avoir des ordinateurs mais aussi des équipement réseaux.

Nous avons donc modifié le plugin pour qu'il prenne en compte les adresses IPs de cette catégorie.

Voici le patch pour les gens intéréssés :

*** plugin_addressing.classes.php.original    2008-05-28 14:08:05.040187268 +0200
--- plugin_addressing.classes.php    2008-05-28 14:10:31.571792219 +0200
***************
*** 86,96 ****
          if ($this->fields["network"]) 
              $sql .= " AND network=".$this->fields["network"];
              
!         foreach ($CFG_GLPI["netport_types"] as $type) if ($type != NETWORKING_TYPE) {
!             $sql .= " UNION SELECT port.ID, device_type, on_device, dev.name AS dname, port.name AS pname, ifaddr, ifmac, FK_users, INET_ATON(ifaddr) AS ipnum " .
!                     "FROM glpi_networking_ports port, " . $LINK_ID_TABLE[$type] . " dev " .
!                     "WHERE device_type=$type AND port.on_device=dev.ID AND INET_ATON(ifaddr)>=$ipdeb AND INET_ATON(ifaddr)<=$ipfin AND deleted=0 AND is_template =0 " .
!                     getEntitiesRestrictRequest(" AND ", "dev");
              if ($this->fields["network"] && $type!=PERIPHERAL_TYPE && $type!=PHONE_TYPE) 
                  $sql .= " AND network=".$this->fields["network"];
          }
--- 86,96 ----
          if ($this->fields["network"]) 
              $sql .= " AND network=".$this->fields["network"];
              
!         foreach ($CFG_GLPI["netport_types"] as $type) {
!             $sql .= " UNION SELECT port.ID, device_type, on_device, dev.name AS dname, port.name AS pname, port.ifaddr AS ifaddr, port.ifmac AS ifmac, FK_users, INET_ATON(port.ifaddr) AS ipnum " .
!                     "FROM glpi_networking_ports port, " . $LINK_ID_TABLE[$type] . " dev " .
!                     "WHERE device_type=$type AND port.on_device=dev.ID AND INET_ATON(port.ifaddr)>=$ipdeb AND INET_ATON(port.ifaddr)<=$ipfin AND deleted=0 AND is_template =0 " .
!                     getEntitiesRestrictRequest(" AND ", "dev");
              if ($this->fields["network"] && $type!=PERIPHERAL_TYPE && $type!=PHONE_TYPE) 
                  $sql .= " AND network=".$this->fields["network"];
          }

tsmr · 2008-05-28 15:11:06

Quelle version du plugin pour quelle version de glpi ???

remi · 2008-05-28 16:17:38

Pas bon.

Il ne faut pas prendre les IP associés aux ports des équipements réseaux (qui sont, normalement celle des équipements connectés), mais l'IP du matériel.

++

benoit_glpi · 2008-05-28 17:16:33

humm, désolé Rémi, je comprend pas ton message.

Les IPs sont (chez nous) associée à des ports (Ordinateur -> Connexion -> ajouter un port)

Je ne vois pas ce que tu entend par "IP du matériel".

Peux tu préciser pour m'aider à comprendre ?

Pour Tsmr : plugin version 1.6 (checkout du trunk du 20 mai à 12:52), GLPI version 0.70.2

Last edited by benoit_glpi (2008-05-28 17:17:05)

remi · 2008-05-28 17:23:47

Pour les équipements (ordinateurs, imprimantes, ...) les IP sont bien associés à un port.

Quand tu connectes le port de l'ordinateur sur le port du matériel réseau, l'IP est reporté sur le port du switch.
Donc l'IP apparait 2 fois (port de l'ordinateur + port du switch) => donc avec ta solution cela va provoquer des doublons.

Pour les équipements réseau, l'IP est directement renseignée sur la fiche (pas dans les ports)

++

tsmr · 2008-05-28 17:43:17

Et d'ailleurs la v1.6 de addressing est pour glpi 0.71 pas 0.70.2

benoit_glpi · 2008-05-29 09:58:19

@Rémi :

Ok, vu. C'est vrai que dans de cas, GLPI annonce un doublon.

Par contre je ne trouve pas ca très logique que l'IP soit déportée sur le switch dès lors que l'on relie un ordinateur à un port, car dans les faits ce n'est pas le switch qui porte l'IP mais bien l'interface réseau de l'ordinateur.

Donc quand on liste les ips occupées, les informations affichées ne sont pas correctes

Si sur les équipements réseaux on utilise la fiche générale pour indiquer l'IP de l'équipement, on ne peut plus indiquer de VLAN sur cette l'interface (il n'y a pas de champ prévu)

Dernier chose, lorsque l'on attache sur un switch, une interface réseau d'un ordinateur qui à déjà un VLAN de déclaré. Alors l'interface perd son VLAN. Il faut alors retourner sur le détail de l'interface pour l'ajouter de nouveau.
Edit : peut-être que cela vient du fait que je n'utilise pas la bonne version de GLPI pour ce plugin.

@tsmr : ok. c'est noté.

Last edited by benoit_glpi (2008-05-29 09:59:02)

fledorze · 2008-12-16 16:58:43

Bjr.
Cela rejoint le sujet http://www.glpi-project.org/forum/viewtopic.php?id=8713
Un équipement réseau n'est pas forcément un commutateur. Cela peut être un routeur, un pare-feu matériel, avec de multiples interfaces et autant d'IP.
Je pensais utiliser ce plugin pour faire la liste des IPs d'un réseau :
1- qui sont utilisées soit par matériel répertorié dans GLPI (ordinateur, imprimante, équipement réseau)
ou
2- qui répondent au ping

et ainsi avoir une liste sûre des IPs libres.

Dans son état actuel, c'est impossible. Par exemple , j'ai un routeur VPN dans 2 réseaux différents : dans le réseau public et dans une DMZ interne. Son interface dans le réseau public ne répond volontairement pas au ping. Le plugin ne prenant en compte que l'IP dans la fiche principale et non les IPs de ses interfaces ou ports, l'IP publique est annoncée comme ne répondant pas au ping et donc libre. Le risque, c'est de l'affecter à un autre usage, ce qui serait pour nous une catastrophe.
Voilà un exemple de problématique.

Forum GLPI-Project

Announcement

#1 2008-05-28 14:48:54

Patch pour plugins adressing

#2 2008-05-28 15:11:06

Re: Patch pour plugins adressing

#3 2008-05-28 16:17:38

Re: Patch pour plugins adressing

#4 2008-05-28 17:16:33

Re: Patch pour plugins adressing

#5 2008-05-28 17:23:47

Re: Patch pour plugins adressing

#6 2008-05-28 17:43:17

Re: Patch pour plugins adressing

#7 2008-05-29 09:58:19

Re: Patch pour plugins adressing

#8 2008-12-16 16:58:43

Re: Patch pour plugins adressing

Board footer