Re: [Glpi-dev] Patch : Ajout nouvelle option import OCS des moniteurs

2012-01-26 Thread Thierry Barrau 
Bonjour Moyo,

Merci de ta réponse.

Je ne sais pas si on aura beaucoup de retour, car c'est un problème qui ne 
concerne que les personnes qui travaillent en multi écran (voir en multi écran 
ordinateur portable + écran externe), et ils ne sont pas légion ... même si il 
y en a de plus en plus .

Le fonctionnement actuel permet soit :

-  Import unique :

o   Importe tous les écrans dont tous les « unknow XXX » qui sont les écrans 
intégrés des ordinateurs portables.

-  Import unique sur numéro de série :

o   N'importe que si « tous » les écrans ont un numéro de série, si l'un d'eux 
n'en a pas, alors celui qui en a bien un sera ignoré.

Au-delà du problème lui-même, c'est la description de la fonctionnalité qui est 
trompeuse dans la doc :

-  « Import unique sur numéro de série : ne s'applique qu'aux 
moniteurs. N'importe l'écran dans GLPI que si le moniteur a un numéro de série. 
»
De base, on se dit que le comportement désiré n'est l'import que des écrans 
disposants d'un numéro de série et pas des autres (« unknow », etc...). D'où le 
fait je pense aussi d'éclaircir la doc sur le sujet.

Pour rappel mon post à l'époque sur le forum : 
http://www.glpi-project.org/forum/viewtopic.php?id=25074
Et le post de cenid35 qui avait le même problème que moi : 
http://www.glpi-project.org/forum/viewtopic.php?id=25811

Je peux lui proposer de tester le patch pour voir si ça corrige son problème ...

Bonne soirée et merci du temps consacré à la lecture de ce mail ;)

--
Thierry BARRAU

De : glpi-dev-boun...@gna.org [mailto:glpi-dev-boun...@gna.org] De la part de 
MoYo
Envoyé : jeudi 26 janvier 2012 17:03
À : glpi-dev@gna.org
Objet : Re: [Glpi-dev] Patch : Ajout nouvelle option import OCS des moniteurs

Bonjour,

désolé pour le délai de la réponse.
Nous allons étudier la proposition.
Ce qui nous intéresserai c'est surtout d'avoir des avis sur la fonctionnalité 
en elle même.
Intéresse t'elle d'autres personnes ou pas ?
Cela impactera sur l'intégration rapide (0.83) ou pas (version corrective 0.83).

Cordialement,

Julien Dombre


Le 09/01/2012 18:18, Thierry Barrau a écrit :
Bonjour,

Comme discuté à plusieurs reprises, l'option actuelle « Import sur numéro de 
série » n'importe, dans le cas de multi écran sur un ordinateur, que si 
l'ensemble des écrans de cette machine dispose d'un numéro de série.

Remi, ayant développé cette partie, m'a conseillé de partir sur une nouvelle 
option si je souhaitais modifier le comportement pour que seuls les écrans 
disposant d'un numéro de série ne soient traités.

Voici donc les patchs (pour la version 0.83) pour l'ajout d'une nouvelle option 
que nommée « Import unique numéro de série uniquement » (je pense qu'il 
faudrait peut être revoir les dénominations des deux intitulés pour que cela 
soit plus explicite).

Egalement, voici une proposition de modification du paragraphe décrivant les 
imports de la documentation :

Import unique sur numéro de série : ne s'applique qu'aux moniteurs. N'importe 
l'écran dans GLPI que si tous les moniteurs ont un numéro de série.

Import unique numéro de série uniquement : ne s'applique qu'aux moniteurs. 
N'importe dans GLPI que les moniteurs ayant un numéro de série.

Espérant que ça aide.

Cordialement,
--
Thierry BARRAU





___

Glpi-dev mailing list

Glpi-dev@gna.org

https://mail.gna.org/listinfo/glpi-dev

___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

Re: [Glpi-dev] Fwd: [glpi] walid | rev 17223 - [0.83] see #3289 simplier way to sychronize vms (thanks Remi)

