From f784201c632594a300965e4ea939357d2882e416 Mon Sep 17 00:00:00 2001 From: Thomas Lamprecht Date: Mon, 25 Nov 2024 20:22:31 +0100 Subject: [PATCH] ui: datastore summary: start store updates after mount task is done Without this, we immediately start the store updates even before the browser created the (async) mount API request. So it's very likely that the first store load will still get an error due to the backing device of the datastore not being mounted yet. That in turn will trigger our error detection behavior in the load even listener and disable periodic store updates again. Move the start of the update into the taskDone handler. We do not need to check if the task succeeded, as either it did, and we will do periodic updates, or it did not and we do at least one update to load the current status and then stop again auto-loading the store anyway. Signed-off-by: Thomas Lamprecht --- www/datastore/Summary.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/datastore/Summary.js b/www/datastore/Summary.js index db65f788..be638f7c 100644 --- a/www/datastore/Summary.js +++ b/www/datastore/Summary.js @@ -272,9 +272,9 @@ Ext.define('PBS.DataStoreSummary', { method: 'POST', failure: response => Ext.Msg.alert(gettext('Error'), response.htmlStatus), success: function(response, options) { - me.up('panel').statusStore.startUpdate(); Ext.create('Proxmox.window.TaskViewer', { upid: response.result.data, + taskDone: () => me.up('panel').statusStore.startUpdate(), }).show(); }, });