Re: [pve-devel] spice problems

2017-01-05 Thread Michael Rasmussen
On Thu, 5 Jan 2017 16:21:38 +0100 (CET) Dietmar Maurer wrote: > > If you do not want to debug yourself, please can you file a > bug at bugzilla.proxmox.com? > https://bugzilla.proxmox.com/show_bug.cgi?id=1243 -- Hilsen/Regards Michael Rasmussen Get my public GnuPG

Re: [pve-devel] spice problems

2017-01-05 Thread Dietmar Maurer
> On January 5, 2017 at 8:02 PM Michael Rasmussen wrote: > > > On Thu, 5 Jan 2017 16:21:38 +0100 (CET) > Dietmar Maurer wrote: > > > > > If you do not want to debug yourself, please can you file a > > bug at bugzilla.proxmox.com? > > > I will do

Re: [pve-devel] spice problems

2017-01-05 Thread Michael Rasmussen
On Thu, 5 Jan 2017 16:21:38 +0100 (CET) Dietmar Maurer wrote: > > If you do not want to debug yourself, please can you file a > bug at bugzilla.proxmox.com? > I will do that but I think I have nailed the problem down to either wrong instructions on the wiki or some kind

Re: [pve-devel] [PATCH manager 0/4] add start/stop/migration filtering

2017-01-05 Thread Dietmar Maurer
> if someone has a good argument why this is a bad idea, please share it > (or any other suggestion for this) IMHO it is confusing to display all VMs ... ___ pve-devel mailing list pve-devel@pve.proxmox.com

[pve-devel] VM config: User Interface <-> Online Help mismatch

2017-01-05 Thread Waschbüsch IT-Services GmbH
Hi all, I just stumbled across the following: When configuring memory for a VM, you can choose between the options 'Use fixed size memory' and 'Automatically allocate memory within this range'. The online help explains the ballooning feature quite nicely, but there is a mismatch: Under the

Re: [pve-devel] [PATCH manager 0/4] add start/stop/migration filtering

2017-01-05 Thread Dominik Csapak
On 01/05/2017 04:04 PM, Dietmar Maurer wrote: BulkStop: Why do we list already stopped guests? i wanted to preserve the old behaviour (which included all vms) this is interesting for one case: you open the bulk stop window -> someone starts a vm -> you click stop in the old (and current)

Re: [pve-devel] spice problems

2017-01-05 Thread Dietmar Maurer
> > You get an error after installing custom certs? > Yes, getting the error after following the new instructions for > installing custom certs. I had to renew my custom certs and chose the > new instructions for doing that. If you do not want to debug yourself, please can you file a bug at

[pve-devel] applied: [PATCH manager v2] Reload the grid list of backups after the backup task completion

2017-01-05 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH manager 0/4] add start/stop/migration filtering

2017-01-05 Thread Dietmar Maurer
BulkStop: Why do we list already stopped guests? ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] applied: [PATCH manager 0/4] add start/stop/migration filtering

2017-01-05 Thread Dietmar Maurer
applied, thanks! > this patch series adds a vmid filter to the > startall/stopall/migrateall calls of nodes > > and a gui for selecting this > > so you can selectively start, stop and migrate guests in bulk > > i will also send a documentation patch later and add a help button to the > window

[pve-devel] applied: [PATCH storage] fix #1099 (again): map cciss/c0d0 to cciss!c0d0

2017-01-05 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] applied: [PATCH manager v2] ha: re-add node to service status

2017-01-05 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH manager v2] Reload the grid list of backups after the backup task completion

2017-01-05 Thread Emmanuel Kasper
This allows visual feedback for first time users doing a backup. --- change the way we reload by hiding the backup window instead of passing aroung the reload() function www/manager6/grid/BackupView.js | 7 ++- www/manager6/window/Backup.js | 12 ++-- 2 files changed, 16

Re: [pve-devel] [PATCH manager v2] ha: re-add node to service status

2017-01-05 Thread Dominik Csapak
Reviewed-by: Dominik Csapak On 01/05/2017 12:23 PM, Thomas Lamprecht wrote: On the old HA status we saw where a service was located currently, this information was lost when we merged the resource and the status tab. Add this information again. Signed-off-by: Thomas

[pve-devel] applied: [PATCH ha-manager v2] is_node_shutdown: check for correct systemd targets

2017-01-05 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH ha-manager v2] is_node_shutdown: check for correct systemd targets

2017-01-05 Thread Thomas Lamprecht
any comments? On 12/20/2016 09:33 AM, Thomas Lamprecht wrote: shutdown.target is active every time when the node shuts down, be it reboot, poweroff, halt or kexec. As we want to return true only when the node powers down without a restart afterwards this was wrong. Match only poweroff.target

[pve-devel] [PATCH manager 4/4] change Start/Stop/Migrate all to Bulk actions

2017-01-05 Thread Dominik Csapak
this uses the new vmselector and the new vmid filter in the backend to allow starting/stopping/migrating selected vms instead of all by default all vms are selected to have the same default behaviour Signed-off-by: Dominik Csapak --- www/manager6/Makefile | 3 ++-

[pve-devel] [PATCH manager 2/4] add BulkAction window class

