[pve-devel] [PATCH manager] ui: storage: find the correct storage record

2020-11-25 Thread Dominik Csapak
by default, findRecord only anchors at the beginning, e.g. findRecord('id', 'foo'); could find either an entry with id 'foo', 'foobar', 'foobaz', etc. extend the call to set 'exactMatch' to true, else we sometimes used the content types of storage e.g., 'local-lvm' for showing the panels for 'loca

Re: [pve-devel] [PATCH pve-zsync 0/1] Allow pve-zsync jobs to share dest

2020-11-25 Thread Dietmar Maurer
> It has been 5 months since my patch has been applied, however the version > for pve-zsync has not been incremented and this patch is not in the version > presented by the repo. What needs to be done? Ok, just bumped the version and created a new package. So this will be part of the next release

Re: [pve-devel] [PATCH pve-zsync 0/1] Allow pve-zsync jobs to share dest

2020-11-25 Thread Bruce Wainer
Hello, It has been 5 months since my patch has been applied, however the version for pve-zsync has not been incremented and this patch is not in the version presented by the repo. What needs to be done? Thank you, Bruce Wainer On Tue, Jun 23, 2020 at 4:48 AM Wolfgang Link wrote: > Hi Bruce, > >

Re: [pve-devel] [PATCH v7 pve-manager 12/15] sdn: display pending values

2020-11-25 Thread Thomas Lamprecht
On 24.11.20 14:52, Alexandre Derumier wrote: > Signed-off-by: Alexandre Derumier > --- > www/manager6/Utils.js | 29 ++ > www/manager6/sdn/ControllerView.js | 39 +--- > www/manager6/sdn/SubnetView.js | 49 +++--- > www/

[pve-devel] [PATCH widget-toolkit] grid/DiskList: restore info about ceph

2020-11-25 Thread Dominik Csapak
previously we printed this ceph info, but it got lost during refactoring to widget-toolkit, restore a slightly modified version of the code Signed-off-by: Dominik Csapak --- src/grid/DiskList.js | 25 +++-- 1 file changed, 23 insertions(+), 2 deletions(-) diff --git a/src/gr

[pve-devel] applied: [PATCH docs] add encryption section for PBS

2020-11-25 Thread Thomas Lamprecht
On 25.11.20 15:53, Fabian Ebner wrote: > Some parts from the PBS docs where re-used. > > Signed-off-by: Fabian Ebner > --- > pve-storage-pbs.adoc | 42 ++ > vzdump.adoc | 5 + > 2 files changed, 47 insertions(+) > > applied, thanks! _

[pve-devel] [PATCH docs] add encryption section for PBS

2020-11-25 Thread Fabian Ebner
Some parts from the PBS docs where re-used. Signed-off-by: Fabian Ebner --- pve-storage-pbs.adoc | 42 ++ vzdump.adoc | 5 + 2 files changed, 47 insertions(+) diff --git a/pve-storage-pbs.adoc b/pve-storage-pbs.adoc index 9527237..1bb0721 10

[pve-devel] [PATCH container 2/2] restore: add more informational messages

2020-11-25 Thread Fabian Grünbichler
to provide context for warnings/output created by various sub-operations. Signed-off-by: Fabian Grünbichler --- src/PVE/API2/LXC.pm | 5 + 1 file changed, 5 insertions(+) diff --git a/src/PVE/API2/LXC.pm b/src/PVE/API2/LXC.pm index a263104..824da40 100644 --- a/src/PVE/API2/LXC.pm +++ b/src

[pve-devel] [PATCH container 1/2] restore: pass target vmid to config recovery

2020-11-25 Thread Fabian Grünbichler
so that we get the correct warning prefix when the config contains bogus lines. Signed-off-by: Fabian Grünbichler --- src/PVE/API2/LXC.pm | 4 ++-- src/PVE/LXC/Create.pm | 19 +++ 2 files changed, 13 insertions(+), 10 deletions(-) diff --git a/src/PVE/API2/LXC.pm b/src/PVE/AP