2012-01-26 Thread MoYo 

Le 26/01/2012 17:01, Eltharin a écrit :


Bonjour,

Concernant la 0.83, voici un retour :

Sous configuration / authentification / autres, le formulaire pointe 
vers auth.form.php qui affiche une erreur 404.


J’ai fait un copié-collé de la page auth.other.php et ça marche pour 
info je n’ai pas trouvé ou le changé dans le formulaire.


Cordialement,



Bonjour,

merci du retour, je crois que ce problème est déjà corrigé.
Pouvez vous tester avec le dernier tarball ?

Cordialement,

Julien Dombre

___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

Re: [Glpi-dev] Patch : Ajout nouvelle option import OCS des moniteurs

2012-01-26 Thread MoYo 

Bonjour,

désolé pour le délai de la réponse.
Nous allons étudier la proposition.
Ce qui nous intéresserai c'est surtout d'avoir des avis sur la 
fonctionnalité en elle même.

Intéresse t'elle d'autres personnes ou pas ?
Cela impactera sur l'intégration rapide (0.83) ou pas (version 
corrective 0.83).


Cordialement,

Julien Dombre


Le 09/01/2012 18:18, Thierry Barrau a écrit :


Bonjour,

Comme discuté à plusieurs reprises, l'option actuelle « Import sur 
numéro de série » n'importe, dans le cas de multi écran sur un 
ordinateur, que si l'ensemble des écrans de cette machine dispose d'un 
numéro de série.


Remi, ayant développé cette partie, m'a conseillé de partir sur une 
nouvelle option si je souhaitais modifier le comportement pour que 
seuls les écrans disposant d'un numéro de série ne soient traités.


Voici donc les patchs (pour la version 0.83) pour l'ajout d'une 
nouvelle option que nommée « Import unique numéro de série 
uniquement » (je pense qu'il faudrait peut être revoir les 
dénominations des deux intitulés pour que cela soit plus explicite).


Egalement, voici une proposition de modification du paragraphe 
décrivant les imports de la documentation :


·*Import unique sur numéro de série *: ne s'applique qu'aux moniteurs. 
N'importe l'écran dans GLPI que si tous les moniteurs ont un numéro de 
série.


·*Import unique numéro de série uniquement *: ne s'applique qu'aux 
moniteurs. N'importe dans GLPI que les moniteurs ayant un numéro de série.


Espérant que ça aide.

Cordialement,

--

Thierry BARRAU


___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev


___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

Re: [Glpi-dev] Fwd: [glpi] walid | rev 17223 - [0.83] see #3289 simplier way to sychronize vms (thanks Remi)

2012-01-26 Thread Eltharin 
Bonjour,

 

Concernant la 0.83, voici un retour : 

 

Sous configuration / authentification / autres, le formulaire pointe vers 
auth.form.php qui affiche une erreur 404.

 

J’ai fait un copié-collé de la page auth.other.php et ça marche pour info je 
n’ai pas trouvé ou le changé dans le formulaire.

 

Cordialement,

 

Roman

 

De : glpi-dev-boun...@gna.org [mailto:glpi-dev-boun...@gna.org] De la part de 
MoYo
Envoyé : jeudi 26 janvier 2012 16:43
À : glpi-dev@gna.org
Objet : Re: [Glpi-dev] Fwd: [glpi] walid | rev 17223 - [0.83] see #3289 
simplier way to sychronize vms (thanks Remi)

 

Le 26/01/2012 16:06, Walid nouh a écrit : 

Bonjour,

Remi a très justement faire remarquer que j'ai oublié d'ajouter les indexes sur 
la table glpi_computervirtualmachines en 0.80.
Est-ce que je peux ajouter les indexes pour la 0.83 ?


Salut,

Ca commence à faire vraiment tard pour modifier le schéma de DB... 
Mais bon on est toujours en RC...
Donc vas-y.

Perso vu le peu de retour actuellement sur la 0.83, je serais même partant pour 
sortir une 0.83 en l'état.

++

Julien





