[pve-devel] applied-series: [PATCH manager 1/2] ui: ceph/ServiceList: stop leaky versionsstores
Thomas Lamprecht
t.lamprecht at proxmox.com
Thu Apr 2 12:49:15 CEST 2020
On 4/2/20 10:53 AM, Dominik Csapak wrote:
> on the view destroy we have to stop the updatestore again,
> they do not get cleaned up by a destroy because until they are stopped
> the have a reference on themselves, which prevent a garbage collection
>
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
> www/manager6/ceph/ServiceList.js | 1 +
> 1 file changed, 1 insertion(+)
>
applied both patches with a followup, as 2/2 missed another autoLoad:
commit e475a7788c9cc6f194f8f6bdc6bc55e6e566bf30 (HEAD -> master, proxdev/master)
Author: Thomas Lamprecht <t.lamprecht at proxmox.com>
Date: Thu Apr 2 12:48:02 2020 +0200
ui: ceph/ServiceList: do not autoLoad the rstore
The 'autoStart' triggers a initial load already
Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
diff --git a/www/manager6/ceph/ServiceList.js b/www/manager6/ceph/ServiceList.js
index ce688eae..581bd554 100644
--- a/www/manager6/ceph/ServiceList.js
+++ b/www/manager6/ceph/ServiceList.js
@@ -134,7 +134,6 @@ Ext.define('PVE.node.CephServiceList', {
view.on('destroy', view.versionsstore.stopUpdate);
view.rstore = Ext.create('Proxmox.data.UpdateStore', {
- autoLoad: true,
autoStart: true,
interval: 3000,
storeid: 'ceph-' + view.type + '-list' + view.nodename,
More information about the pve-devel
mailing list