Re: [pve-devel] [PATCH] Nodeselector: change default sort to name instead of mem

2013-04-23 Thread Dietmar Maurer
> Do you see a problem in using the current master with squeeze? (sure i need to > revert some binary path changes). I'll stay on Squeeze until summer. I have no idea (I do not test that). ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve

Re: [pve-devel] [PATCH] Nodeselector: change default sort to name instead of mem

2013-04-23 Thread Stefan Priebe - Profihost AG
Hi, sorry also based on PVE 2.3 instead of current master. Another question before rebasing. Do you see a problem in using the current master with squeeze? (sure i need to revert some binary path changes). I'll stay on Squeeze until summer. Stefan Am 24.04.2013 06:02, schrieb Dietmar Maurer: >

Re: [pve-devel] [PATCH] Nodeselector: change default sort to name instead of mem

2013-04-23 Thread Stefan Priebe - Profihost AG
Yes but to have the sorting correct the value shouldn't be changed in the render function it must be changed via a converter to a sortable value and then later the renderer can add % and stuff like that to the value. But i think you already know this ;-) Am 24.04.2013 07:52, schrieb Dietmar Maure

Re: [pve-devel] [PATCH] Nodeselector: change default sort to name instead of mem

2013-04-23 Thread Dietmar Maurer
We just need another render function, which display the absolute value and the percentage? From: Alexandre DERUMIER [mailto:aderum...@odiso.com] Sent: Dienstag, 23. April 2013 18:05 To: Stefan Priebe - Profihost AG Cc: pve-devel@pve.proxmox.com; Dietmar Maurer Subject: Re: [pve-devel] [PATCH] Nod

Re: [pve-devel] [PATCH] Nodeselector: change default sort to name instead of mem

2013-04-23 Thread Stefan Priebe - Profihost AG
Yes we display another value as we have in the data field. We should use converter instead of renderer or both. Stefan Am 24.04.2013 um 06:04 schrieb Dietmar Maurer : >> do you have any idea why memory sorting does not work? I tried to solve it >> but i >> did not find the Problem. > > I gues

Re: [pve-devel] [PATCH] Nodeselector: change default sort to name instead of mem

2013-04-23 Thread Stefan Priebe - Profihost AG
That's it I'm trying to put a converter to that field but as soon as I add one it stays empty :-( Am 23.04.2013 um 18:40 schrieb Alexandre DERUMIER : > Oh,I think I understand, > > It's ordered by free amount of ram in GB, but it's displaying % > > my first 3 servers have 512GB servers, the ot

Re: [pve-devel] [PATCH] Nodeselector: change default sort to name instead of mem

2013-04-23 Thread Dietmar Maurer
> do you have any idea why memory sorting does not work? I tried to solve it > but i > did not find the Problem. I guess sorting work, but we display missleading % with the render function? ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pv

Re: [pve-devel] [PATCH] Nodeselector: change default sort to name instead of mem

2013-04-23 Thread Dietmar Maurer
I also get errors when I try to apply that patch. error: patch failed: www/manager/form/NodeSelector.js:79 error: www/manager/form/NodeSelector.js: patch does not apply > -Original Message- > From: pve-devel-boun...@pve.proxmox.com [mailto:pve-devel- > boun...@pve.proxmox.com] On Behalf O

Re: [pve-devel] [PATCH] Qemu CreateWizard preselect current node

2013-04-23 Thread Dietmar Maurer
applied, thanks. > -Original Message- > From: pve-devel-boun...@pve.proxmox.com [mailto:pve-devel- > boun...@pve.proxmox.com] On Behalf Of Stefan Priebe > Sent: Montag, 22. April 2013 11:54 > To: pve-devel@pve.proxmox.com > Subject: [pve-devel] [PATCH] Qemu CreateWizard preselect current n

Re: [pve-devel] [PATCH] Nodeselector: change default sort to name instead of mem

2013-04-23 Thread Alexandre DERUMIER
Oh,I think I understand, It's ordered by free amount of ram in GB, but it's displaying % my first 3 servers have 512GB servers, the others have 64GB or 32GB. - Mail original - De: "Alexandre DERUMIER" À: "Stefan Priebe - Profihost AG" Cc: pve-devel@pve.proxmox.com Envoyé: Mardi 23 A

Re: [pve-devel] [PATCH] Nodeselector: change default sort to name instead of mem

2013-04-23 Thread Alexandre DERUMIER
I can confirm that it's not sorting correctly for me toohere a default sorting migrateDe: "Stefan Priebe - Profihost AG" À: "Dietmar Maurer" Cc: pve-devel@pve.proxmox.com Envoyé: Mardi 23 Avril 2013 16:10:29 Objet: Re: [pve-devel] [PATCH] Nodeselector: change default sort to name instead

Re: [pve-devel] [PATCH] Nodeselector: change default sort to name instead of mem

2013-04-23 Thread Stefan Priebe - Profihost AG
Hi Dietmar, do you have any idea why memory sorting does not work? I tried to solve it but i did not find the Problem. Stefan Am 22.04.2013 11:21, schrieb Stefan Priebe - Profihost AG: > Am 22.04.2013 11:20, schrieb Dietmar Maurer: So this does not really make sense anymore, because we want