[pve-devel] [PATCH novnc] Replace 'migrating' info strings with more generic 'connecting' messages

Emmanuel Kasper e.kasper at proxmox.com
Thu Jun 29 16:16:30 CEST 2017


This fixes the problem that a restarted container was seen as migrating
in the UI.

Background:
The check for migration happens on a 'disconnected' event, but can very well
come for other reasons, like entering reboot on the LXC console.
Since we have no idea about what have caused the disconnection, it is
better to display a more generic message.
---
 debian/patches/0001-add-pve-specific-js-code.patch | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/debian/patches/0001-add-pve-specific-js-code.patch b/debian/patches/0001-add-pve-specific-js-code.patch
index 401e53b..f4f5067 100644
--- a/debian/patches/0001-add-pve-specific-js-code.patch
+++ b/debian/patches/0001-add-pve-specific-js-code.patch
@@ -191,7 +191,7 @@ index 0000000..668835a
 +		if (migrating === undefined ||
 +		    migrating === true) {
 +		    // check (again) if migrating
-+		    me.UI.showStatus('Checking for Migration', 'warning', 5000);
++		    me.UI.showStatus('Waiting for connection...', 'warning', 5000);
 +		    me.API2Request({
 +			url: me.baseUrl + '/config',
 +			method: 'GET',
@@ -210,7 +210,7 @@ index 0000000..668835a
 +		    });
 +		} else {
 +		    // not migrating any more
-+		    me.UI.showStatus('Migration finished', 'warning');
++		    me.UI.showStatus('Connection resumed', 'warning');
 +		    clearInterval(check);
 +		    me.API2Request({
 +			url: '/cluster/resources',
-- 
2.11.0





More information about the pve-devel mailing list