Re: [pve-devel] [PATCH] spiceproxy : allow only spice port range

2013-07-23 Thread Dietmar Maurer
About password, I wonder if we could not retrieve the current ticket from spice server, and copy it to the target vm. Libvirt seem to simply store the current password in the vm config xml :/ graphics type='spice' port='5900' autoport='no' passwd='secret'

Re: [pve-devel] [PATCH] spiceproxy : allow only spice port range

2013-07-23 Thread Alexandre DERUMIER
I just committed a patch for pve-qemu-kvm to return the last ticktet  with qmp query-spice.  What do you think? Can you try to use that for your patches?  Great ! Thanks ! I'll try that today. (BTW, the ca=... fix has been commited to spice-gtk git) - Mail original - De: Dietmar

Re: [pve-devel] [PATCH] spiceproxy : allow only spice port range

2013-07-23 Thread Dietmar Maurer
I just committed a patch for pve-qemu-kvm to return the last ticktet with qmp query-spice. What do you think? Can you try to use that for your patches? Great ! Thanks ! I'll try that today. Besides, I consider this a workaround - hopefully the spice people provide a better solution in

Re: [pve-devel] [PATCH] spiceproxy : allow only spice port range

2013-07-23 Thread Alexandre DERUMIER
Besides, I consider this a workaround - hopefully the spice people  provide a better solution in future (which work with multiple sessions).  Yes, sure. I have got some reply on mailing, I'll hope things will move. - Mail original - De: Dietmar Maurer diet...@proxmox.com À: Alexandre

[pve-devel] qemu-server : spice migration v3

2013-07-23 Thread Alexandre Derumier
changelog: retreive spice ticket with qmp query-spice. ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH] spice migration

2013-07-23 Thread Alexandre Derumier
Signed-off-by: Alexandre Derumier aderum...@odiso.com --- PVE/API2/Qemu.pm | 21 + PVE/QMPClient.pm |3 ++- PVE/QemuMigrate.pm | 41 - PVE/QemuServer.pm | 32 +--- 4 files changed, 80

[pve-devel] qemu-server : spice migration v4

2013-07-23 Thread Alexandre Derumier
changelog : wait for migrate spice session, after the resume of the target vm, to avoid downtime ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH] spice migration

2013-07-23 Thread Alexandre Derumier
Signed-off-by: Alexandre Derumier aderum...@odiso.com --- PVE/API2/Qemu.pm | 21 + PVE/QMPClient.pm |3 ++- PVE/QemuMigrate.pm | 41 - PVE/QemuServer.pm | 32 +--- 4 files changed, 80

[pve-devel] SPICE for Proxmox VE (pvetest)

2013-07-23 Thread Martin Maurer
Hi all! We just uploaded new packages to enable the use of SPICE on Proxmox VE. I already love it and I use it every day! In order to test it, you need to use the pvetest repository (http://pve.proxmox.com/wiki/Package_repositories) For more details, see: http://pve.proxmox.com/wiki/SPICE A

Re: [pve-devel] [PATCH] spice migration

2013-07-23 Thread Dietmar Maurer
+if($conf-{vga} eq 'qxl'){ + my $res = PVE::QemuServer::vm_mon_cmd($vmid, 'query-spice'); + push @$cmd, '--spiceticket', $res-{ticket} if $res-{ticket}; +} + Using passwords/ticktes on the command line is a bad idea, because any system user can see it with a simple ps auxww,

Re: [pve-devel] [PATCH] spice migration

2013-07-23 Thread Alexandre DERUMIER
May I suggest to use a separate patch for that cleanup?  Ok, no problem, I'll split the big patch into smaller patches, should be more readable in git history. - Mail original - De: Dietmar Maurer diet...@proxmox.com À: Alexandre Derumier aderum...@odiso.com,

[pve-devel] [PATCH 1/3] add read_x509_subject_spice

2013-07-23 Thread Alexandre Derumier
Signed-off-by: Alexandre Derumier aderum...@odiso.com --- PVE/API2/Qemu.pm | 12 +--- PVE/QemuServer.pm | 17 + 2 files changed, 18 insertions(+), 11 deletions(-) diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm index 2c86487..29e7647 100644 --- a/PVE/API2/Qemu.pm

[pve-devel] qemu-server : spice migration v5

2013-07-23 Thread Alexandre Derumier
changelog : split in multiple commits ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH 3/3] add spice migration

2013-07-23 Thread Alexandre Derumier
Signed-off-by: Alexandre Derumier aderum...@odiso.com --- PVE/QemuMigrate.pm | 39 +++ PVE/QemuServer.pm | 15 --- 2 files changed, 51 insertions(+), 3 deletions(-) diff --git a/PVE/QemuMigrate.pm b/PVE/QemuMigrate.pm index dd48f78..ce5f17e

[pve-devel] [PATCH 2/3] allow to pass spiceticket to vm_start

2013-07-23 Thread Alexandre Derumier
we need it add spiceticket to target vm for migration Signed-off-by: Alexandre Derumier aderum...@odiso.com --- PVE/API2/Qemu.pm |9 - PVE/QemuServer.pm |2 +- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm index

Re: [pve-devel] [PVE-User] SPICE for Proxmox VE (pvetest)

2013-07-23 Thread Alexandre DERUMIER
I think that you should add to wiki something like this: When you click on spice console button, a connection config file is download. You need to configure your browser to always open this configuration file and associate it when remote-viewer The association is automatic for chrome,internet

Re: [pve-devel] [PVE-User] SPICE for Proxmox VE (pvetest)

2013-07-23 Thread Martin Maurer
I think that you should add to wiki something like this: When you click on spice console button, a connection config file is download. You need to configure your browser to always open this configuration file and associate it when remote-viewer The association is automatic for

Re: [pve-devel] [PATCH] spice migration

2013-07-23 Thread Dietmar Maurer
+if($conf-{vga} eq 'qxl'){ + my $res = PVE::QemuServer::vm_mon_cmd($vmid, 'query-spice'); + push @$cmd, '--spiceticket', $res-{ticket} if $res-{ticket}; +} + Using passwords/ticktes on the command line is a bad idea, because any system user can see it with a simple ps