public inbox for pbs-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [pbs-devel] [PATCH proxmox-backup] ui: tape/BackupOverview: increase timeout for media-set content
@ 2021-07-13  7:09 Dominik Csapak
  2021-07-13  7:45 ` [pbs-devel] applied: " Dietmar Maurer
  0 siblings, 1 reply; 2+ messages in thread
From: Dominik Csapak @ 2021-07-13  7:09 UTC (permalink / raw)
  To: pbs-devel

a single catalog can be over 100MiB, and a media-set can have multiple
catalogs to read (no technical upper limit). On slow disks, this can
take much longer than 30 seconds (the default timeout).

The real solution would be to have some kind of index only for the gui
relevant part, e.g. a table in the beginning of the catalog, or
alternatively a seperate file with that info. Until we have such a
solution increase the timeout as a stopgap.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 www/tape/BackupOverview.js | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/www/tape/BackupOverview.js b/www/tape/BackupOverview.js
index 4ba53157..c368640c 100644
--- a/www/tape/BackupOverview.js
+++ b/www/tape/BackupOverview.js
@@ -125,6 +125,9 @@ Ext.define('PBS.TapeManagement.BackupOverview', {
 		let list = await Proxmox.Async.api2({
 		    method: 'GET',
 		    url: `/api2/extjs/tape/media/content`,
+		    // a big media-set with large catalogs can take a while to load
+		    // so we give a big (5min) timeout
+		    timeout: 5*60*1000,
 		    params: {
 			'media-set': media_set_uuid,
 		    },
-- 
2.30.2





^ permalink raw reply	[flat|nested] 2+ messages in thread

* [pbs-devel] applied: [PATCH proxmox-backup] ui: tape/BackupOverview: increase timeout for media-set content
  2021-07-13  7:09 [pbs-devel] [PATCH proxmox-backup] ui: tape/BackupOverview: increase timeout for media-set content Dominik Csapak
@ 2021-07-13  7:45 ` Dietmar Maurer
  0 siblings, 0 replies; 2+ messages in thread
From: Dietmar Maurer @ 2021-07-13  7:45 UTC (permalink / raw)
  To: Proxmox Backup Server development discussion, Dominik Csapak

applied

On 7/13/21 9:09 AM, Dominik Csapak wrote:
> a single catalog can be over 100MiB, and a media-set can have multiple
> catalogs to read (no technical upper limit). On slow disks, this can
> take much longer than 30 seconds (the default timeout).
>
> The real solution would be to have some kind of index only for the gui
> relevant part, e.g. a table in the beginning of the catalog, or
> alternatively a seperate file with that info. Until we have such a
> solution increase the timeout as a stopgap.
>
> Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
> ---
>   www/tape/BackupOverview.js | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/www/tape/BackupOverview.js b/www/tape/BackupOverview.js
> index 4ba53157..c368640c 100644
> --- a/www/tape/BackupOverview.js
> +++ b/www/tape/BackupOverview.js
> @@ -125,6 +125,9 @@ Ext.define('PBS.TapeManagement.BackupOverview', {
>   		let list = await Proxmox.Async.api2({
>   		    method: 'GET',
>   		    url: `/api2/extjs/tape/media/content`,
> +		    // a big media-set with large catalogs can take a while to load
> +		    // so we give a big (5min) timeout
> +		    timeout: 5*60*1000,
>   		    params: {
>   			'media-set': media_set_uuid,
>   		    },




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-07-13  7:45 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-13  7:09 [pbs-devel] [PATCH proxmox-backup] ui: tape/BackupOverview: increase timeout for media-set content Dominik Csapak
2021-07-13  7:45 ` [pbs-devel] applied: " Dietmar Maurer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal