yesBad/pve-storage
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix #4189: pbs: bump list_volumes timeout to 2mins

Browse Source 
When switching this from calling the external binary to
using the perl api client the timeout got reduced to 7
seconds, which is definitely insufficient for larger stores.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
This commit is contained in:
Wolfgang Bumiller
2022-08-17 12:32:37 +02:00
committed by Fabian Grünbichler
parent 78e3ebb040
commit c560cb58a5
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
PVE/Storage/PBSPlugin.pm
View File

@ -688,7 +688,7 @@ my sub snapshot_files_encrypted {
# TODO: use a client with native rust/proxmox-backup bindings to profit from # TODO: use a client with native rust/proxmox-backup bindings to profit from
# API schema checks and types # API schema checks and types
my sub pbs_api_connect { my sub pbs_api_connect {
my ($scfg, $password) = @_; my ($scfg, $password, $timeout) = @_;
my $params = {}; my $params = {};
@ -709,7 +709,7 @@ my sub pbs_api_connect {
%$params, %$params,
host => $scfg->{server}, host => $scfg->{server},
port => $scfg->{port} // 8007, port => $scfg->{port} // 8007,
timeout => 7, # cope with a 401 (3s api delay) and high latency timeout => ($timeout // 7), # cope with a 401 (3s api delay) and high latency
cookie_name => 'PBSAuthCookie', cookie_name => 'PBSAuthCookie',
); );
@ -724,7 +724,7 @@ sub list_volumes {
return $res if !grep { $_ eq 'backup' } @$content_types; return $res if !grep { $_ eq 'backup' } @$content_types;
my $password = pbs_get_password($scfg, $storeid); my $password = pbs_get_password($scfg, $storeid);
my $conn = pbs_api_connect($scfg, $password); my $conn = pbs_api_connect($scfg, $password, 120);
my $datastore = $scfg->{datastore}; my $datastore = $scfg->{datastore};
my $param = {}; my $param = {};