Page MenuHomeVyOS Platform

Add support for global-parameters, shared-network-parameters, subnet-parameters and static-mapping-parameters in dhcpv6-server as already implemented in (v4) dhcp-server
Closed, WontfixPublicFEATURE REQUEST

Description

As mentioned in the title:
service dhcpv6-server shared-network-name [name] subnet [subnet] static-mapping [mapping-name] static-mapping-parameters
This would be very useful for e.g. setting custom client DNS options, etc.

Details

Difficulty level
Unknown (require assessment)
Version
-
Why the issue appeared?
Will be filled on close

Event Timeline

jjakob changed the task status from Open to In progress.Jun 16 2019, 5:55 PM
jjakob claimed this task.
jjakob renamed this task from Add support for static-mapping-parameters in dhcpv6-server as it's already implemented in (v4) dhcp-server to Add support for global-parameters, shared-network-parameters, subnet-parameters and static-mapping-parameters in dhcpv6-server as already implemented in (v4) dhcp-server.Jun 16 2019, 5:57 PM

Looking at the dhcp_server.py for reference, I see the *-parameters were considered to be deprecated:

# HACKS AND TRICKS
#
# check for 'raw' ISC DHCP parameters configured by users
# actually this is a bad idea in general to pass raw parameters
# from any user
#
# deprecate this and issue a warning like we do for DNS forwarding?
if conf.exists('shared-network-parameters'):
    config['network_parameters'] = conf.return_values('shared-network-parameters')

Now I'm not sure whether this would be a welcome addition to dhcpv6 server.
My particular use-case that I considered this for is assigning per-static-mapping DNS server config (manually migrating clients enrolled into a IPA domain means I have to have some clients pointed to old DNS servers and some to new, currently I get by with setting static v6 addresses on the clients)

syncer triaged this task as Wishlist priority.Aug 31 2019, 12:13 AM
syncer edited projects, added VyOS 1.3 Equuleus; removed VyOS 1.2 Crux.
Viacheslav added a subscriber: Viacheslav.

We agree not to use raw options anymore.
If some options are required, it should be a separate PR per configured option.