Skip to content

Append default mask field if user has not provided one#972

Merged
prsunny merged 2 commits intosonic-net:masterfrom
prsunny:pfx_filter
Jul 1, 2020
Merged

Append default mask field if user has not provided one#972
prsunny merged 2 commits intosonic-net:masterfrom
prsunny:pfx_filter

Conversation

@prsunny
Copy link
Copy Markdown
Contributor

@prsunny prsunny commented Jun 30, 2020

- What I did
If user configures IP (v4 or v6) address without a mask, append the default /32 or /128 mask

- How I did it

- How to verify it
Redis command

config interface ip add Vlan1000 2001::1
config interface ip add Vlan1000 1.1.1.3
127.0.0.1:6379[4]> KEYS *VLAN_INTERFACE*
1) "VLAN_INTERFACE|Vlan1000|2001::1/128"
2) "VLAN_INTERFACE|Vlan1000|1.1.1.3/32"

- Previous command output (if the output of a command-line utility has changed)

- New command output (if the output of a command-line utility has changed)

Copy link
Copy Markdown
Contributor

@pavel-shirshov pavel-shirshov left a comment

Choose a reason for hiding this comment

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

As the comment

@prsunny prsunny merged commit fd52e93 into sonic-net:master Jul 1, 2020
@prsunny prsunny deleted the pfx_filter branch July 1, 2020 00:31
abdosi pushed a commit to abdosi/sonic-utilities that referenced this pull request Aug 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants