[pve-devel] [PATCH qemu-server] Fix #2343 Add USB3 support for Spice USB redirection

2019-08-28 Thread Aaron Lauterer
If USB3 is enabled for a USB port of host type spice add a USB3 controller and use it. To not break live migration this is only enabled for qemu 4.1 and higher. Signed-off-by: Aaron Lauterer --- PVE/QemuServer.pm | 4 ++-- PVE/QemuServer/USB.pm | 22 -- 2 files changed

Re: [pve-devel] [PATCH qemu-server] Fix #2343 Add USB3 support for Spice USB redirection

2019-08-28 Thread Aaron Lauterer
In a discussion with Stefan we came to the conclusion / realization that all the checks for $kvmver and $machine may not be necessary. This patch is most likely not correct the way it is now. More feedback is welcome :) On 8/28/19 4:26 PM, Aaron Lauterer wrote: If USB3 is enabled for a USB

[pve-devel] [PATCH manager] Add USB3 support for Spice USB redirection

2019-08-30 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/qemu/USBEdit.js | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/www/manager6/qemu/USBEdit.js b/www/manager6/qemu/USBEdit.js index 8fc95c99..20c30fe6 100644 --- a/www/manager6/qemu/USBEdit.js +++ b/www/manager6/qemu

[pve-devel] [PATCH qemu-server 1/2] Add USB3 support to Spice USB redirection

2019-08-30 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- PVE/QemuServer/USB.pm | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/PVE/QemuServer/USB.pm b/PVE/QemuServer/USB.pm index a2097b9..05c78cf 100644 --- a/PVE/QemuServer/USB.pm +++ b/PVE/QemuServer/USB.pm @@ -87,9 +87,12 @@ sub

[pve-devel] [PATCH qemu-server 2/2] Fix local resources check for USB3 Spice devices

2019-08-30 Thread Aaron Lauterer
The check relied on the fact, that spice usb devices could not be usb3 in the past. Signed-off-by: Aaron Lauterer --- PVE/QemuServer.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 6e3b19e..0a0fda7 100644 --- a/PVE/QemuServer.pm

[pve-devel] [PATCH v2 qemu-server 0/2 pve-manager 0/1] Fix #2343 Spice USB3 support

2019-08-30 Thread Aaron Lauterer
t; v2: * no qemu version checks * fix local resource check on migration * add GUI support [0]: https://pve.proxmox.com/pipermail/pve-devel/2019-August/038672.html Aaron Lauterer (2): Add USB3 support to Spice USB redirection Fix local resources check for USB3 Spice devices Add USB3 support fo

[pve-devel] [PATCH manager] Add support for up to 10 PCI devices

2019-09-02 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/qemu/HardwareView.js | 4 ++-- www/manager6/qemu/PCIEdit.js | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/www/manager6/qemu/HardwareView.js b/www/manager6/qemu/HardwareView.js index 606c66e5..28384d37 100644 --- a/www

[pve-devel] [PATCH qemu-server 0/1 pve-manager 0/1] Fix #2347 Support 10 PCI(e) devices

2019-09-02 Thread Aaron Lauterer
ports here is probably a bad idea and we need to create a new `pve-q35-4.1.cfg` file? But this would mean that the other changes could only be available once we ship Qemu 4.1? Aaron Lauterer (1): Add support for up to 10 PCI(e) devices qemu-server: PVE/QemuServer.pm | 2 +- PVE

[pve-devel] [PATCH qemu-server 1/1] Add support for up to 10 PCI(e) devices

2019-09-02 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- PVE/QemuServer.pm | 2 +- PVE/QemuServer/PCI.pm | 18 + pve-q35-4.0.cfg | 60 +++ 3 files changed, 79 insertions(+), 1 deletion(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 6e3b19e

Re: [pve-devel] [PATCH qemu-server 1/2] Add USB3 support to Spice USB redirection

2019-09-02 Thread Aaron Lauterer
On 9/2/19 1:57 PM, Thomas Lamprecht wrote: On 8/30/19 9:40 AM, Aaron Lauterer wrote: Signed-off-by: Aaron Lauterer --- PVE/QemuServer/USB.pm | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/PVE/QemuServer/USB.pm b/PVE/QemuServer/USB.pm index a2097b9..05c78cf

[pve-devel] [PATCH manager] Add support for 16 PCI(e) devices

2019-09-05 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/qemu/HardwareView.js | 4 ++-- www/manager6/qemu/PCIEdit.js | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/www/manager6/qemu/HardwareView.js b/www/manager6/qemu/HardwareView.js index 606c66e5..cad29cc6 100644 --- a/www

[pve-devel] [PATCH v2 qemu-server 1/1] Add support for up to 16 PCI(e) devices

2019-09-05 Thread Aaron Lauterer
are created on demand. This helps to keep live migration possible without adding a new pve-q35*.cfg file. For the windows 7 like guests additional addresses are reserved as well. Signed-off-by: Aaron Lauterer --- I decided to move the creation of the device string for the additional root ports to

[pve-devel] [PATCH v2 qemu-server 0/1 pve-manager 0/1] Fix #2347 Support 16 PCI(e) devices

2019-09-05 Thread Aaron Lauterer
com/pipermail/pve-devel/2019-September/038743.html Aaron Lauterer (2): Add support for up to 16 PCI(e) devices qemu-server: PVE/QemuServer.pm | 8 +++-- PVE/QemuServer/PCI.pm | 70 +++ 2 files changed, 76 insertions(+), 2 deletions(-) pve-manager:

Re: [pve-devel] [PATCH v2 qemu-server 1/1] Add support for up to 16 PCI(e) devices

2019-09-06 Thread Aaron Lauterer
On 9/6/19 2:45 PM, Dominik Csapak wrote: looks mostly ok, one (important) comment inline   # win7 is picky about pcie assignments   hostpci0bus0 => { bus => "pcie.0", addr => 16 },   hostpci1bus0 => { bus => "pcie.0", addr => 17 },   hostpci2bus0 => { bus => "pcie.0", addr =>

[pve-devel] [PATCH v3 qemu-server 2/3] Fix local resource check of Spice USB devices

2019-09-06 Thread Aaron Lauterer
set. Signed-off-by: Aaron Lauterer --- The proposed[0] /^spice(,)?$/ regex does not match the `usb3=1` part and fails. I opted for the simpler regex that checks if `spice` is that the beginning. [0]: https://pve.proxmox.com/pipermail/pve-devel/2019-September/038749.html PVE/QemuServer.pm | 2

[pve-devel] [PATCH v3 qemu-server pve-manager 0/3] Fix #2343 Spice USB3 support

2019-09-06 Thread Aaron Lauterer
GUI support Aaron Lauterer (3): (qemu-server) Add USB3 capablities to Spice USB devices (qemu-server) Fix local resource check of Spice USB devices (pve-manager) Enable USB3 for Spice USB passthrough qemu-server: PVE/QemuServer.pm | 4 ++-- PVE/QemuServer/USB.pm | 10 +++--- 2 files

[pve-devel] [PATCH v3 manager 3/3] Enable USB3 for Spice USB passthrough

2019-09-06 Thread Aaron Lauterer
Instead of having two times the check if the USB3 setting needs to be added to the config string it is now checked at one place only. If USB3 is checked for a non USB3 device it will be attached to the USB2 root hub of the xhci controller. Signed-off-by: Aaron Lauterer --- @Thomas: I hope I

[pve-devel] [PATCH v3 qemu-server 1/3] Add USB3 capablities to Spice USB devices

2019-09-06 Thread Aaron Lauterer
machine type was set to Q35 an ehci controller was present and the VM was able to boot. With this patch the behaviour is changing and the bus is set to xhci if USB3 is set for the Spice USB device and the VM is running under Qemu v4.1. Signed-off-by: Aaron Lauterer --- I use the

[pve-devel] [PATCH docs 2/9] Rewrite Improve PVE Docs

2019-09-10 Thread Aaron Lauterer
Polished phrasing Signed-off-by: Aaron Lauterer --- howto-improve-pve-docs.adoc | 42 + 1 file changed, 19 insertions(+), 23 deletions(-) diff --git a/howto-improve-pve-docs.adoc b/howto-improve-pve-docs.adoc index c0d277e..940e050 100644 --- a/howto-improve

[pve-devel] [PATCH docs 6/9] Rewrite Install from USB flash drive

2019-09-10 Thread Aaron Lauterer
Polished the phrasing, aligned different names for a USB flash drive, renamed OS X to macOS, changed primary recommendation for Windows to Rufus, removed mention of os forensics. Rufus now supports `dd` mode and is a much smaller download than etcher. Signed-off-by: Aaron Lauterer --- pve

[pve-devel] [PATCH docs 4/9] Rewrite Installation

2019-09-10 Thread Aaron Lauterer
Polished phrasing, added mentions of the EULA and summary page in the installer, unified style of cli commands. Signed-off-by: Aaron Lauterer --- pve-installation.adoc | 278 -- 1 file changed, 134 insertions(+), 144 deletions(-) diff --git a/pve

[pve-devel] [PATCH docs 1/9] Rewrite Getting Help

2019-09-10 Thread Aaron Lauterer
Polished phrasing Signed-off-by: Aaron Lauterer --- getting-help.adoc | 42 -- 1 file changed, 20 insertions(+), 22 deletions(-) diff --git a/getting-help.adoc b/getting-help.adoc index 850d7a3..49d9085 100644 --- a/getting-help.adoc +++ b/getting

[pve-devel] [PATCH docs 7/9] Rewrite Sysadmin

2019-09-10 Thread Aaron Lauterer
Polished phrasing Signed-off-by: Aaron Lauterer --- sysadmin.adoc | 40 +++- 1 file changed, 15 insertions(+), 25 deletions(-) diff --git a/sysadmin.adoc b/sysadmin.adoc index e045610..9592c4a 100644 --- a/sysadmin.adoc +++ b/sysadmin.adoc @@ -5,31 +5,21

[pve-devel] [PATCH docs 0/9] Documentation overhaul chapt. 1.9 to 3.2

2019-09-10 Thread Aaron Lauterer
e are not yet happy with few things there. Aaron Lauterer (9): Rewrite Getting Help Rewrite Improve PVE Docs Rewrite Translation Rewrite Installation Rewrite System Requirements Rewrite Install from USB flash drive Rewrite Sysadmin Rewrite Package Repositories Rewrite System Sof

[pve-devel] [PATCH docs 3/9] Rewrite Translation

2019-09-10 Thread Aaron Lauterer
Polished phrasing Signed-off-by: Aaron Lauterer --- translation.adoc | 38 -- 1 file changed, 20 insertions(+), 18 deletions(-) diff --git a/translation.adoc b/translation.adoc index ff99296..21dd1eb 100644 --- a/translation.adoc +++ b/translation.adoc

[pve-devel] [PATCH docs 5/9] Rewrite System Requirements

2019-09-10 Thread Aaron Lauterer
Polished phrasing and restructured the requirements list a little bit Signed-off-by: Aaron Lauterer --- pve-system-requirements.adoc | 69 +++- 1 file changed, 36 insertions(+), 33 deletions(-) diff --git a/pve-system-requirements.adoc b/pve-system

[pve-devel] [PATCH docs 9/9] Rewrite System Software Updates

2019-09-10 Thread Aaron Lauterer
Polished phrasing and aligned cli command styling Signed-off-by: Aaron Lauterer --- system-software-updates.adoc | 25 ++--- 1 file changed, 10 insertions(+), 15 deletions(-) diff --git a/system-software-updates.adoc b/system-software-updates.adoc index 04c17f3..bd5ff30

[pve-devel] [PATCH docs 8/9] Rewrite Package Repositories

2019-09-10 Thread Aaron Lauterer
Polished phrasing, aligned cli command styling and line width Signed-off-by: Aaron Lauterer --- pve-package-repos.adoc | 154 + 1 file changed, 79 insertions(+), 75 deletions(-) diff --git a/pve-package-repos.adoc b/pve-package-repos.adoc index 06d1b2f

Re: [pve-devel] [PATCH v3 qemu-server 1/3] Add USB3 capablities to Spice USB devices

2019-09-10 Thread Aaron Lauterer
On 9/10/19 10:46 AM, Thomas Lamprecht wrote: On 06.09.19 15:26, Aaron Lauterer wrote: To not change current behaviour and thus breaking live migration USB3 for a Spice USB device requires Qemu v4.1. The old behavior was that even though technically it was possible to the set `usb3=1` setting

Re: [pve-devel] [PATCH v3 qemu-server 2/3] Fix local resource check of Spice USB devices

2019-09-10 Thread Aaron Lauterer
On 9/10/19 11:03 AM, Thomas Lamprecht wrote: On 06.09.19 15:26, Aaron Lauterer wrote: The check relied on the fact that in a regular use case a usb device of type spice would not have any other settings like `usb3=1`. An exact match worked fine for this. This patch changes the behaviour and

[pve-devel] [PATCH v4 qemu-server 2/4] usb: Add USB3 capabilities to Spice USB devices

2019-09-11 Thread Aaron Lauterer
machine type was set to Q35 an ehci controller was present and the VM was able to boot. With this patch the behaviour is changing and the bus is set to xhci if USB3 is set for the Spice USB device and the VM is running under Qemu v4.1. Signed-off-by: Aaron Lauterer --- I created the new

[pve-devel] [PATCH v4 qemu-server 3/4] usb: Fix local resource check of Spice USB devices

2019-09-11 Thread Aaron Lauterer
set. Signed-off-by: Aaron Lauterer --- PVE/QemuServer.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 78ccf8c..05f0ab5 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -2925,7 +2925,7 @@ sub check_local_resources

[pve-devel] [PATCH v4 qemu-server 1/4] usb: Cleanup redundant if condition

2019-09-11 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- PVE/QemuServer/USB.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/QemuServer/USB.pm b/PVE/QemuServer/USB.pm index a2097b9..2c09490 100644 --- a/PVE/QemuServer/USB.pm +++ b/PVE/QemuServer/USB.pm @@ -86,7 +86,7 @@ sub get_usb_devices

[pve-devel] [PATCH v4 0/4] Fix #2343 Spice USB3 support

2019-09-11 Thread Aaron Lauterer
ith a following comma. Thanks Thomas for pointing out how to make it future proof v2 -> v3: * don't modify current behavior * fix local resource check * fix and cleanup GUI code v1 -> v2: * no qemu version checks * fix local resource check on migration * add GUI support Aaron Laute

[pve-devel] [PATCH v4 manager 4/4] usb: Enable USB3 for Spice USB passthrough

2019-09-11 Thread Aaron Lauterer
Instead of having two times the check if the USB3 setting needs to be added to the config string it is now checked at one place only. If USB3 is checked for a non USB3 device it will be attached to the USB2 root hub of the xhci controller. Signed-off-by: Aaron Lauterer --- www/manager6/qemu

[pve-devel] [PATCH manager 2/4] spice: Add enhancements form component

2019-09-13 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/Makefile | 1 + www/manager6/form/SpiceEnhancementSelector.js | 66 +++ 2 files changed, 67 insertions(+) create mode 100644 www/manager6/form/SpiceEnhancementSelector.js diff --git a/www/manager6/Makefile

[pve-devel] [PATCH manager 3/4] spice: Add enhancements to VM Options panel

2019-09-13 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/Utils.js| 18 ++ www/manager6/qemu/Options.js | 13 + 2 files changed, 31 insertions(+) diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js index 6a489e7e..139200c3 100644 --- a/www/manager6/Utils.js +++ b

[pve-devel] [PATCH manager 1/4] Cleanup: align backslashes in manager6 Makefile

2019-09-13 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/Makefile | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/www/manager6/Makefile b/www/manager6/Makefile index 7ee7cd40..82e25c79 100644 --- a/www/manager6/Makefile +++ b/www/manager6/Makefile @@ -14,7 +14,7 @@ JSSRC

[pve-devel] [PATCH manager 4/4] spice: Add enhancements to VM Creation wizard

2019-09-13 Thread Aaron Lauterer
enabled if qxl/spice display selected Signed-off-by: Aaron Lauterer --- www/manager6/qemu/SystemEdit.js | 18 +- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/www/manager6/qemu/SystemEdit.js b/www/manager6/qemu/SystemEdit.js index 846baa73..15ae9a73 100644 --- a

[pve-devel] [PATCH manager 0/4] Fix #2041, #2272 GUI for Spice Enhancements

2019-09-13 Thread Aaron Lauterer
about/appreciate feedback for: * the phrasing in the options panel * right now with this implementation the System tab in the VM creation wizard looks a bit unbalanced. [0]: https://pve.proxmox.com/pipermail/pve-devel/2019-September/038799.html Aaron Lauterer (4): Cleanup: align backslashes in

Re: [pve-devel] [PATCH manager 2/4] spice: Add enhancements form component

2019-09-17 Thread Aaron Lauterer
On 9/16/19 2:44 PM, Stefan Reiter wrote: Tried my hand at a proper code review, I like the patches in general. On 9/13/19 3:16 PM, Aaron Lauterer wrote: Signed-off-by: Aaron Lauterer ---   www/manager6/Makefile |  1 +   www/manager6/form/SpiceEnhancementSelector.js

Re: [pve-devel] [PATCH manager 3/4] spice: Add enhancements to VM Options panel

2019-09-17 Thread Aaron Lauterer
On 9/16/19 2:44 PM, Stefan Reiter wrote: On 9/13/19 3:16 PM, Aaron Lauterer wrote: Signed-off-by: Aaron Lauterer ---   www/manager6/Utils.js    | 18 ++   www/manager6/qemu/Options.js | 13 +   2 files changed, 31 insertions(+) diff --git a/www/manager6

Re: [pve-devel] [PATCH manager 3/4] spice: Add enhancements to VM Options panel

2019-09-17 Thread Aaron Lauterer
On 9/16/19 2:56 PM, Christian Ebner wrote: On September 16, 2019 2:44 PM Stefan Reiter wrote: +render_spice_enhancements: function(value) { + if (!value) { + return Proxmox.Utils.disabledText; + } + var props = PVE.Parser.parsePropertyString(value); +

Re: [pve-devel] [PATCH manager 4/4] spice: Add enhancements to VM Creation wizard

2019-09-17 Thread Aaron Lauterer
ome sort titled "SPICE"? Yeah, the next version of this series will contain a different layout of that panel that should make it clearer. On 9/13/19 3:16 PM, Aaron Lauterer wrote: enabled if qxl/spice display selected Signed-off-by: Aaron Lauterer ---   www/manager

[pve-devel] [PATCH manager v2 3/4] spice: Add enhanecements to VM Options panel

2019-09-17 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/Utils.js| 18 ++ www/manager6/qemu/Options.js | 13 + 2 files changed, 31 insertions(+) diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js index 6a489e7e..a9adb382 100644 --- a/www/manager6/Utils.js +++ b

[pve-devel] [PATCH manager v2 1/4] Cleanup: align backslashes in manager6 Makefile

2019-09-17 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/Makefile | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/www/manager6/Makefile b/www/manager6/Makefile index 7ee7cd40..82e25c79 100644 --- a/www/manager6/Makefile +++ b/www/manager6/Makefile @@ -14,7 +14,7 @@ JSSRC

[pve-devel] [PATCH manager v2 2/4] spice: Add enhancements form component

2019-09-17 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/Makefile | 1 + www/manager6/form/SpiceEnhancementSelector.js | 72 +++ 2 files changed, 73 insertions(+) create mode 100644 www/manager6/form/SpiceEnhancementSelector.js diff --git a/www/manager6/Makefile

[pve-devel] [PATCH manager v2 4/4] spice: Add enhancements to VM Creation wizard

2019-09-17 Thread Aaron Lauterer
For a cleaner UI the SCSI Controller (pveScsiHwSelector) is moved to the left column below the VGA selector. The new Spice enhancements components is placed in the right column and enabled if qxl/spice is selected in the VGA selector. Signed-off-by: Aaron Lauterer --- www/manager6/qemu

[pve-devel] [PATCH manager v2 0/4] Fix #2041, #2272 GUI for Spice Enhancements

2019-09-17 Thread Aaron Lauterer
-> v2: * code cleanup / improvement * rearranged the System panel in the VM creation wizard for a cleaner and easier to understand UI [0]: https://pve.proxmox.com/pipermail/pve-devel/2019-September/038799.html Aaron Lauterer (4): Cleanup: align backslashes in manager6 Makefile spice:

[pve-devel] [PATCH qemu-server] Whitespace cleanup

2019-09-17 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- PVE/CLI/qmrestore.pm | 18 +- PVE/QMPClient.pm | 38 +++--- PVE/QemuConfig.pm| 2 +- PVE/QemuServer/Memory.pm | 2 +- PVE/VZDump/QemuServer.pm | 2 +- 5 files changed, 31 insertions(+), 31

[pve-devel] [PATCH manager 2/2] Add online help to VM bios settings

2019-09-18 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/qemu/QemuBiosEdit.js | 1 + 1 file changed, 1 insertion(+) diff --git a/www/manager6/qemu/QemuBiosEdit.js b/www/manager6/qemu/QemuBiosEdit.js index a0fad159..7283df74 100644 --- a/www/manager6/qemu/QemuBiosEdit.js +++ b/www/manager6/qemu

[pve-devel] [PATCH manager 1/2] Add online help to VM display settings

2019-09-18 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/qemu/DisplayEdit.js | 1 + 1 file changed, 1 insertion(+) diff --git a/www/manager6/qemu/DisplayEdit.js b/www/manager6/qemu/DisplayEdit.js index e5af00aa..8c54b7a7 100644 --- a/www/manager6/qemu/DisplayEdit.js +++ b/www/manager6/qemu/DisplayEdit.js

[pve-devel] [PATCH manager 0/2] Add online help links where missing

2019-09-18 Thread Aaron Lauterer
I found two spots where documentation is available but the help button was missing. Aaron Lauterer (2): Add online help to VM display settings Add online help to VM bios settings www/manager6/qemu/DisplayEdit.js | 1 + www/manager6/qemu/QemuBiosEdit.js | 1 + 2 files changed, 2 insertions

Re: [pve-devel] [PATCH manager v2 2/4] spice: Add enhancements form component

2019-09-18 Thread Aaron Lauterer
On 9/18/19 12:11 PM, Dominik Csapak wrote: On 9/17/19 11:35 AM, Aaron Lauterer wrote: Signed-off-by: Aaron Lauterer ---   www/manager6/Makefile |  1 +   www/manager6/form/SpiceEnhancementSelector.js | 72 +++   2 files changed, 73 insertions(+)   create

Re: [pve-devel] [PATCH manager v2 3/4] spice: Add enhanecements to VM Options panel

2019-09-18 Thread Aaron Lauterer
On 9/18/19 12:12 PM, Dominik Csapak wrote: On 9/17/19 11:35 AM, Aaron Lauterer wrote: Signed-off-by: Aaron Lauterer ---   www/manager6/Utils.js    | 18 ++   www/manager6/qemu/Options.js | 13 +   2 files changed, 31 insertions(+) diff --git a/www/manager6

[pve-devel] [PATCH widget-toolkit] Cleanup whitespace

2019-09-18 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- button/Button.js | 4 ++-- data/DiffStore.js | 4 ++-- data/reader/JsonObject.js | 14 +++--- mixin/CBind.js| 14 +++--- node/TimeView.js | 14 +++--- panel/RRDChart.js | 14

Re: [pve-devel] applied-series: [PATCH v4 0/4] Fix #2343 Spice USB3 support

2019-09-23 Thread Aaron Lauterer
recht wrote: On 9/23/19 7:51 AM, Thomas Lamprecht wrote: On 9/11/19 2:43 PM, Aaron Lauterer wrote: This patch series enables USB3 for the passthrough / redirection of USB devices via the Spice client. v3 -> v4: * cleanup of redundant if condition * avoiding cyclic module dependency from U

Re: [pve-devel] applied-series: [PATCH v4 0/4] Fix #2343 Spice USB3 support

2019-09-24 Thread Aaron Lauterer
:23 PM, Aaron Lauterer wrote: I did some short tests with the latest commits and there is the following problem: Adding a USB3 device or port will result in such an error msg: Parameter verification failed. (400) usb1: invalid format - duplicate key in comma-separated list property

Re: [pve-devel] [PATCH manager v2 4/4] spice: Add enhancements to VM Creation wizard

2019-09-24 Thread Aaron Lauterer
The fields relation is not visible, also it looks like a checkbox misses "Spice enhancements" as it's really not clear that this should be some sort of "option group heading" On 9/17/19 11:35 AM, Aaron Lauterer wrote: For a cleaner UI the SCSI Controller (pveScsiHwSelector

[pve-devel] [PATCH manager 1/2] ui: vm/usbedit: add gettext to label

2019-09-25 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/qemu/USBEdit.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/qemu/USBEdit.js b/www/manager6/qemu/USBEdit.js index 8cad8fc5..f41c3d21 100644 --- a/www/manager6/qemu/USBEdit.js +++ b/www/manager6/qemu/USBEdit.js

[pve-devel] [PATCH manager 2/2] ui: vm/usbedit: refactor usb3 checkbox handling

2019-09-25 Thread Aaron Lauterer
support of the selected device / port. Signed-off-by: Aaron Lauterer --- What started out to fix the problem discussed in the previous patch series [0] turned into a refactoring of the controller. I hope I got the regexes right. Since we do not disable the USB3 checkbox anymore I am not sure why

[pve-devel] [PATCH manager 2/2] ui: vm/usbselector: cleanup unused code

2019-09-26 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/form/USBSelector.js | 10 -- 1 file changed, 10 deletions(-) diff --git a/www/manager6/form/USBSelector.js b/www/manager6/form/USBSelector.js index ed35bde0..82b59722 100644 --- a/www/manager6/form/USBSelector.js +++ b/www/manager6/form

[pve-devel] [PATCH manager 1/2] ui: vm/usbedit: simplify USB3 handling

2019-09-26 Thread Aaron Lauterer
devices and leave the decision to the user if USB3 should be enabled or not. Signed-off-by: Aaron Lauterer --- I don't want to call this v2 as we do things very differently than in the previous patch [0] that dealt with this. [0]: https://pve.proxmox.com/pipermail/pve-devel/2019-September/0

[pve-devel] [PATCH manager] ui: qemubiosedit: add gettext to efi disk hint

2019-09-30 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- Put the hint in one line because AFAIU our tooling cannot handle multiline gettext calls. www/manager6/qemu/QemuBiosEdit.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/www/manager6/qemu/QemuBiosEdit.js b/www/manager6/qemu

[pve-devel] [PATCH v3 manager 2/3] ui: add spice enhancements form component

2019-10-04 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- www/manager6/Makefile | 1 + www/manager6/form/SpiceEnhancementSelector.js | 60 +++ 2 files changed, 61 insertions(+) create mode 100644 www/manager6/form/SpiceEnhancementSelector.js diff --git a/www/manager6/Makefile

[pve-devel] [PATCH docs 1/3] add spice enhancements documentation

2019-10-04 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- qm.adoc | 56 1 file changed, 56 insertions(+) diff --git a/qm.adoc b/qm.adoc index 0a1dfa6..583a248 100644 --- a/qm.adoc +++ b/qm.adoc @@ -819,6 +819,62 @@ start after those where the parameter is set

[pve-devel] [PATCH v3 0/3] Fix #2041, #2272 GUI for Spice Enhancements

2019-10-04 Thread Aaron Lauterer
for a cleaner and easier to understand UI [0]: https://pve.proxmox.com/pipermail/pve-devel/2019-September/038799.html pve-docs: Aaron Lauterer (1): qm.adoc | 56 1 file changed, 56 insertions(+) pve-manager: Aaron Lauterer (2): ui: add

[pve-devel] [PATCH v3 manager 3/3] ui: vm-options: add spice enhancements

2019-10-04 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- removing the check if values is true and changing the parameters in parsePropertyString to `parsePropertyString(values || {})` as suggested in [0] does not work. parsePropertyString expects a string. Passing anything except a string containing a `key=value

[pve-devel] [PATCH docs] display: add URL and short explanation of SPICE

2019-10-07 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- There was no explanation of what SPICE is or a link to the project. qm.adoc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/qm.adoc b/qm.adoc index 5f79547..e55fa50 100644 --- a/qm.adoc +++ b/qm.adoc @@ -636,7 +636,8 @@ necessary

[pve-devel] [PATCH v4 docs 1/3] add spice enhancements documentation

2019-10-07 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- qm.adoc | 63 + 1 file changed, 63 insertions(+) diff --git a/qm.adoc b/qm.adoc index 0a1dfa6..6eada34 100644 --- a/qm.adoc +++ b/qm.adoc @@ -819,6 +819,69 @@ start after those where the parameter is set

[pve-devel] [PATCH v4 manager 3/3] ui: vm-options: add spice enhancements

2019-10-07 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- As suggested by Thomas in [0] I removed the gettext around the feature names to have the same feature names accross all translations. This should help if people start searching for a problem with them. Removed the check if values is falsy because

[pve-devel] [PATCH v4 manager 2/3] ui: add spice enhancements form component

2019-10-07 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- As suggested by Thomas in [0] I removed the gettext around the feature names to have the same feature names accross all translations. This should help if people start searching for a problem with them. The check if the hint to set SPICE in display is using a

[pve-devel] [PATCH v4 0/3] Fix #2041, #2272 GUI for Spice Enhancements

2019-10-07 Thread Aaron Lauterer
a cleaner and easier to understand UI [0]: https://pve.proxmox.com/pipermail/pve-devel/2019-September/038799.html [1]: https://git.proxmox.com/?p=pve-manager.git;a=commit;h=a017d8aaa1c7338745b0be55bc1d8d386cdc2b74 docs: Aaron Lauterer (1): qm.

Re: [pve-devel] applied: [PATCH v4 docs 1/3] add spice enhancements documentation

2019-10-07 Thread Aaron Lauterer
On 10/7/19 4:06 PM, Thomas Lamprecht wrote: On 10/7/19 3:23 PM, Aaron Lauterer wrote: Signed-off-by: Aaron Lauterer --- qm.adoc | 63 + 1 file changed, 63 insertions(+) applied, thanks! Did a small fixup (4th order headings do

Re: [pve-devel] Trying to use spice enhancements

2019-10-07 Thread Aaron Lauterer
Thanks for finding that! I'm looking into it. On 10/7/19 7:53 PM, Gilberto Nunes wrote: Never mind I forget to enable qemu agent in the VM... Thanks --- Gilberto Nunes Ferreira (47) 3025-5907 (47) 99676-7530 - Whatsapp / Telegram Skype: gilberto.nunes36 Em seg, 7 de out de 2019 às

[pve-devel] [PATCH qemu-server] cfg2cmd: fix serial-bus for spice foldersharing

2019-10-08 Thread Aaron Lauterer
the spice vdagent it seems sensible to use that also for the foldersharing device by moving it in front of the other spice devices. [0]: https://pve.proxmox.com/pipermail/pve-devel/2019-October/039441.html Signed-off-by: Aaron Lauterer --- PVE/QemuServer.pm | 10 +- 1 file changed, 5

Re: [pve-devel] Spice foldersharing

2019-10-09 Thread Aaron Lauterer
The spice-webdavd daemon is part of the Spice project, not something we develop here at Proxmox. Also as Thomas already pointed out, Windows XP is very old and end of life -> It's very much possible that it just doesn't work on Windows XP. The error you are getting sounds like it :/ I never

Re: [pve-devel] [PATCH manager] ui: vm opts: clarify label for QGA

2019-10-10 Thread Aaron Lauterer
On 10/10/19 12:21 PM, Dominik Csapak wrote: On 10/10/19 11:54 AM, Thomas Lamprecht wrote: To make it more clear that PVE does not somehow magically injects a QHA into the VM, but that this can be set if one has installed the QGA in the VM themself. good idea to make it clearer, but i think t

Re: [pve-devel] [PATCH pve-docs] Add section for ZFS Special Device

2019-11-05 Thread Aaron Lauterer
Nicely written. I have some suggestions inline: * splitting long sentences * adding more info as to what is valid for the size in special_small_blocks (taken from the zfs man page) * rewrote the last paragraph a bit On 10/22/19 12:33 PM, Fabian Ebner wrote: > Signed-off-by: Fabian Ebner > ---

Re: [pve-devel] [PATCH docs 02/11] pveceph: add section - Destroying Ceph OSDs

2019-11-05 Thread Aaron Lauterer
Some suggestions inline. On 11/4/19 2:52 PM, Alwin Antreich wrote: Signed-off-by: Alwin Antreich --- pveceph.adoc | 31 + images/screenshot/gui-ceph-osd-destroy.png | Bin 0 -> 146184 bytes 2 files changed, 31 insertions(+) create mode 10

Re: [pve-devel] [PATCH docs 03/11] pveceph: add section - Destroying Ceph Monitor

2019-11-05 Thread Aaron Lauterer
On 11/4/19 2:52 PM, Alwin Antreich wrote: Signed-off-by: Alwin Antreich --- pveceph.adoc | 19 ++ .../screenshot/gui-ceph-monitor-destroy.png | Bin 0 -> 154084 bytes 2 files changed, 19 insertions(+) create mode 100644 images/screensh

Re: [pve-devel] [PATCH docs 02/11] pveceph: add section - Destroying Ceph OSDs

2019-11-05 Thread Aaron Lauterer
On 11/5/19 10:40 AM, Aaron Lauterer wrote: To remove an OSD via the GUI first select the {PVE} node in the tree view. Select the **Ceph -> OSD** panel and select the OSD to destroy. Next click the **OUT** button. Once the OSD's status changed from `in` to `out` click the **STOP*

Re: [pve-devel] [PATCH docs 05/11] pveceph: add section - Destroying Ceph Manager

2019-11-05 Thread Aaron Lauterer
On 11/4/19 2:52 PM, Alwin Antreich wrote: Signed-off-by: Alwin Antreich --- pveceph.adoc | 20 ++ .../screenshot/gui-ceph-manager-destroy.png | Bin 0 -> 153596 bytes 2 files changed, 20 insertions(+) create mode 100644 images/screensh

Re: [pve-devel] [PATCH docs 06/11] pveceph: add section - Destroying Ceph Pools

2019-11-05 Thread Aaron Lauterer
On 11/4/19 2:52 PM, Alwin Antreich wrote: Signed-off-by: Alwin Antreich --- pveceph.adoc | 21 +++ images/screenshot/gui-ceph-pools-destroy.png | Bin 0 -> 141532 bytes 2 files changed, 21 insertions(+) create mode 100644 images/screensho

Re: [pve-devel] [PATCH docs 08/11] pveceph: Reorganize TOC for new sections

2019-11-05 Thread Aaron Lauterer
In general wouldn't it be better to change the headlines to active voice? Create Monitors instead of Creating Monitors? More suggestions inline. On 11/4/19 2:52 PM, Alwin Antreich wrote: Put the previous added sections into subsection for a better outline of the TOC. With the rearrangement of

Re: [pve-devel] [PATCH docs 10/11] Fix #1958: pveceph: add section Ceph maintenance

2019-11-05 Thread Aaron Lauterer
On 11/4/19 2:52 PM, Alwin Antreich wrote: Signed-off-by: Alwin Antreich --- pveceph.adoc | 54 1 file changed, 54 insertions(+) diff --git a/pveceph.adoc b/pveceph.adoc index 087c4d0..127e3bb 100644 --- a/pveceph.adoc +++ b/pveceph.adoc

[pve-devel] [PATCH manager] ui: vm opts: add hint for spice foldersharing

2019-11-06 Thread Aaron Lauterer
Spice foldersharing needs the webdavd daemon installed inside the guest. This patch adds a hint to remind the user to install it in the VM. Signed-off-by: Aaron Lauterer --- www/manager6/form/SpiceEnhancementSelector.js | 13 + 1 file changed, 13 insertions(+) diff --git a/www

[pve-devel] [PATCH docs] qm: spice foldersharing: Add experimental warning

2019-11-06 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- qm.adoc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/qm.adoc b/qm.adoc index 9ee4460..c0fe892 100644 --- a/qm.adoc +++ b/qm.adoc @@ -856,6 +856,8 @@ Select the folder to share and then enable the checkbox. NOTE: Folder sharing currently only works in

Re: [pve-devel] [PATCH docs] qm: spice foldersharing: Add experimental warning

2019-11-06 Thread Aaron Lauterer
Hmm, What about: Currently this feature does not work reliably. On 11/6/19 3:29 PM, Alwin Antreich wrote: On Wed, Nov 06, 2019 at 03:20:59PM +0100, Aaron Lauterer wrote: Signed-off-by: Aaron Lauterer --- qm.adoc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/qm.adoc b/qm.adoc

[pve-devel] [PATCH v2 docs] qm: spice foldersharing: Add experimental warning

2019-11-06 Thread Aaron Lauterer
Signed-off-by: Aaron Lauterer --- v1 -> v2: added Currently qm.adoc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/qm.adoc b/qm.adoc index 9ee4460..35d7d57 100644 --- a/qm.adoc +++ b/qm.adoc @@ -856,6 +856,8 @@ Select the folder to share and then enable the checkbox. NOTE: Fol

[pve-devel] [PATCH qemu-server] api/migration: fix autocomplete for targetstorage

2019-11-18 Thread Aaron Lauterer
Show storages configured for the target node and not for the current one because they can be different. Duplicated the `complete_storage` sub and extended it to extract the targetnode from the parameters to pass it into the storage_check_enabled function. Signed-off-by: Aaron Lauterer --- PVE

[pve-devel] [PATCH v2 qemu-server] api/migration: fix autocomplete for targetstorage

2019-11-18 Thread Aaron Lauterer
Show storages configured for the target node and not for the current one because they can be different. Duplicated the `complete_storage` sub and extended it to extract the targetnode from the parameters to pass it into the storage_check_enabled function. Signed-off-by: Aaron Lauterer --- v1

[pve-devel] [PATCH qemu-server] print_vga_device: fix qxl displays on Linux guests

2019-11-19 Thread Aaron Lauterer
with pve-qemu-4.0.1-3 or higher it was not possible in a spice remote session to enable more displays on the fly in linux guests. Adding the `max_outputs` parameter to the qxl device manually restores the functionality. Signed-off-by: Aaron Lauterer --- PVE/QemuServer.pm| 10

Re: [pve-devel] [PATCH qemu-server] print_vga_device: fix qxl displays on Linux guests

2019-11-19 Thread Aaron Lauterer
On 11/19/19 4:13 PM, Thomas Lamprecht wrote: On 11/19/19 4:01 PM, Aaron Lauterer wrote: with pve-qemu-4.0.1-3 or higher it was not possible in a spice remote session to enable more displays on the fly in linux guests. Adding the `max_outputs` parameter to the qxl device manually restores

[pve-devel] [PATCH v2 qemu-server] print_vga_device: fix qxl displays on Linux guests

2019-11-19 Thread Aaron Lauterer
with pve-qemu-4.0.1-3 or higher it was not possible in a spice remote session to enable more displays on the fly in linux guests. Adding the `max_outputs` parameter to the qxl device manually restores the functionality. Signed-off-by: Aaron Lauterer --- v1 -> v2: change `my $max_outputs` f

Re: [pve-devel] [PATCH] Revision of the pvesr documentation

2019-11-22 Thread Aaron Lauterer
Some hints from my side inline. Rephrasing some passages trying to make it easier to read and understand. Did some sentence splitting an smaller corrections as well. Please check if the rephrased sections are still correct from a technical POV. On 11/15/19 9:51 AM, Wolfgang Link wrote: Imp

[pve-devel] [PATCH v2 docs 01/10] Overhaul Getting Help

2019-11-25 Thread Aaron Lauterer
general overhaul of the documentation, improving phrasing, move open source info Signed-off-by: Aaron Lauterer --- v1 -> v2: feedback from thomas[0]: * moved open source hint to mailing list for devs * added suggestions * regarding spaces around dashes (--): our current technical writ

[pve-devel] [PATCH v2 docs 00/10] Documenation overhaul chapt. 1.9 to 3.2

2019-11-25 Thread Aaron Lauterer
mail/pve-devel/2019-September/038938.html Aaron Lauterer (10): Overhaul Getting Help Overhaul How To Improve Docs Overhaul Translation Overhaul Installation Overhaul System Requirements Overhaul Install from USB flash drive Overhaul Sysadmin Overhaul Package Repositories OVerhau

[pve-devel] [PATCH v2 docs 02/10] Overhaul How To Improve Docs

2019-11-25 Thread Aaron Lauterer
general overhauling, improve phrasing Signed-off-by: Aaron Lauterer --- howto-improve-pve-docs.adoc | 38 + 1 file changed, 17 insertions(+), 21 deletions(-) diff --git a/howto-improve-pve-docs.adoc b/howto-improve-pve-docs.adoc index c0d277e..d96bb03 100644

[pve-devel] [PATCH v2 docs 03/10] Overhaul Translation

2019-11-25 Thread Aaron Lauterer
improve phrasing Signed-off-by: Aaron Lauterer --- v1 -> v2: applied suggestion from oguz [0] [0] https://pve.proxmox.com/pipermail/pve-devel/2019-September/038952.html translation.adoc | 37 +++-- 1 file changed, 19 insertions(+), 18 deletions(-) diff --gi

  1   2   3   4   >