Bonne journée à tous,
Walid.

 Original Message  


Subject: 

[glpi] walid | rev 17223 - [0.83] see #3289 simplier way to sychronize vms 
(thanks Remi)


Date: 

Thu, 26 Jan 2012 15:57:22 +0100


From: 

nore...@indepnet.net


To: 

undisclosed-recipients:;

 

walid has commited revision 17223 for the GLPI-PROJECT project.

Comments : [0.83] see #3289 simplier way to sychronize vms (thanks Remi) 

Please take a moment to review their checkin : 
https://forge.indepnet.net/repositories/revision/glpi/17223 

  _  

You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
https://forge.indepnet.net/my/account

 
 
___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

 

___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

Re: [Glpi-dev] Fwd: [glpi] walid | rev 17223 - [0.83] see #3289 simplier way to sychronize vms (thanks Remi)

2012-01-26 Thread Walid nouh 

On 26/01/2012 16:43, MoYo wrote:

Le 26/01/2012 16:06, Walid nouh a écrit :

Bonjour,

Remi a très justement faire remarquer que j'ai oublié d'ajouter les 
indexes sur la table glpi_computervirtualmachines en 0.80.

Est-ce que je peux ajouter les indexes pour la 0.83 ?



Salut,

Ca commence à faire vraiment tard pour modifier le schéma de DB...
Mais bon on est toujours en RC...
Donc vas-y.


Ca marche,
je commite ça.

Merci
Perso vu le peu de retour actuellement sur la 0.83, je serais même 
partant pour sortir une 0.83 en l'état.


++

Julien



Bonne journée à tous,
Walid.

 Original Message 
Subject: 	[glpi] walid | rev 17223 - [0.83] see #3289 simplier way to 
sychronize vms (thanks Remi)

Date:   Thu, 26 Jan 2012 15:57:22 +0100
From:   nore...@indepnet.net
To: undisclosed-recipients:;



walid has commited revision 17223 for the GLPI-PROJECT project.

Comments : [0.83] see #3289 simplier way to sychronize vms (thanks Remi)

Please take a moment to review their checkin : 
https://forge.indepnet.net/repositories/revision/glpi/17223



You have received this notification because you have either 
subscribed to it, or are involved in it.
To change your notification preferences, please click here: 
https://forge.indepnet.net/my/account



___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev




___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev


___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

Re: [Glpi-dev] Fwd: [glpi] walid | rev 17223 - [0.83] see #3289 simplier way to sychronize vms (thanks Remi)

2012-01-26 Thread MoYo 

Le 26/01/2012 16:06, Walid nouh a écrit :

Bonjour,

Remi a très justement faire remarquer que j'ai oublié d'ajouter les 
indexes sur la table glpi_computervirtualmachines en 0.80.

Est-ce que je peux ajouter les indexes pour la 0.83 ?



Salut,

Ca commence à faire vraiment tard pour modifier le schéma de DB...
Mais bon on est toujours en RC...
Donc vas-y.

Perso vu le peu de retour actuellement sur la 0.83, je serais même 
partant pour sortir une 0.83 en l'état.


++

Julien



Bonne journée à tous,
Walid.

 Original Message 
Subject: 	[glpi] walid | rev 17223 - [0.83] see #3289 simplier way to 
sychronize vms (thanks Remi)

Date:   Thu, 26 Jan 2012 15:57:22 +0100
From:   nore...@indepnet.net
To: undisclosed-recipients:;



walid has commited revision 17223 for the GLPI-PROJECT project.

Comments : [0.83] see #3289 simplier way to sychronize vms (thanks Remi)

Please take a moment to review their checkin : 
https://forge.indepnet.net/repositories/revision/glpi/17223



You have received this notification because you have either subscribed 
to it, or are involved in it.
To change your notification preferences, please click here: 
https://forge.indepnet.net/my/account



___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev


___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

[Glpi-dev] Fwd: [glpi] walid | rev 17223 - [0.83] see #3289 simplier way to sychronize vms (thanks Remi)

2012-01-26 Thread Walid nouh 

