Re: [pve-devel] RRD graphics in webfrontend broken

2012-10-31 Thread Dietmar Maurer
Subject: Re: [pve-devel] RRD graphics in webfrontend broken Sorry last patch still contains a bug. The code can rename a .png.tmp file of another process which has already started to write to that file. So we need file locking too. I finally committed a slightly different solution. I

Re: [pve-devel] RRD graphics in webfrontend broken

2012-10-31 Thread Stefan Priebe - Profihost AG
Am 31.10.2012 07:28, schrieb Dietmar Maurer: Subject: Re: [pve-devel] RRD graphics in webfrontend broken Sorry last patch still contains a bug. The code can rename a .png.tmp file of another process which has already started to write to that file. So we need file locking too. I finally

Re: [pve-devel] snapshot timeout problems

2012-10-31 Thread Stefan Priebe - Profihost AG
i think there are several reasons - high disk I/O - slow network speed or whatever. You don't know how old the hardware is where people run proxmox ;-) I think even a value of 300s could be fine. Stefan Am 31.10.2012 06:18, schrieb Alexandre DERUMIER: mmm,maybe 30s timeout is not big enough

Re: [pve-devel] [PATCH 2/3] PVE::API2::OpenVZ: use build_ssh_opt instead of static blowfish encryption

2012-10-31 Thread Stefan Priebe - Profihost AG
Am 31.10.2012 05:59, schrieb Dietmar Maurer: You mean for VNC Stuff - OpenVZ and Qemu? Yes we can as it is already TLS but otherwise the overhead is low for VNC stuff. Yes, I guess it is not worth to optimize that now. The bad thing is that we can't just fire one command. We've to create

Re: [pve-devel] PATH V1: central cipher config

2012-10-31 Thread Stefan Priebe - Profihost AG
Am 31.10.2012 09:47, schrieb Dietmar Maurer: Many thanks for that patch. But I am still unsure if we should go that way. I just found another option - /root/.ssh/config # man ssh_config What if we simply create that file if it does not exist? - create /.ssh/config with reasonable value for

Re: [pve-devel] PATH V1: central cipher config

2012-10-31 Thread Dietmar Maurer
Good idea - i want to extent your idea. What about providing /etc/pve/ssh_config and then pass -F /etc/pve/ssh_config to every ssh / scp command? So we have a custom default ssh config only for pve. /root/.ssh/config is also used for 'root' account, which is quite convenient. Note: root

Re: [pve-devel] RRD graphics in webfrontend broken

2012-10-31 Thread Alexandre DERUMIER
Works fine here too ! - Mail original - De: Stefan Priebe - Profihost AG s.pri...@profihost.ag À: Dietmar Maurer diet...@proxmox.com Cc: Alexandre DERUMIER aderum...@odiso.com, pve-devel@pve.proxmox.com Envoyé: Mercredi 31 Octobre 2012 08:47:30 Objet: Re: [pve-devel] RRD graphics in

[pve-devel] [PATCH] rbd: purge snapshots before delete volume

2012-10-31 Thread Alexandre Derumier
we can't delete a rbd volume if snapshots exist Signed-off-by: Alexandre Derumier aderum...@odiso.com --- PVE/Storage/RBDPlugin.pm |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/PVE/Storage/RBDPlugin.pm b/PVE/Storage/RBDPlugin.pm index 33ae853..cb7a823 100644 ---

[pve-devel] [PATCH] sheepdog : delete snapshots when deleting volume

2012-10-31 Thread Alexandre Derumier
sheepdog don't delete snapshots automatically when deleting volume Signed-off-by: Alexandre Derumier aderum...@odiso.com --- PVE/Storage/SheepdogPlugin.pm | 23 +++ 1 file changed, 23 insertions(+) diff --git a/PVE/Storage/SheepdogPlugin.pm b/PVE/Storage/SheepdogPlugin.pm

Re: [pve-devel] PATH V1: central cipher config

2012-10-31 Thread Dietmar Maurer
AFAIK pve-cluster (package PVE::Cluster). -Original Message- From: Stefan Priebe - Profihost AG [mailto:s.pri...@profihost.ag] Sent: Mittwoch, 31. Oktober 2012 11:25 To: Dietmar Maurer Cc: pve-devel@pve.proxmox.com Subject: Re: [pve-devel] PATH V1: central cipher config Am

Re: [pve-devel] PATH V1: central cipher config

2012-10-31 Thread Stefan Priebe - Profihost AG
Am 31.10.2012 14:26, schrieb Dietmar Maurer: applied, thanks. The following is needed to automatically create the config on package update: https://git.proxmox.com/?p=pve-cluster.git;a=commitdiff;h=c1eb6bdfa127ff548e7e83141336019c7e06f38f Thx - please fix the blowfish cipher - it's