-
Notifications
You must be signed in to change notification settings - Fork 389
T7485: ia-na ipv6 for PPPoE link #4693
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: current
Are you sure you want to change the base?
Conversation
❌ |
CI integration 👍 passed! Details
|
send client-id {{ dhcpv6_options.duid }}; | ||
{% endif %} | ||
{% if address is vyos_defined and 'dhcpv6' in address %} | ||
{% if (address is vyos_defined and 'dhcpv6' in address) or (ipv6 is vyos_defined and ipv6.address is vyos_defined and 'dhcpv6' in ipv6.address) %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please try if it is possible to drop the entire line 8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-
(address is vyos_defined and 'dhcpv6' in address)
might be removed because the user explicitly expects to have an ipv6 address provided by dhcpv6 for common interfaces. -
the removal of
(ipv6 is vyos_defined and ipv6.address is vyos_defined and 'dhcpv6' in ipv6.address)
will always provide an IA_NA address to the user when dhcpv6-pd is setup on a pppoe interface: leaving this part ensure the user have the choice to request an ipv6 address for the pppoe interface with or without dhcpv6-pd and vice-versa.
I very much prefer to leave this line (and line 29) as it is proposed on commit e0cefe0, because it is specifically handling these scenarios:
- requesting explicitly an ipv6 via dhcpv6 on a common interface (not pppoe) via
set interface type name address dhcpv6
- requesting explicitly an ipv6 via dhcpv6 on a pppoe interface independently of any dhcpv6-pd configuration via
set interface pppoe pppoeN ipv6 address dhcpv6
- requesting explicitly dhcpv6-pd on a pppoe interface without requesting a dedicated ipv6 for this pppoe interface.
Note: set interface type name dhcpv6-options temporary
is not explicit and might be confusing for pppoe's case, in my opinion.
@c-po what do you think about it?
Change summary
If a pppoe interface define
ipv6 address autoconf
, this interface doesn't register the provided IA-NA address from RA's/DHPCv6 announces (dhcp6c).By modifying the dhcp6c configuration template and handling pppoe interfaces edge cases, the right ia-na entry can be written and handled by dhcp6c to the pppoe interface.
See : https://vyos.dev/T7485#234216
NOTE
It doesn't handle requesting a specific IA-NA address (
send ia_na
is always defaulting to0
).Is a new parameter in
interfaces pppoe [pppoeN] ipv6
xml definition should be handled has well?Types of changes
Related Task(s)
https://vyos.dev/T7485
Related PR(s)
How to test / Smoketest result
Checklist: