@dmbaturin @zsdc Perhaps we could discuss the possibility of changing the dhcp(v6) server to a better server (and what feature changes might exist), which I hope will address the older ISC's better support for interfaces such as PPPOE and the latest dhcp(v6) standard

DHCPd shold be vonsistent for both v4 and v6 - running different daemons is simply bad.

Currently, vyos-configd will explicitly report ConfigError to the console; however, non-fatal warnings or information will not be reported. An example of a non-fatal warning is:

SolarFlare modules present in the kernel for 1.3 and 1.4

vyos@vyos:~$ sudo modinfo sfc
filename:       /lib/modules/5.10.14-amd64-vyos/kernel/drivers/net/ethernet/sfc/sfc.ko
license:        GPL
description:    Solarflare network driver
author:         Solarflare Communications and Michael Brown <[email protected]>
alias:          pci:v00001924d00001B03sv*sd*bc*sc*i*
alias:          pci:v00001924d00000B03sv*sd*bc*sc*i*
alias:          pci:v00001924d00001A03sv*sd*bc*sc*i*
alias:          pci:v00001924d00000A03sv*sd*bc*sc*i*
alias:          pci:v00001924d00001923sv*sd*bc*sc*i*
alias:          pci:v00001924d00000923sv*sd*bc*sc*i*
alias:          pci:v00001924d00001903sv*sd*bc*sc*i*
alias:          pci:v00001924d00000903sv*sd*bc*sc*i*
alias:          pci:v00001924d00000813sv*sd*bc*sc*i*
alias:          pci:v00001924d00000803sv*sd*bc*sc*i*
alias:          pci:v000010EEd00001100sv*sd*bc*sc*i*
alias:          pci:v000010EEd00000100sv*sd*bc*sc*i*
depends:        mdio
retpoline:      Y
intree:         Y
name:           sfc
vermagic:       5.10.14-amd64-vyos SMP mod_unload modversions 
parm:           vf_max_tx_channels:Limit the number of TX channels VFs can use (uint)
parm:           max_vfs:Reduce the number of VFs initialized by the driver (int)
parm:           mcdi_logging_default:Enable MCDI logging on newly-probed functions (bool)
parm:           rx_refill_threshold:RX descriptor ring refill threshold (%) (uint)
parm:           irq_adapt_low_thresh:Threshold score for reducing IRQ moderation (uint)
parm:           irq_adapt_high_thresh:Threshold score for increasing IRQ moderation (uint)
parm:           interrupt_mode:Interrupt mode (0=>MSIX 1=>MSI 2=>legacy) (uint)
parm:           rss_cpus:Number of CPUs to use for Receive-Side Scaling (uint)
parm:           efx_separate_tx_channels:Use separate channels for TX and RX (bool)
parm:           phy_flash_cfg:Set PHYs into reflash mode initially (bool)
parm:           debug:Bitmapped debugging message enable value (uint)

Looks good on 1.4-rolling-202102162107 (including migration from self-built 1.2.0-rolling+202102162120).

The translation is working properly now but it is not showing in the command output:

Maybe we should consider using a better proxy server to complete the dhcpv6 agent

Somehow this was lost in translation in my git repo....

Nackported to 1.3 equuleus

PR https://github.com/vyos/vyos-build/pull/146

Yes, I very much like this, and is what I am imaging with 3., above.

With this new information I see little to none reason to keep the key_mangling() workaround. If we manage to transform all nodes into "proper" syntax we can one day drop it.

PR https://github.com/vyos/vyatta-cfg-firewall/pull/20 is still pending to clean up the following log entry:

@c-po this is an excellent idea, and a useful reference for the bracket notation; this will need a bit of design of how best to slice the problem:

1. keeping key_mangling but excluding on tag nodes
2. removing key_mangling completely
3. removing on a case by case basis with the existing argument to get_config_dict()

In the new xml/py rewrite, do they disapear if you rerun the bgp configurator script? Eg. Can we execute frr-reload twice as a workaround?

This actually feels like an FRR bug as this still occurs with the new XML/Python rewrite

This is fixed in 1.4 as proper input validation happens.

Implemented/fixed for VyOS 1.4

Issue no longer persists in VyOS 1.4. Tested using: 1.4-rolling-20210214

Nice idea - it should be generated by https://github.com/vyos/vyos-build/blob/current/scripts/make-version-file

Created PR: https://github.com/vyos/vyos-1x/pull/731

Tested with:

Possible fix, pending testing: https://github.com/varesa/vyos-1x/commit/4a0504a96cf0f3078e964ed201f196fb55172e00