[pve-devel] applied-series: Re: [PATCH-SERIES proxmox-apt/pve-rs] better detection of standard repositories

2021-07-30 Thread Thomas Lamprecht
On 29/07/2021 14:25, Fabian Ebner wrote: > I'll send the corresponding patches for pmg-rs and proxmox-backup > separately. pmg-rs and pve-rs really need to merge, at least into one repo, could be a workspace with a common, pmg and pve subcrates that produces two binary packages, just one package a

[pve-devel] [PATCH pve-storage] fix #3555: BTRFSPlugin: call free_image correctly

2021-07-30 Thread Hannes Laimer
Signed-off-by: Hannes Laimer --- PVE/Storage/BTRFSPlugin.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/Storage/BTRFSPlugin.pm b/PVE/Storage/BTRFSPlugin.pm index 4596b30..411cab9 100644 --- a/PVE/Storage/BTRFSPlugin.pm +++ b/PVE/Storage/BTRFSPlugin.pm @@ -410,7 +410,7

[pve-devel] [PATCH qemu-server] bootorder: fix double entry on cdrom edit

2021-07-30 Thread Dominik Csapak
We unconditionally added an entry into the bootorder whenever we edited the drive, even if it was already in there. Instead we only want to do that if the bootorder list does not contain it already. Signed-off-by: Dominik Csapak --- PVE/API2/Qemu.pm | 2 +- 1 file changed, 1 insertion(+), 1 dele

[pve-devel] [PATCH manager 1/1] ui: node/ZFS: add zstd to possible compression methods

2021-07-30 Thread Dominik Csapak
Signed-off-by: Dominik Csapak --- www/manager6/node/ZFS.js | 1 + 1 file changed, 1 insertion(+) diff --git a/www/manager6/node/ZFS.js b/www/manager6/node/ZFS.js index 515a19e9..8ea364bf 100644 --- a/www/manager6/node/ZFS.js +++ b/www/manager6/node/ZFS.js @@ -85,6 +85,7 @@ Ext.define('PVE.node.C

[pve-devel] [PATCH storage/manager] allow zstd compression for new zpools

2021-07-30 Thread Dominik Csapak
pve-storage: Dominik Csapak (1): api: disks: allow zstd compression for zfs pools PVE/API2/Disks/ZFS.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) pve-manager: Dominik Csapak (1): ui: node/ZFS: add zstd to possible compression methods www/manager6/node/ZFS.js | 1 + 1 file cha

[pve-devel] [PATCH storage 1/1] api: disks: allow zstd compression for zfs pools

2021-07-30 Thread Dominik Csapak
Signed-off-by: Dominik Csapak --- PVE/API2/Disks/ZFS.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/API2/Disks/ZFS.pm b/PVE/API2/Disks/ZFS.pm index b815ee7..0418794 100644 --- a/PVE/API2/Disks/ZFS.pm +++ b/PVE/API2/Disks/ZFS.pm @@ -316,7 +316,7 @@ __PACKAGE__->register

[pve-devel] applied: Re: [PATCH docs] fix #3557: qm/pci-passthrouh: add note about the 'All Functions' checkbox

2021-07-30 Thread Thomas Lamprecht
On 30/07/2021 08:38, Dominik Csapak wrote: > So that a user coming from the web-interface can see what the checkbox does. > > Signed-off-by: Dominik Csapak > --- > qm-pci-passthrough.adoc | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > applied, thanks!

[pve-devel] applied: Re: [PATCH manager] ui: qemu/PCIEdit: change onlineHelp link to vm config section

2021-07-30 Thread Thomas Lamprecht
On 30/07/2021 08:40, Dominik Csapak wrote: > link the user directly to the vm config section of the pci > passthrough docs, since that is whats happening on that panel. > > It still is on the same page as the whole passthrough docs, so > the remaining info is still there. > > Signed-off-by: Domin

[pve-devel] applied: Re: [PATCH pve-storage] fix #3555: BTRFSPlugin: call free_image correctly