Bonjour,

Remi a très justement faire remarquer que j'ai oublié d'ajouter les 
indexes sur la table glpi_computervirtualmachines en 0.80.

Est-ce que je peux ajouter les indexes pour la 0.83 ?

Bonne journée à tous,
Walid.

 Original Message 
Subject: 	[glpi] walid | rev 17223 - [0.83] see #3289 simplier way to 
sychronize vms (thanks Remi)

Date:   Thu, 26 Jan 2012 15:57:22 +0100
From:   nore...@indepnet.net
To: undisclosed-recipients:;



walid has commited revision 17223 for the GLPI-PROJECT project.

Comments : [0.83] see #3289 simplier way to sychronize vms (thanks Remi)

Please take a moment to review their checkin : 
https://forge.indepnet.net/repositories/revision/glpi/17223



You have received this notification because you have either subscribed 
to it, or are involved in it.
To change your notification preferences, please click here: 
https://forge.indepnet.net/my/account


Index: branches/0.83-bugfixes/inc/ocsserver.class.php
===
--- branches/0.83-bugfixes/inc/ocsserver.class.php	(revision 17222)
+++ branches/0.83-bugfixes/inc/ocsserver.class.php	(revision 17223)
@@ -4327,7 +4327,7 @@
 
static function updateVirtualMachines($computers_id, $ocsid, $ocsservers_id, $cfg_ocs, $import_vm,
  $dohistory) {
-  global $DBocs, $DB;
+  global $DBocs;
 
   // No VM before OCS 1.3
   if ($cfg_ocs['ocs_version'] < self::OCS1_3_VERSION_LIMIT) {
@@ -4341,62 +4341,40 @@
 FROM `virtualmachines`
 WHERE `HARDWARE_ID` = '$ocsid'";
   $result = $DBocs->query($query);
-  
+
   $virtualmachine = new ComputerVirtualMachine();
   if ($DBocs->numrows($result) > 0) {
  while ($line = $DBocs->fetch_array($result)) {
 $line = Toolbox::clean_cross_side_scripting_deep(Toolbox::addslashes_deep($line));
-$vm['name'] = $line['NAME'];
-$vm['vcpu'] = $line['VCPU'];
-$vm['ram']  = $line['MEMORY'];
-$vm['uuid'] = $line['UUID'];
-$vm['computers_id'] = $computers_id;
+$vm['name'] = $line['NAME'];
+$vm['vcpu'] = $line['VCPU'];
+$vm['ram']  = $line['MEMORY'];
+$vm['uuid'] = $line['UUID'];
+$vm['computers_id'] = $computers_id;
+
 $vm['virtualmachinestates_id']  = Dropdown::importExternal('VirtualMachineState',
$line['STATUS']);
 $vm['virtualmachinetypes_id']   = Dropdown::importExternal('VirtualMachineType',
$line['VMTYPE']);
 $vm['virtualmachinesystems_id'] = Dropdown::importExternal('VirtualMachineType',
$line['SUBSYSTEM']);
-//VM already exists on this computer
-if (in_array(stripslashes($line["ID"]), $import_vm)) {
+
+if (!in_array(stripslashes($line["UUID"]), $import_vm)) {
$virtualmachine->reset();
-   $vm['id'] = $line['ID'];
-   $virtualmachine->update($vm);
-   
-   //vm processed, remove it from import_vm
-   $id = array_search(stripslashes($line["ID"]), $import_vm);
-   unset($import_vm[$id]);
+   if (!$dohistory) {
+  $vm['_no_history'] = true;
+   }
+   $id_vm = $virtualmachine->add($vm);
+   if ($id_vm) {
+  self::addToOcsArray($computers_id, array($id_vm => $line['UUID']), "import_vm");
+   }
 } else {
-   //VM doesn't exists on the host, but may exist on another one
-   $query = "SELECT `id`, `computers_id` FROM `glpi_computervirtualmachines` " .
-   "WHERE `uuid`='".$line['UUID']."'";
-   $result_vms = $DB->query($query);
-   if ($DB->numrows($result_vms) > 0) {
-  $old_computers_id = $DB->result($result_vms, 0, "computers_id");
-  $id   = $DB->result($result_vms, 0, "id");
-  //If vm has changed of host
-  if ($old_computers_id != $computers_id) {
- self::deleteInOcsArray($old_computers_id, $id, "import_vm");
- self::addToOcsArray($computers_id, array($id => $line['ID']), "import_vm");
-  } else {
- //vm ID has changed in OCS, update import_vm
- self::deleteInOcsArray($computers_id, $id, "import_vm");
- self::addToOcsArray($computers_id, array($id => $line['ID']), "import_vm");
-  }
-

Re: [Glpi-dev] Evolution synchro/import

2012-01-26 Thread David DURIEUX 
Le Thu, 19 Jan 2012 08:34:12 +0100
Remi Collet  a écrit:

>Une proposition d'évolution pour la gestion des éléments importés
>
>Exemple, table glpi_computervirtualmachines
>
>Ajout de 2 champs
>   - is_dynamic => importé
>   - is_deleted => supprimé (verrouillé dans ce cas)
>
>Lors de la suppression
>
>   si is_dynamic=1 => passer is_deleted=1 (sinon purger, comme
>   d'hab)
>
>Gestion des verrous
>
>   VM verrouillées : is_dynamic=1 ET is_deleted=1
>
>   Deverrouiller => is_deleted=0
>
>Lors de l'import OCS, au lieu de prendre le contenu de
>glpi_ocslinks.import_vm, on fait un
>   SELECT ... WHERE computers_id=xx AND is_dynamic
>
>
>Avantages :
>   - méthode standard déjà utilisée pour les utilisateurs (droits
>   et groupes)
>   - méthode indépendante de l'outil d'import
> (Fusion doit pouvoir l'utiliser, à confirmer, David ?)
>   - gestion plus légère
> glpi_ocslinks, je trouve ça lourd, avec 2Gio (sur 13Gio),
>   c'est la plus grosse table chez nous, après glpi_logs (9Gio)
>   et avant les install (1.4Gio)
>   - moins d'UPDATE (maj de glpi_ocslinks)
>
>Inconvénients :
>   - plus de SELECT pendant la synchro, mais bon, on utilise un
>   index.
>   - ce que j'ai probablement raté
>
>
>Ensuite, à généraliser aux autres données :
>   - glpi_computers_device*
>   (là de toutes manières faut revoir tout le schéma pour les
>   champs "specificity")
>   - glpi_computerdisks
>   - glpi_computers_softwareversions
>   - glpi_computers_items
>   (monitor, peripheral, printer et donc phone, même si pas
>   utiliser par OCS)
>   - glpi_networkports (qu'il faut réécrire suite aux travaux de
>   Damien)
>
>Il me semblerait intéressant de réaliser ce chantier en pré-requis à
>la sortie d'OCS en plugin.
>
>Voila, pour avis... (oui, c'est un gros chantier, avec une très grosse
>migration)
>
>Remi.
>
>


C'est bon pour moi, excepté que le "is_deleted => supprimé (verrouillé
dans ce cas)" n'est pas aproprié par rapport aux is_deleted des autres
items, de ce fait, je mettrai plutôt quelquechose du genre "is_locked" 

David
++

___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

Re: [Glpi-dev] Add plugin monitoring in GLPI plugin catalog

2012-01-26 Thread MoYo 

Hi,

done.

Regards


Le 26/01/2012 11:58, David DURIEUX a écrit :

Can you add plugin monitoring into GLPI plugin catalog?


https://forge.indepnet.net/projects/monitoring/repository/entry/monitoring.xml

Thanks you very much

David Durieux
++

___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev



___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

[Glpi-dev] Add plugin monitoring in GLPI plugin catalog

2012-01-26 Thread David DURIEUX 
Can you add plugin monitoring into GLPI plugin catalog? 


https://forge.indepnet.net/projects/monitoring/repository/entry/monitoring.xml

Thanks you very much

David Durieux
++

___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev