Re: [pve-devel] applied: [PATCH firewall] increase default nf_conntrack_max to kernel default

2021-07-08 Thread alexandre derumier
Hi, you can change it in the proxmox node firewall options. Le jeudi 08 juillet 2021 à 09:36 +0200, wb a écrit : > Hello Thomas, > > Currently with Proxmox, I have a Kubernetes node running on LXC. > However, I have encountered an issue on the Container Network > Interface (CNI) side and in orde

[pve-devel] [PATCH widget-toolkit] node: add MTU column in NetworkView

2021-07-08 Thread Oguz Bektas
hidden by default since it's an "advanced" setting Signed-off-by: Oguz Bektas --- src/node/NetworkView.js | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/node/NetworkView.js b/src/node/NetworkView.js index 2e83130..dff503a 100644 --- a/src/node/NetworkView.js +++ b/src/node/Network

[pve-devel] [PATCH v2 qemu-server 3/3] restore: remove efidisk from config if not backed up

2021-07-08 Thread Stefan Reiter
If it doesn't exist, there's no need to keep it around at all. Such a backup might easily be created if an efidisk is configured with BIOS set to anything but 'OVMF'. Unbreaks live-restore for such cases too. Signed-off-by: Stefan Reiter --- v1->v2: * Actually remove efidisk from config, not ju

[pve-devel] [PATCH pve-cluster] dc.cfg: Add notes to datacenter config

2021-07-08 Thread Dominic Jäger
Similar to notes for nodes. datacenter.cfg normally uses key-value pairs defined in the schema. We bypass this to allow potentially long comments at the top. Signed-off-by: Dominic Jäger --- data/PVE/DataCenterConfig.pm | 34 +- 1 file changed, 33 insertions(+), 1

[pve-devel] [PATCH manager] dc menu: close 3504: Add datacenter notes

2021-07-08 Thread Dominic Jäger
Like notes for nodes. Signed-off-by: Dominic Jäger --- Depends on the pve-cluster patch. www/manager6/dc/Config.js | 6 www/manager6/panel/NotesView.js | 49 + 2 files changed, 32 insertions(+), 23 deletions(-) diff --git a/www/manager6/dc/Config.js

[pve-devel] [PATCH qemu-server 2/3] cfg2cmd: only warn on non-existant ISO image

2021-07-08 Thread Stefan Reiter
...instead of hard failing. Requires a RPCEnvironment to issue the warnings, so init a dummy one in run_config2command_tests. Signed-off-by: Stefan Reiter --- PVE/QemuServer.pm| 10 ++ test/run_config2command_tests.pl | 2 ++ 2 files changed, 12 insertions(+) diff --git

[pve-devel] [PATCH qemu-server 1/3] live-restore: fail early if target storage doesn't exist

2021-07-08 Thread Stefan Reiter
Signed-off-by: Stefan Reiter --- PVE/QemuServer.pm | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index b0fe257..0d8affa 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -6381,13 +6381,13 @@ sub restore_proxmox_backup_arc

[pve-devel] [PATCH qemu-server 3/3] live-restore: ignore missing efidisk

2021-07-08 Thread Stefan Reiter
Don't attempt to restore a configured efidisk for SeaBIOS VMs when it is missing. This can happen, as configured efidisks do not get backed up if the BIOS setting is not OVMF. Note that in case of OVMF VMs a missing efidisk is still treated as an error, as that indicates a broken backup in general

[pve-devel] applied: [pbs-devel] [PATCH v3 proxmox-widget-toolkit 1/1] node: tasks: use helper to format status again

2021-07-08 Thread Thomas Lamprecht
On 08.07.21 11:14, Fabian Ebner wrote: > which avoids the all-caps "WARNINGS:" and uses the localized version. > The call was thrown out by the big overhaul in > 9e059d560c872f14aef0e586010a6b6a91ce2729. > > Signed-off-by: Fabian Ebner > --- > > New in v3, but essentially restoring the already a

[pve-devel] applied: [pbs-devel] [PATCH v3 proxmox-backup 1/1] ui: tasks: use format_task_status

2021-07-08 Thread Thomas Lamprecht
On 08.07.21 11:14, Fabian Ebner wrote: > Signed-off-by: Fabian Ebner > --- > > No changes from v2. > > www/panel/Tasks.js | 9 + > 1 file changed, 1 insertion(+), 8 deletions(-) > > applied, thanks! ___ pve-devel mailing list pve-devel@lis

[pve-devel] [PATCH/RFC v3 pve-container 1/1] restore: sanitize config: use new warn() function

2021-07-08 Thread Fabian Ebner
to make it more visible that the task finished with warnings. Signed-off-by: Fabian Ebner --- Dependency bump for pve-common needed. No changes from v2. src/PVE/LXC/Create.pm | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/src/PVE/LXC/Create.pm b/src/PVE/L

[pve-devel] [PATCH v3 proxmox-backup 1/1] ui: tasks: use format_task_status

2021-07-08 Thread Fabian Ebner
Signed-off-by: Fabian Ebner --- No changes from v2. www/panel/Tasks.js | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/www/panel/Tasks.js b/www/panel/Tasks.js index a194e478..a64e0eee 100644 --- a/www/panel/Tasks.js +++ b/www/panel/Tasks.js @@ -372,14 +372,7 @@ Ext.d

[pve-devel] [PATCH v3 proxmox-widget-toolkit 1/1] node: tasks: use helper to format status again

2021-07-08 Thread Fabian Ebner
which avoids the all-caps "WARNINGS:" and uses the localized version. The call was thrown out by the big overhaul in 9e059d560c872f14aef0e586010a6b6a91ce2729. Signed-off-by: Fabian Ebner --- New in v3, but essentially restoring the already applied 4294143f92f43c3288304b35de3419b035d38cd2. src/

[pve-devel] [PATCH-SERIES v3] improve warnings handling in UI and add to PVE

2021-07-08 Thread Fabian Ebner
Changes from v2: * dropped already applied patches * new patch to re-add helper usage to widget-toolkit's task list Not much remaining of the series, just re-use the helper where possible, and the example usage of the feature in PVE for container restore as non-root. The last patch requir

[pve-devel] [PATCH manager] ui: disk edit: Split out bandwidth limits

2021-07-08 Thread Dominic Jäger
Signed-off-by: Dominic Jäger Here two inputpanels 1. diskData with volid, storage... 2. diskBasicOptions with the checkboxes are on the first tab. This required the least changes to get a working version so that we can quickly get on the same page about what we want it to look like. I can also

[pve-devel] applied-series: [PATCH pve-kernel-meta 0/5] proxmox-boot-tool improvements

2021-07-08 Thread Thomas Lamprecht
On 07.07.21 23:09, Stoiko Ivanov wrote: > The following patchset addresses a few small issues reported during the PVE > 7.0 beta and after the 7.0 stable release. > > * patches 1+2 deal with grub-install being called during a distribution > upgrade on some systems (I did not manage to get a VM i

[pve-devel] [PATCH docs] pveceph: update pgcalc url

2021-07-08 Thread Dominik Csapak
currently the pgcalc tool is only available under 'old.ceph.com' Signed-off-by: Dominik Csapak --- pveceph.adoc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pveceph.adoc b/pveceph.adoc index b3b82dc..743866a 100644 --- a/pveceph.adoc +++ b/pveceph.adoc @@ -485,7 +485,7 @@

Re: [pve-devel] applied: [PATCH firewall] increase default nf_conntrack_max to kernel default

2021-07-08 Thread Thomas Lamprecht
Hello Julien, On 08.07.21 09:36, wb wrote: > Hello Thomas, > > Currently with Proxmox, I have a Kubernetes node running on LXC. However, I > have encountered an issue on the Container Network Interface (CNI) side and > in order for it to work, the parameter > /proc/sys/net/netfilter/nf_conntra

Re: [pve-devel] [PATCH manager 2/2] pve6to7: storage content: ignore misconfigured unreferenced volumes

2021-07-08 Thread Thomas Lamprecht
On 08.07.21 09:29, Fabian Grünbichler wrote: > On July 7, 2021 12:22 pm, Fabian Ebner wrote: >> If the same local storage is configured twice with content type >> separation, migration in PVE 6 would lead to the volumes being >> duplicated. As that would happen for every migration, such an issue >>

[pve-devel] applied: [PATCH firewall] increase default nf_conntrack_max to kernel default

2021-07-08 Thread wb
Hello Thomas, Currently with Proxmox, I have a Kubernetes node running on LXC. However, I have encountered an issue on the Container Network Interface (CNI) side and in order for it to work, the parameter /proc/sys/net/netfilter/nf_conntrack_max must be raised. You know that the container sett

[pve-devel] applied: [PATCH proxmox-archive-keyring] bump version to 2.0

2021-07-08 Thread Fabian Grünbichler
with your tags and a slighlty more explanatory commit message ;) On July 7, 2021 6:42 pm, Thomas Lamprecht wrote: > On 06.07.21 14:04, Fabian Grünbichler wrote: >> Signed-off-by: Fabian Grünbichler >> --- >> debian/changelog | 6 ++ >> debian/proxmox-archive-keyri

Re: [pve-devel] [PATCH manager 2/2] pve6to7: storage content: ignore misconfigured unreferenced volumes

2021-07-08 Thread Fabian Grünbichler
On July 7, 2021 12:22 pm, Fabian Ebner wrote: > If the same local storage is configured twice with content type > separation, migration in PVE 6 would lead to the volumes being > duplicated. As that would happen for every migration, such an issue > would likely be noticed already, and in PVE 7 such

[pve-devel] applied: [PATCH manager 1/2] pve6to7: storage content: skip scanning storage if shared

2021-07-08 Thread Thomas Lamprecht
On 07.07.21 12:22, Fabian Ebner wrote: > Shared storages are not scanned for migration either, so they cannot > be problematic in this context. This could lead to false positives > where it actually is completely unproblematic: > > https://forum.proxmox.com/threads/proxmox-ve-7-0-released.92007/po

Re: [pve-devel] [PATCH manager 1/2] pve6to7: storage content: skip scanning storage if shared

2021-07-08 Thread Fabian Grünbichler
On July 7, 2021 12:22 pm, Fabian Ebner wrote: > Shared storages are not scanned for migration either, so they cannot > be problematic in this context. This could lead to false positives > where it actually is completely unproblematic: > > https://forum.proxmox.com/threads/proxmox-ve-7-0-released.9

Re: [pve-devel] [PATCH pve-kernel-meta 2/5] proxmox-boot: divert call to grub-install to p-b-t init

2021-07-08 Thread Fabian Grünbichler
On July 7, 2021 11:09 pm, Stoiko Ivanov wrote: > This way all ESPs (in case of a legacy booted system) get an > updated grub installation. > > running only once between reboots (the markerfile is in /tmp) should > be enough. Sadly the environment does not provide a hint which version > grub is ins