[PATCH manager] ui: fabrics: skip unsupported ip6 interface properties on submit

[PATCH manager] ui: fabrics: skip unsupported ip6 interface properties on submit

[Lukas Sichert]

The interface grid keeps the node interface data in the record even if
BGP disables the IPv6 column via 'hasIpv6Support=false'. As getValue()
serializes the full record data, BGP interfaces could still submit an
'ip6' property. This results in the error 'format error
interfaces[0].ip6: property is not defined in schema'.

Skip 'ip6' during serialization when IPv6 support is disabled for the
panel.

Signed-off-by: Lukas Sichert <l.sichert@proxmox.com>
---
 www/manager6/sdn/fabrics/InterfacePanel.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/www/manager6/sdn/fabrics/InterfacePanel.js b/www/manager6/sdn/fabrics/InterfacePanel.js
index f07e7859..74781267 100644
--- a/www/manager6/sdn/fabrics/InterfacePanel.js
+++ b/www/manager6/sdn/fabrics/InterfacePanel.js
@@ -174,7 +174,10 @@ Ext.define('PVE.sdn.Fabric.InterfacePanel', {
                     continue;
                 }
 
-                if (['type', 'isDisabled'].includes(key)) {
+                if (
+                    ['type', 'isDisabled'].includes(key) ||
+                    (key === 'ip6' && !me.hasIpv6Support)
+                ) {
                     continue;
                 }
 
-- 
2.47.3

Re: [PATCH manager] ui: fabrics: skip unsupported ip6 interface properties on submit

[Hannes Laimer]

On 2026-06-03 14:49, Lukas Sichert wrote:
> The interface grid keeps the node interface data in the record even if
> BGP disables the IPv6 column via 'hasIpv6Support=false'. As getValue()
> serializes the full record data, BGP interfaces could still submit an
> 'ip6' property. This results in the error 'format error
> interfaces[0].ip6: property is not defined in schema'.
> 
> Skip 'ip6' during serialization when IPv6 support is disabled for the
> panel.
> 

the same happens if a IPv4 address is configured for the interface
(in `/etc/network/interfaces`)

we should probably add `hasIpv4Support` to the InterfacePanel, like we
already have it for FabricEdit. then we can also handle the ipv6 col the
same way we do the ipv4 col.. and drop the `initComponent`-thingy in
`bgp/InterfacePanel.js`

> Signed-off-by: Lukas Sichert <l.sichert@proxmox.com>
> ---
>  www/manager6/sdn/fabrics/InterfacePanel.js | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/www/manager6/sdn/fabrics/InterfacePanel.js b/www/manager6/sdn/fabrics/InterfacePanel.js
> index f07e7859..74781267 100644
> --- a/www/manager6/sdn/fabrics/InterfacePanel.js
> +++ b/www/manager6/sdn/fabrics/InterfacePanel.js
> @@ -174,7 +174,10 @@ Ext.define('PVE.sdn.Fabric.InterfacePanel', {
>                      continue;
>                  }
>  
> -                if (['type', 'isDisabled'].includes(key)) {
> +                if (
> +                    ['type', 'isDisabled'].includes(key) ||
> +                    (key === 'ip6' && !me.hasIpv6Support)
> +                ) {
>                      continue;
>                  }
>  

superseded: [PATCH manager] ui: fabrics: skip unsupported ip6 interface properties on submit

[Lukas Sichert]

superseded-by v2:
https://lore.proxmox.com/all/20260603155521.125175-1-l.sichert@proxmox.com/