Skip to content

T7973: do not disallow all-zero host part IPv6 host addresses#19

Merged
sever-sever merged 1 commit intovyos:currentfrom
dmbaturin:T7973-allow-all-zero-ipv6-hosts
Dec 5, 2025
Merged

T7973: do not disallow all-zero host part IPv6 host addresses#19
sever-sever merged 1 commit intovyos:currentfrom
dmbaturin:T7973-allow-all-zero-ipv6-hosts

Conversation

@dmbaturin
Copy link
Member

Change Summary

As discussed in the task, all-zero host part IPv6 addresses are valid unicast addresses and other network vendors including Cisco and Arista allow them, and so should we.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Other (please describe):

Related Task(s)

Related PR(s)

Proposed changes

How to test

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

Copy link
Contributor

@jestabro jestabro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Drop restriction: logic is clear; simple test passes; integration tests included.

since they are valid unicast addresses, unlike in IPv4
@dmbaturin dmbaturin force-pushed the T7973-allow-all-zero-ipv6-hosts branch from f6ff925 to 779b5be Compare December 5, 2025 15:10
Copy link
Member

@sever-sever sever-sever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Manually tested

vyos@r14# run show conf com | match dum
set interfaces dummy dum1 address '2001:db8::1/64'
set interfaces dummy dum5 address '2001:db8::/32'
set interfaces dummy dum10 address '2001:db8::/128'
[edit]
vyos@r14# 
[edit]
vyos@r14# run show interfaces 
Codes: S - State, L - Link, u - Up, D - Down, A - Admin Down
Interface    IP Address         MAC                VRF        MTU  S/L    Description
-----------  -----------------  -----------------  -------  -----  -----  -------------
dum1         2001:db8::1/64     b6:f1:ac:db:59:2a  default   1500  u/u
dum5         2001:db8::/32      82:ad:57:e0:0a:b8  default   1500  u/u
dum10        2001:db8::/128     62:b3:8e:5e:28:a1  default   1500  u/u

@sever-sever sever-sever merged commit e358b4e into vyos:current Dec 5, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants