[pve-devel] [PATCH manager] ui: qemu/Config: disable xtermjs and spice until status is loaded

[Dominik Csapak]

We enable/disable spice/xtermjs for the console button in the 'load'
callback of the statusstore, depending on the vms capabilities,
but until the first load there, the only safe option is novnc.

So we have to disable xtermjs and spice on start, else a click on
the button might open a window that cannot connect to the vm.

a forum user probably triggered this:
https://forum.proxmox.com/threads/unable-to-find-serial-interface-console-problem.87705

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
 www/manager6/qemu/Config.js | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/www/manager6/qemu/Config.js b/www/manager6/qemu/Config.js
index 10bf10a4..b5f8cc9c 100644
--- a/www/manager6/qemu/Config.js
+++ b/www/manager6/qemu/Config.js
@@ -199,6 +199,8 @@ Ext.define('PVE.qemu.Config', {
 	    disabled: !caps.vms['VM.Console'],
 	    hidden: template,
 	    consoleType: 'kvm',
+	    enableSpice: false,
+	    enableXtermjs: false,
 	    consoleName: vm.name,
 	    nodename: nodename,
 	    vmid: vmid,
-- 
2.20.1