with first patch correctly attributed to me, since I wrote the change ;)
and fixed up the first warn to also be a log_warn.

also added two followups, one for the error message, the other for style
of the second patch - please shout if you notice something off with
either of those!

On June 23, 2023 2:49 pm, Alexandre Derumier wrote:
> - add a check at config generation
> - display specific message on status if vnet is not generated
> 
> 
> Alexandre Derumier (2):
>   sdn: config generation : check if interfaces.d/sdn is sourced
>   zones: status: display specific message if vnet is not generated.
> 
>  src/PVE/Network/SDN.pm                    | 11 ++++++++++-
>  src/PVE/Network/SDN/Zones/Plugin.pm       | 18 +++++++++++++-----
>  src/PVE/Network/SDN/Zones/QinQPlugin.pm   |  8 +-------
>  src/PVE/Network/SDN/Zones/SimplePlugin.pm | 17 -----------------
>  src/PVE/Network/SDN/Zones/VlanPlugin.pm   |  8 +-------
>  5 files changed, 25 insertions(+), 37 deletions(-)
> 
> -- 
> 2.39.2
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel@lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 
> 
> 


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to