Re: [pve-devel] [PATCH qemu-server] migrate: use correct target storage id for checks

2021-09-14 Thread Fabian Ebner
Ping Am 25.06.21 um 14:32 schrieb Fabian Ebner: The '--targetstorage' parameter does not apply to shared storages. Example for a problem solved with the enabled check: Given a VM with images only on a shared storage 'storeA', not available on the target node (i.e. restricted by the nodes proper

Re: [pve-devel] hetzner bug with pve-firewall

2021-09-14 Thread Josef Per Johansson
Hi, I can check it out for sure, not touching ebtables would be nice. Sent from Nine From: alexandre derumier Sent: Tuesday, 14 September 2021 02:28 To: Proxmox VE development discussion Subject: Re: [pve-devel] hetzner bug with pve-firewall Hi, I just send ano

Re: [pve-devel] [PATCH pve-docs 1/2] pmxcfs: language and style fixup

2021-09-14 Thread Lorenz Stechauner
patch looks good to me in general. see inline for notes On 13.09.21 18:00, Dylan Whyte wrote: minor language fixup replace usage of 'Proxmox VE' with '{pve}' Signed-off-by: Dylan Whyte --- pmxcfs.adoc | 62 ++--- 1 file changed, 30 insertions

Re: [pve-devel] [PATCH pve-docs 2/2] pmxcfs: add more config files and discuss symlinks

2021-09-14 Thread Lorenz Stechauner
also looks good, one comment inline On 13.09.21 18:00, Dylan Whyte wrote: adds an entry for some config files found in /etc/pve, which were missing. alphabetize the list, for better readability and add some minor fixes also adds an introduction section to the symbolic links section, to clarify t

Re: [pve-devel] [PATCH storage 1/1] fix #3609 add support to SMB3_11

2021-09-14 Thread Fabian Ebner
Works for me. Configuring 'smbversion 3.11' when the server has 'min protocol = SMB3' or 'min protocol = SMB3_11' makes mounting the CIFS possible again. Tested-by: Fabian Ebner Am 13.09.21 um 14:15 schrieb Moayad Almalat: --- PVE/Storage/CIFSPlugin.pm | 2 +- 1 file changed, 1 insertion(

Re: [pve-devel] [PATCH storage 1/1] fix #3580: plugins: make preallocation mode selectable for qcow2 and raw images

2021-09-14 Thread Fabian Ebner
Am 06.09.21 um 15:15 schrieb Lorenz Stechauner: the plugins for file based storages * BTRFS * CIFS * Dir * Glusterfs * NFS now allow the option 'preallocation'. 'preallocation' can have four values: * default * off * metadata * falloc * full see man pages for `qemu-img` for w

Re: [pve-devel] [PATCH pve-docs 2/2] pmxcfs: add more config files and discuss symlinks

2021-09-14 Thread Thomas Lamprecht
On 14.09.21 09:50, Lorenz Stechauner wrote: > On 13.09.21 18:00, Dylan Whyte wrote: >>       Symbolic links >>   ~~ >>   +Certain directories within the cluster file system use symbolic links, in >> order >> +to point to a node's own configuration files. Thus, the files pointed to in

Re: [pve-devel] [PATCH manager 2/2] fix 3850: ui: storage: using PreallocationSelector for file based storage types

2021-09-14 Thread Fabian Ebner
Am 06.09.21 um 15:15 schrieb Lorenz Stechauner: Signed-off-by: Lorenz Stechauner --- www/manager6/controller/StorageEdit.js | 6 ++ www/manager6/storage/Base.js | 18 ++ 2 files changed, 24 insertions(+) diff --git a/www/manager6/controller/StorageEdit.js b/w

[pve-devel] [PATCH v2 pve-docs 1/2] pmxcfs: language and style fixup

2021-09-14 Thread Dylan Whyte
minor language fixup replace usage of 'Proxmox VE' with '{pve}' Signed-off-by: Dylan Whyte --- Thanks for the feedback @lorenz! changes v2: - Refer to offline nodes as 'offline', rather than 'dead' pmxcfs.adoc | 68 ++--- 1 file changed, 33 inser

[pve-devel] [PATCH v2 pve-docs 2/2] pmxcfs: add more config files and discuss symlinks

2021-09-14 Thread Dylan Whyte
- adds an entry for some config files found in /etc/pve, which were missing. - alphabetize the lists, for better readability and add some minor fixes - also adds an introduction section to the symbolic links section, to clarify that they're specific to each host Signed-off-by: Dylan Whyte ---