2021-07-30 Thread Thomas Lamprecht
On 30/07/2021 13:04, Hannes Laimer wrote: > Signed-off-by: Hannes Laimer > --- > PVE/Storage/BTRFSPlugin.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > applied, thanks! Albeit the commit message was lacking background, so I amended it with: > The method is only derived in the D

[pve-devel] applied: Re: [PATCH qemu-server] bootorder: fix double entry on cdrom edit

2021-07-30 Thread Thomas Lamprecht
On 30/07/2021 13:28, Dominik Csapak wrote: > We unconditionally added an entry into the bootorder whenever we > edited the drive, even if it was already in there. argh. > Instead we only want to do > that if the bootorder list does not contain it already. > > Signed-off-by: Dominik Csapak > ---

Re: [pve-devel] [PATCH v2 storage 1/3] status: move unlink from http-server to enpoint

2021-07-30 Thread Thomas Lamprecht
On 22/07/2021 15:06, Lorenz Stechauner wrote: > this is the first step in which not the http server removes the > temporary file, but the worker itself. > > Signed-off-by: Lorenz Stechauner > --- > PVE/API2/Storage/Status.pm | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/PVE/API2/Stor

Re: [pve-devel] [PATCH v2 storage 1/1] status: add checksum and algorithm to file upload

2021-07-30 Thread Thomas Lamprecht
Missing prefix "fix #3505: " On 22/07/2021 15:06, Lorenz Stechauner wrote: > Signed-off-by: Lorenz Stechauner > --- > PVE/API2/Storage/Status.pm | 34 ++ > 1 file changed, 34 insertions(+) > > diff --git a/PVE/API2/Storage/Status.pm b/PVE/API2/Storage/Status.pm >

Re: [pve-devel] [PATCH-SERIES v2 http-server/storage/manager] add checksum and algorithm to iso upload

2021-07-30 Thread Thomas Lamprecht
On 22/07/2021 15:06, Lorenz Stechauner wrote: > changes to v1: > * dropped commits, that were already applied in the mean time > * better commit messages > * dropped the 'new-filename' - using 'filename' instead > * error handling for unlink > * dropped check for `new FormData()` in pve-manager > *

Re: [pve-devel] [PATCH storage 2/2] Ceph: add keyring parameter for external clusters

2021-07-30 Thread Thomas Lamprecht
On 21/07/2021 17:13, Aaron Lauterer wrote: > By adding the keyring for RBD storage or the secret for CephFS ones, it > is possible to add an external Ceph cluster with only one API call. > > Previously the keyring / secret file needed to be placed in > /etc/pve/priv/ceph/$storeID.{keyring,secret}

[pve-devel] applied: Re: [PATCH storage 1/1] api: disks: allow zstd compression for zfs pools

2021-07-30 Thread Thomas Lamprecht
On 30/07/2021 13:34, Dominik Csapak wrote: > Signed-off-by: Dominik Csapak > --- > PVE/API2/Disks/ZFS.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > applied, thanks! ___ pve-devel mailing list pve-devel@lists.proxmox.com https://list

[pve-devel] applied: Re: [PATCH manager 1/1] ui: node/ZFS: add zstd to possible compression methods

2021-07-30 Thread Thomas Lamprecht
On 30/07/2021 13:34, Dominik Csapak wrote: > Signed-off-by: Dominik Csapak > --- > www/manager6/node/ZFS.js | 1 + > 1 file changed, 1 insertion(+) > > applied, with d/control version bump for storage as follow up, thanks! ___ pve-devel mailing list

[pve-devel] applied: Re: [PATCH v2 manager] vzdump: allow backups from plugins

2021-07-30 Thread Thomas Lamprecht
On 19/07/2021 15:07, Oguz Bektas wrote: > remove type check for the specific plugins, instead we can deduce it > from the supported content type in config (this can only be set on > storages that declare to support backups). should also work with > external storage plugins. > > Signed-off-by: Oguz