[pve-devel] applied-series: [PATCH V2 pve-network 0/4] add ebgp-evpn support

2020-11-25 Thread Thomas Lamprecht
On 25.11.20 10:01, Alexandre Derumier wrote: > This add support for a new controller plugin "bgp", > to manage specific bgp section by host. > This is allowing ebgp > > (I'll send doc soon) > > Also some fixes on pending parser > > Changelog V2: > > - fix some bug in bgp plugin > > Alexandre D

[pve-devel] applied: [PATCH manager] api: metrics/server: test connection on add/update

2020-11-25 Thread Thomas Lamprecht
On 25.11.20 13:56, Dominik Csapak wrote: > just a basic check, but better than not checking at all > > Signed-off-by: Dominik Csapak > --- > PVE/API2/Cluster/MetricServer.pm | 5 + > PVE/Status/Plugin.pm | 7 +++ > 2 files changed, 12 insertions(+) > > applied, thanks! __

Re: [pve-devel] [PATCH manager v2] storage: base edit: Use the GUI keep-all default value

2020-11-25 Thread Thomas Lamprecht
On 25.11.20 14:42, Dominic Jäger wrote: > ... if the user hasn't seen the prune panel yet. > The GUI has as default value a ticked keep-all checkbox => keep-all=1. > Previously we sent nothing in this case which led to the no-keeps default of > keep-last=1. > > Signed-off-by: Dominic Jäger > ---

[pve-devel] applied: [PATCH manager] ui: storage edit: ensure to set keep-all by default

2020-11-25 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- www/manager6/storage/Base.js | 2 ++ 1 file changed, 2 insertions(+) diff --git a/www/manager6/storage/Base.js b/www/manager6/storage/Base.js index 70e18a81..13ba5b7b 100644 --- a/www/manager6/storage/Base.js +++ b/www/manager6/storage/Base.js @@ -67,6 +67,8 @

[pve-devel] [PATCH manager v2] storage: base edit: Use the GUI keep-all default value

2020-11-25 Thread Dominic Jäger
... if the user hasn't seen the prune panel yet. The GUI has as default value a ticked keep-all checkbox => keep-all=1. Previously we sent nothing in this case which led to the no-keeps default of keep-last=1. Signed-off-by: Dominic Jäger --- v2: Move the keep-all=1 default up @Thomas: Or did yo

[pve-devel] applied: [PATCH qemu-server] qmeventd: flush after verbose printing

2020-11-25 Thread Thomas Lamprecht
On 24.11.20 15:04, Dominik Csapak wrote: > if one would try to use -v in a systemd service, systemd would disable > line buffering for stdout and no output would happen (until the buffer > is full) > > Signed-off-by: Dominik Csapak > --- > qmeventd/qmeventd.h | 2 +- > 1 file changed, 1 insertio

[pve-devel] applied-series: [PATCH V2 frr 0/3] update frr to 7.4

2020-11-25 Thread Thomas Lamprecht
On 25.11.20 13:22, Alexandre Derumier wrote: > Please update mirror to frr stable/7.4 branch (on last commit) > > 7.4 tag release have an evpn bug, and 7.4.1 is still not released. > (and 7.5 have a new libyang version depencency not available in buster) > > This release fix some evpn bugs, > I h

[pve-devel] [PATCH manager] api: metrics/server: test connection on add/update

2020-11-25 Thread Dominik Csapak
just a basic check, but better than not checking at all Signed-off-by: Dominik Csapak --- PVE/API2/Cluster/MetricServer.pm | 5 + PVE/Status/Plugin.pm | 7 +++ 2 files changed, 12 insertions(+) diff --git a/PVE/API2/Cluster/MetricServer.pm b/PVE/API2/Cluster/MetricServer.pm

[pve-devel] applied: [PATCH v3 manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Thomas Lamprecht
On 25.11.20 11:36, Wolfgang Bumiller wrote: > Signed-off-by: Wolfgang Bumiller > --- > changes to v2: > * use `for of` loop in verify_ip64_address_list > > www/manager6/Toolkit.js | 17 ++--- > www/manager6/Utils.js | 27 +++ > www/manager6/lxc/DNS.js | 2 +

Re: [pve-devel] [PATCH manager] storage: base edit: Use the GUI keep-all default value

2020-11-25 Thread Thomas Lamprecht
On 25.11.20 12:36, Dominic Jäger wrote: > ... if the user hasn't seen the prune panel yet. > The GUI has as default value a ticked keep-all checkbox => keep-all=1. > Previously we sent nothing in this case which led to the no-keeps default of > keep-last=1. > > Furthermore, refactor so that "The a

[pve-devel] [PATCH V2 frr 0/3] update frr to 7.4

2020-11-25 Thread Alexandre Derumier
Please update mirror to frr stable/7.4 branch (on last commit) 7.4 tag release have an evpn bug, and 7.4.1 is still not released. (and 7.5 have a new libyang version depencency not available in buster) This release fix some evpn bugs, I have also added a patch for ebgp, I'm currently trying to ge

[pve-devel] [PATCH V2 frr 2/3] patch : add autort as derivation

2020-11-25 Thread Alexandre Derumier
pending upstream https://github.com/FRRouting/frr/pull/7535/ This is for ebgp, to avoid to reload frr each time when we define new vxlan. Signed-off-by: Alexandre Derumier --- ...on-for-RT-auto-derivation-to-force-A.patch | 151 ++ debian/patches/series |

[pve-devel] [PATCH V2 frr 1/3] patch: enable bfd daemon

2020-11-25 Thread Alexandre Derumier
Signed-off-by: Alexandre Derumier --- .../patches/pve/0001-enable-bgp-daemon.patch | 25 +-- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/debian/patches/pve/0001-enable-bgp-daemon.patch b/debian/patches/pve/0001-enable-bgp-daemon.patch index d972f18..a9818c9 10

[pve-devel] [PATCH V2 frr 3/3] bump to 7.4

2020-11-25 Thread Alexandre Derumier
Signed-off-by: Alexandre Derumier --- Makefile | 2 +- debian/changelog | 6 ++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 596ac53..5ed61ca 100644 --- a/Makefile +++ b/Makefile @@ -1,5 +1,5 @@ PACKAGE=frr -VER=7.2.1 +VER=7.4 PKGREL=1+pve

[pve-devel] applied: [PATCH manager 2/2] ui: fix labelWidth for 'Key'

2020-11-25 Thread Thomas Lamprecht
On 25.11.20 13:04, Dominik Csapak wrote: > it is not that short in other languages, e.g. German > > Signed-off-by: Dominik Csapak > --- > www/manager6/storage/PBSEdit.js | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > applied, thanks! _

[pve-devel] applied: [PATCH manager] ui: storage: pbsedit: Fix spelling

2020-11-25 Thread Thomas Lamprecht
On 25.11.20 13:15, Dominic Jäger wrote: > No e in unusable. > > to loose means to free from restraint, or to make less tight or strict [1]. > to lose means to fail to keep, or to miss something from one's possession [2]. > So "losing" (instead of "loosing") would be bad. > > [0] https://www.merri

[pve-devel] applied: [PATCH manager] ui: storage edit: avoid returning prune values if unsupported

2020-11-25 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- www/manager6/storage/Base.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/www/manager6/storage/Base.js b/www/manager6/storage/Base.js index 80bc8002..70e18a81 100644 --- a/www/manager6/storage/Base.js +++ b/www/manager6/storage/Base.js @@ -65,6 +65,9

[pve-devel] [PATCH manager] ui: storage: pbsedit: Fix spelling

2020-11-25 Thread Dominic Jäger
No e in unusable. to loose means to free from restraint, or to make less tight or strict [1]. to lose means to fail to keep, or to miss something from one's possession [2]. So "losing" (instead of "loosing") would be bad. [0] https://www.merriam-webster.com/dictionary/unusable [1] https://www.mer

Re: [pve-devel] [PATCH manager 1/2] ui: fix typo

2020-11-25 Thread Dominik Csapak
On 11/25/20 1:04 PM, Dominik Csapak wrote: see https://www.merriam-webster.com/dictionary/unusable Signed-off-by: Dominik Csapak --- www/manager6/storage/PBSEdit.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/storage/PBSEdit.js b/www/manager6/storage/PBSE

[pve-devel] [PATCH manager 2/2] ui: fix labelWidth for 'Key'

2020-11-25 Thread Dominik Csapak
it is not that short in other languages, e.g. German Signed-off-by: Dominik Csapak --- www/manager6/storage/PBSEdit.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/storage/PBSEdit.js b/www/manager6/storage/PBSEdit.js index 84de238c..394a027b 100644 --- a/www/m

[pve-devel] [PATCH manager 1/2] ui: fix typo

2020-11-25 Thread Dominik Csapak
see https://www.merriam-webster.com/dictionary/unusable Signed-off-by: Dominik Csapak --- www/manager6/storage/PBSEdit.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/storage/PBSEdit.js b/www/manager6/storage/PBSEdit.js index 444a7e44..84de238c 100644 --- a/ww

Re: [pve-devel] [PATCH v3 manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Dietmar Maurer
> What text do you mean exactly? The interface name? > Arbitrary null-terminated byte string... Ok > (Yes I can name an interface "---" or 💩 (poop-emoji)..., > neither of which our iface schema in JSONSchema.pm would allow...) great. ___ pve-devel ma

Re: [pve-devel] [PATCH v3 manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Wolfgang Bumiller
What text do you mean exactly? The interface name? Arbitrary null-terminated byte string... (Yes I can name an interface "---" or 💩 (poop-emoji)..., neither of which our iface schema in JSONSchema.pm would allow...) > On 11/25/2020 12:39 PM Dietmar Maurer wrote: > > > Thanks for the info. But

[pve-devel] applied: [PATCH docs] pvecm: improve SSH section

2020-11-25 Thread Fabian Grünbichler
reword some parts, fix wrong information about port forwarding Signed-off-by: Fabian Grünbichler --- pvecm.adoc | 18 ++ 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/pvecm.adoc b/pvecm.adoc index cbbcf60..0fad203 100644 --- a/pvecm.adoc +++ b/pvecm.adoc @@ -872,

Re: [pve-devel] [PATCH v3 manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Dietmar Maurer
Thanks for the info. But what encoding does that text use? I cannot find that in RFC4007 (they only talk about strings and text). > > Answering myself, it is defined in RFC4007. > > > > But "man resolv.conf" say address must be RFC2373 ? > > It'll still work. It's a very common notation for lin

[pve-devel] [PATCH manager] storage: base edit: Use the GUI keep-all default value

2020-11-25 Thread Dominic Jäger
... if the user hasn't seen the prune panel yet. The GUI has as default value a ticked keep-all checkbox => keep-all=1. Previously we sent nothing in this case which led to the no-keeps default of keep-last=1. Furthermore, refactor so that "The always delete old 'maxfiles' on edit" is now clearly

Re: [pve-devel] [PATCH v3 manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Wolfgang Bumiller
> On 11/25/2020 12:22 PM Dietmar Maurer wrote: > > > Answering myself, it is defined in RFC4007. > > But "man resolv.conf" say address must be RFC2373 ? It'll still work. It's a very common notation for link local addresses, since with multiple interfaces you have multiple routes with the exa

[pve-devel] applied: [PATCH manager] api2/metrics/server: add minimum/maximum port

2020-11-25 Thread Thomas Lamprecht
On 25.11.20 12:06, Dominik Csapak wrote: > we just added the api, so it would be good to only accept valid ports > (they were wrapped before) > > Signed-off-by: Dominik Csapak > --- > PVE/Status/Plugin.pm | 2 ++ > 1 file changed, 2 insertions(+) > > applied, thanks! side note, can we please

Re: [pve-devel] [PATCH v3 manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Dietmar Maurer
Answering myself, it is defined in RFC4007. But "man resolv.conf" say address must be RFC2373 ? > On 11/25/2020 12:08 PM Dietmar Maurer wrote: > > > What kind of format is that? RFC2373 does not mention it. Please can > you give me a hint? > ___

Re: [pve-devel] [PATCH v3 manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Stoiko Ivanov
On Wed, 25 Nov 2020 12:08:39 +0100 (CET) Dietmar Maurer wrote: > What kind of format is that? RFC2373 does not mention it. Please can > you give me a hint? was curious myself - since I know the format but wasn't sure where it's specified. ripe has a nice explanation: https://labs.ripe.net/Members

Re: [pve-devel] [PATCH v3 manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Dietmar Maurer
What kind of format is that? RFC2373 does not mention it. Please can you give me a hint? > On 11/25/2020 11:36 AM Wolfgang Bumiller wrote: > > > Signed-off-by: Wolfgang Bumiller > --- > changes to v2: > * use `for of` loop in verify_ip64_address_list > > www/manager6/Toolkit.js | 17 ++

[pve-devel] [PATCH manager] api2/metrics/server: add minimum/maximum port

2020-11-25 Thread Dominik Csapak
we just added the api, so it would be good to only accept valid ports (they were wrapped before) Signed-off-by: Dominik Csapak --- PVE/Status/Plugin.pm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/PVE/Status/Plugin.pm b/PVE/Status/Plugin.pm index baf670ae..0faf0da6 100644 --- a/PVE/Stat

[pve-devel] applied: [PATCH pve-qemu] update patches with some pbs-state migration cleanups

2020-11-25 Thread Fabian Grünbichler
On November 24, 2020 5:32 pm, Fabian Grünbichler wrote: > Tested-by: Fabian Grünbichler > > requires now updated patch from <20201124154122.22202-2-s.rei...@proxmox.com>: > > "[PATCH v3 pve-qemu 1/3] update patches with squashed in 'include library > version'" and applied now after that one ;)

[pve-devel] applied: [PATCH v3 qemu-server 3/3] print query-proxmox-support result in 'full' status

2020-11-25 Thread Fabian Grünbichler
with mentioned followup On November 24, 2020 4:41 pm, Stefan Reiter wrote: > Extends print_recursive_hash for the CLI to handle JSON booleans so the > result will actually show up in 'qm status --verbose'. > > Signed-off-by: Stefan Reiter > --- > > v3: unchanged > > PVE/CLI/qm.pm | 2 +- >

[pve-devel] applied: [PATCH v3 pve-qemu 1/3] update patches with squashed in 'include library version'

2020-11-25 Thread Fabian Grünbichler
with fixup to bump versioned build-dep and add versioned dependency On November 24, 2020 4:41 pm, Stefan Reiter wrote: > Signed-off-by: Stefan Reiter > --- > > v3: use function instead of macro from lib > > Requires a depends on bumped library to find function. > > ...dd-query_proxmox_support

[pve-devel] applied: [PATCH docs 2/3] qm: add section about VM boot order

2020-11-25 Thread Thomas Lamprecht
On 24.11.20 17:52, Stefan Reiter wrote: > ...and fix outdated example config > > Signed-off-by: Stefan Reiter > --- > qm.adoc | 35 ++- > 1 file changed, 34 insertions(+), 1 deletion(-) > > applied, thanks! Changed the section heading to "Device Boot Order", as

[pve-devel] applied: [PATCH docs 1/3] images: add screenshot of bootorder dialog

2020-11-25 Thread Thomas Lamprecht
On 24.11.20 17:52, Stefan Reiter wrote: > Signed-off-by: Stefan Reiter > --- > images/screenshot/gui-qemu-edit-bootorder.png | Bin 0 -> 24530 bytes > 1 file changed, 0 insertions(+), 0 deletions(-) > create mode 100644 images/screenshot/gui-qemu-edit-bootorder.png > > applied, thanks! _

[pve-devel] [PATCH v3 manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Wolfgang Bumiller
Signed-off-by: Wolfgang Bumiller --- changes to v2: * use `for of` loop in verify_ip64_address_list www/manager6/Toolkit.js | 17 ++--- www/manager6/Utils.js | 27 +++ www/manager6/lxc/DNS.js | 2 +- 3 files changed, 30 insertions(+), 16 deletions(-) diff

[pve-devel] [PATCH manager] ui: metrics: Link help button to correct chapter

2020-11-25 Thread Dominic Jäger
... instead of generic PVE admin guide. Signed-off-by: Dominic Jäger --- www/manager6/dc/Config.js | 1 + 1 file changed, 1 insertion(+) diff --git a/www/manager6/dc/Config.js b/www/manager6/dc/Config.js index 2fdba743..92b94afc 100644 --- a/www/manager6/dc/Config.js +++ b/www/manager6/dc/Confi

[pve-devel] applied: [PATCH manager 0/2] fix small glitch in OSD destruction due to tainting

2020-11-25 Thread Thomas Lamprecht
On 24.11.20 19:55, Stoiko Ivanov wrote: > Saw the forum thread's subject [0] and was reminded of the issue with the LIO > target provider recently [1]. > > Depending on feedback I might slowly be inclined to blindly untaint in > run_command again? > now that we cacthed most stuff already? ^^ If

Re: [pve-devel] [PATCH v2 manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Thomas Lamprecht
On 25.11.20 09:54, Wolfgang Bumiller wrote: > Signed-off-by: Wolfgang Bumiller > --- > Changes to v1: > * moved helper to PVE.Utils > * lint fixups > > www/manager6/Toolkit.js | 17 ++--- > www/manager6/Utils.js | 30 ++ > www/manager6/lxc/DNS.js | 2 +-

Re: [pve-devel] [PATCH v3 qemu-server 3/3] print query-proxmox-support result in 'full' status

2020-11-25 Thread Stefan Reiter
On 25/11/2020 10:09, Fabian Grünbichler wrote: this needs the following fix-up, else pvestatd spams the log for all not runnning VMs: diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 6966717..0103386 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -2750,6 +2750,8 @@ sub vmstatu

Re: [pve-devel] [PATCH docs] pvecm: explain role of ssh in PVE stack

2020-11-25 Thread Thomas Lamprecht
On 25.11.20 10:04, Fabian Grünbichler wrote: Note that there was a v2, I applied that and made some followups, you may want to checkout git for the current state, could just push a fixup if there's still an error or misleading part ___ pve-devel maili

Re: [pve-devel] [PATCH v3 qemu-server 3/3] print query-proxmox-support result in 'full' status

2020-11-25 Thread Fabian Grünbichler
this needs the following fix-up, else pvestatd spams the log for all not runnning VMs: diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 6966717..0103386 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -2750,6 +2750,8 @@ sub vmstatus { foreach my $vmid (keys %$list) {

Re: [pve-devel] [PATCH docs] pvecm: explain role of ssh in PVE stack

2020-11-25 Thread Fabian Grünbichler
On November 17, 2020 12:35 pm, Oguz Bektas wrote: > add a section describing how SSH tunnels are used in conjunction > with PVE. (for #2829) > > Signed-off-by: Oguz Bektas > --- > pvecm.adoc | 35 +++ > 1 file changed, 35 insertions(+) > > diff --git a/pvecm.adoc

[pve-devel] [PATCH V2 pve-network 1/4] controllers: improve bgp-evpn

2020-11-25 Thread Alexandre Derumier
- add new bgp plugin - add ebgp support - add loopback support - move gateway-nodes option to zone as 'exitnodes' - move external-peers to bgp plugin --- PVE/API2/Network/SDN/Controllers.pm | 1 + PVE/Network/SDN/Controllers.pm | 12 +- PVE/Network/SDN/Controllers/BgpPlugin

[pve-devel] [PATCH V2 pve-network 2/4] zones: evpn : add support for loopback

2020-11-25 Thread Alexandre Derumier
--- PVE/Network/SDN/Zones.pm | 2 +- PVE/Network/SDN/Zones/EvpnPlugin.pm | 7 +++ PVE/Network/SDN/Zones/Plugin.pm | 2 +- PVE/Network/SDN/Zones/QinQPlugin.pm | 2 +- PVE/Network/SDN/Zones/SimplePlugin.pm | 2 +- PVE/Network/SDN/Zones/VlanPlugin.pm | 2 +- PVE/Network/S

[pve-devel] [PATCH V2 pve-network 3/4] update test documentation

2020-11-25 Thread Alexandre Derumier
--- test/documentation.txt | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/test/documentation.txt b/test/documentation.txt index 7886966..6ee8ee6 100644 --- a/test/documentation.txt +++ b/test/documentation.txt @@ -12,14 +12,18 @@ pvesh create /cluster/sdn/vnets/

[pve-devel] [PATCH V2 pve-network 4/4] sdn: fix : pending parser

2020-11-25 Thread Alexandre Derumier
--- PVE/API2/Network/SDN/Zones.pm | 12 +++-- PVE/Network/SDN.pm | 45 ++--- PVE/Network/SDN/Zones/Plugin.pm | 28 3 files changed, 52 insertions(+), 33 deletions(-) diff --git a/PVE/API2/Network/SDN/Zones.pm b/PVE/API2/Network/S

[pve-devel] [PATCH V2 pve-network 0/4] add ebgp-evpn support

2020-11-25 Thread Alexandre Derumier
This add support for a new controller plugin "bgp", to manage specific bgp section by host. This is allowing ebgp (I'll send doc soon) Also some fixes on pending parser Changelog V2: - fix some bug in bgp plugin Alexandre Derumier (4): controllers: improve bgp-evpn zones: evpn : add suppor

[pve-devel] [PATCH v2 manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Wolfgang Bumiller
Signed-off-by: Wolfgang Bumiller --- Changes to v1: * moved helper to PVE.Utils * lint fixups www/manager6/Toolkit.js | 17 ++--- www/manager6/Utils.js | 30 ++ www/manager6/lxc/DNS.js | 2 +- 3 files changed, 33 insertions(+), 16 deletions(-) diff --g

Re: [pve-devel] [PATCH manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Wolfgang Bumiller
> On 11/25/2020 9:31 AM Dominik Csapak wrote: > > > hi, > > please use 'eslint' to check new javascript code, it > would have caught some things ;) thought we'd be running that at build time, oops > (though most things were just moved...) > > comments inline > > On 11/25/20 9:16 AM, Wolfga

Re: [pve-devel] [PATCH manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Dominik Csapak
hi, please use 'eslint' to check new javascript code, it would have caught some things ;) (though most things were just moved...) comments inline On 11/25/20 9:16 AM, Wolfgang Bumiller wrote: Signed-off-by: Wolfgang Bumiller --- patch generated with --histogram diff algorithm www/manager6/T

[pve-devel] [PATCH manager] ad #3140: allow interface suffix in dns entries

2020-11-25 Thread Wolfgang Bumiller
Signed-off-by: Wolfgang Bumiller --- patch generated with --histogram diff algorithm www/manager6/Toolkit.js | 47 - www/manager6/lxc/DNS.js | 2 +- 2 files changed, 33 insertions(+), 16 deletions(-) diff --git a/www/manager6/Toolkit.js b/www/manager6/Too