[pve-devel] applied-series: [PATCH storage 1/2] test: bwlimit: fix test description

2022-11-23 Thread Thomas Lamprecht
Am 23/11/2022 um 12:40 schrieb Fiona Ebner: > Signed-off-by: Fiona Ebner > --- > test/run_bwlimit_tests.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > applied both patches, thanks! ___ pve-devel mailing list pve-devel@lists.proxmox.c

Re: [pve-devel] Proxmox VE 7.3 released!

2022-11-23 Thread Thomas Lamprecht
Am 23/11/2022 um 17:32 schrieb Gilberto Ferreira via pve-devel: > Kernel 5.19? Why? It's already EOL! No it isn't EOL for neither us, nor Ubuntu (using it in their 23.04 release). We both backport stable fixes periodically. > Why not jump to 6 or remain to 5.15? 5.15 is still the default and wil

Re: [pve-devel] Proxmox VE 7.3 released!

2022-11-23 Thread Gilberto Ferreira via pve-devel
--- Begin Message --- Kernel 5.19? Why? It's already EOL! Why not jump to 6 or remain to 5.15? --- Gilberto Nunes Ferreira (47) 99676-7530 - Whatsapp / Telegram Em ter., 22 de nov. de 2022 às 13:41, Gilberto Ferreira via pve-devel < pve-devel@lists.proxmox.com> escreveu: > > > > -- F

[pve-devel] [PATCH backup/pmg-api/manager/common v4] fix: #3971 Tasklog download button

2022-11-23 Thread Daniel Tschlatscher
This patch series' aim is to add a download button in the tasklog- viewer GUI so that users may access the tasklog more easily. (The tasklog-viewer only displays 50 lines at a time) Instead of suddenly returning a file stream when the 'limit' parameter is set to 0, now, a new parameter 'download' n

[pve-devel] [PATCH pmg-api v4] make tasklog downloadable in the PMG backend

2022-11-23 Thread Daniel Tschlatscher
The read_tasklog API call now stream the whole log file if the query parameter 'download' is set to true. This is done in preparation for the task log download button to be added in the TaskViewer. I saw an opportunity here to clear some redundant code for displaying the tasklog and replaced it wi

[pve-devel] [PATCH backup v4] make tasklog downloadable in the backup server backend

2022-11-23 Thread Daniel Tschlatscher
The read_tasklog API call now stream the whole log file if the query parameter 'download' is set to true. If the limit parameter is set to 0, all lines in the tasklog will be returned in json format. To make a file stream and a json response in the same API call work, I had to use one of the lower

[pve-devel] [PATCH manager v4] make task log downloadable in the PVE manager backend

2022-11-23 Thread Daniel Tschlatscher
The read_tasklog API call now stream the whole log file if the query parameter 'download' is set to true. This is done in preparation for the task log download button to be added in the TaskViewer. Signed-off-by: Daniel Tschlatscher --- Changes from v3: * API parameter 'download' for directly str

[pve-devel] [PATCH common v4] return whole log file if limit is 0

2022-11-23 Thread Daniel Tschlatscher
The dump_logfile now returns the whole log file if the limit parameter is set to 0. This must be done explicitely though, as in the case of 'limit' being undefined, the default as before, 50 will be used. Signed-off-by: Daniel Tschlatscher --- Changes from v3: * NEW patch This patch also affects

Re: [pve-devel] cluster resource scheduler question

2022-11-23 Thread DERUMIER, Alexandre
Ah ok, got it, thanks. I had totally missed HA/Usage/Static.pm , which is used in recompute_online_node_usage. I was a bit lost between perl && rust, but I begin to understand how it's working. (Nice code BTW ;) Le mercredi 23 novembre 2022 à 12:28 +0100, Fiona Ebner a écrit : > Am 23.11.22

[pve-devel] [PATCH storage 2/2] get bandwidth limit: improve detecting if storages are involved

2022-11-23 Thread Fiona Ebner
Previously, calling with e.g. $storage_list = [undef] would lead to an early return of $override and not consider the limit from datacenter.cfg. Refactoring the bandwidth limit handling for migration introduced calls such as described above, which broke applying the limit from datacenter.cfg for V

[pve-devel] [PATCH storage 1/2] test: bwlimit: fix test description

2022-11-23 Thread Fiona Ebner
Signed-off-by: Fiona Ebner --- test/run_bwlimit_tests.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/run_bwlimit_tests.pl b/test/run_bwlimit_tests.pl index 8488637..5e59bf0 100755 --- a/test/run_bwlimit_tests.pl +++ b/test/run_bwlimit_tests.pl @@ -188,7 +188,7 @@ my @

Re: [pve-devel] cluster resource scheduler question

2022-11-23 Thread Fiona Ebner
Am 23.11.22 um 11:18 schrieb DERUMIER, Alexandre: > After doing some test, > > > if I understand, the node static infos max-mem, max-cpu are only used > to have percents to be able to compare each node. > > and the nodeusage is compute with the sum of static mem/cpu of the > services running on

Re: [pve-devel] cluster resource scheduler question

2022-11-23 Thread Thomas Lamprecht
Am 23/11/2022 um 10:09 schrieb Fiona Ebner: >> I'm not sure to understand, but the video seem to say that vm is >> migrated to the least "loaded" node. > yes, the video is misleading there. The static node CPU and memory > values and the configured CPU and memory for guests will be used to > approx

Re: [pve-devel] cluster resource scheduler question

2022-11-23 Thread DERUMIER, Alexandre
After doing some test, if I understand, the node static infos max-mem, max-cpu are only used to have percents to be able to compare each node. and the nodeusage is compute with the sum of static mem/cpu of the services running on this node. But I really don't see where it's node in the rust co

Re: [pve-devel] cluster resource scheduler question

2022-11-23 Thread Fiona Ebner
Am 22.11.22 um 17:43 schrieb DERUMIER, Alexandre: > Hi, > > I was looking at the proxmox 7.3 video > https://www.proxmox.com/en/training/video-tutorials/item/what-s-new-in-proxmox-ve-7-3 > > showing the new cluster resource scheduling with static-ressource. > > I'm not sure to understand, but th