2017-01-05 Thread Dominik Csapak
this is mostly copied from MigrateAll.js, but a more generic way, to allow startall and stopall to also use it Signed-off-by: Dominik Csapak --- www/manager6/window/BulkAction.js | 141 ++ 1 file changed, 141 insertions(+) create mode

[pve-devel] [PATCH manager 0/4] add start/stop/migration filtering

2017-01-05 Thread Dominik Csapak
this patch series adds a vmid filter to the startall/stopall/migrateall calls of nodes and a gui for selecting this so you can selectively start, stop and migrate guests in bulk i will also send a documentation patch later and add a help button to the window to explain it Dominik Csapak (4):

[pve-devel] [PATCH manager 3/4] add vmselector form field

2017-01-05 Thread Dominik Csapak
this is a form field which is a grid for selecting vms if nodename is given, it will filter the vms only to the given node you can filter the grid with the column header, and only the selected and visible items are in the value of the field Signed-off-by: Dominik Csapak

[pve-devel] [PATCH manager v2] ha: re-add node to service status

2017-01-05 Thread Thomas Lamprecht
On the old HA status we saw where a service was located currently, this information was lost when we merged the resource and the status tab. Add this information again. Signed-off-by: Thomas Lamprecht --- changes since v1: * add 'node' also to the data model

[pve-devel] [PATCH storage] fix #1099 (again): map cciss/c0d0 to cciss!c0d0

2017-01-05 Thread Dominik Csapak
there was still a point where we got the wrong string on createosd we get the devpath (/dev/cciss/c0d0) but need the info from get_disks, which looks in /sys/block where it needs to be cciss!c0d0 Signed-off-by: Dominik Csapak --- i hope this is the final fix for this

Re: [pve-devel] [PATCH qemu-server 3/4] drive-mirror: bump timeout to 5s, add 30s inactivity timeout

2017-01-05 Thread Alexandre DERUMIER
The drive-mirror qmp command have a timeout of 3s by default (QMPCLient.pm), shouldn't we bump it to 6s ? (more than 5s connect-timeout ?) - Mail original - De: "Wolfgang Bumiller" À: "pve-devel" Envoyé: Jeudi 5 Janvier 2017 10:09:28

Re: [pve-devel] applied: live storage migration v9

2017-01-05 Thread Alexandre DERUMIER
Thanks Wolfgang. I had already prepared a v10 with the cleanup ;) I'm currently on holiday, so don't have too much time this week. yes, for nbd tls, this need qemu 2.8 + blockdev-add / blockdev-mirror. Too much change for now. (and blockdev is still experimental and not completed) - Mail

[pve-devel] applied: [PATCH qemu-server 0/4] storage live migration followup

2017-01-05 Thread Wolfgang Bumiller
Sorry, forgot to add the applied tag. ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH qemu-server 4/4] cleanup: error messages

2017-01-05 Thread Wolfgang Bumiller
--- PVE/API2/Qemu.pm | 4 ++-- PVE/QemuMigrate.pm | 2 +- PVE/QemuServer.pm | 16 3 files changed, 11 insertions(+), 11 deletions(-) diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm index e48bf6d..288a9cd 100644 --- a/PVE/API2/Qemu.pm +++ b/PVE/API2/Qemu.pm @@ -1643,7

[pve-devel] [PATCH qemu-server 2/4] drive-mirror: warn use POSIX::_exit on exec failure

2017-01-05 Thread Wolfgang Bumiller
--- PVE/QemuServer.pm | 19 +-- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 0b866cd..31e30fa 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -5926,17 +5926,16 @@ sub qemu_drive_mirror { die

[pve-devel] applied: live storage migration v9

2017-01-05 Thread Wolfgang Bumiller
Applied the series with some followup patches: * Added patches for the timeout and POSIX::_exit() changes I mentioned. * Also added some whitespace & style cleanup patches. Now we need to figure out whether to first add the ssh-tunnel based ecnryption or go with qemu's tls. Saw the thread on

[pve-devel] [PATCH qemu-server 3/4] drive-mirror: bump timeout to 5s, add 30s inactivity timeout

2017-01-05 Thread Wolfgang Bumiller
--- PVE/QemuServer.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 31e30fa..c2fa20b 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -5919,7 +5919,7 @@ sub qemu_drive_mirror { $format = "nbd"; my

[pve-devel] [PATCH qemu-server 1/4] cleanup: whitespaces & style

2017-01-05 Thread Wolfgang Bumiller
--- PVE/API2/Qemu.pm | 7 +++ PVE/QemuMigrate.pm | 16 PVE/QemuServer.pm | 12 ++-- 3 files changed, 13 insertions(+), 22 deletions(-) diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm index 0bae424..e48bf6d 100644 --- a/PVE/API2/Qemu.pm +++ b/PVE/API2/Qemu.pm @@

Re: [pve-devel] spice problems

2017-01-05 Thread datanom.net
On 2017-01-05 09:21, Dietmar Maurer wrote: The default configuration work for you? I do not know exactly since I have been using custom certs since proxmox 2.x and have kept these certs while upgrading (following the old instructions) You get an error after installing custom certs? Yes,