sonic-utilities icon indicating copy to clipboard operation
sonic-utilities copied to clipboard

add neigh-suppress CLI command

Open iris00522 opened this issue 3 years ago • 4 comments

What I did

add neigh-suppress CLI command

How I did it

reference evpn neigh-suppress HLD https://github.com/Azure/SONiC/blob/master/doc/vxlan/EVPN/EVPN_VXLAN_HLD.md

5. ARP suppression
   - config neigh-suppress vlan <vlan-id>  <"on"/"off">
   - vlanid represents the vlan_id and on/off is enabled or disabled. By default ARP/ND suppression is disabled.

add neigh-suppress CLI command

How to verify it

root@as7726-32x-3:/home/admin#  config neigh-suppress vlan 1000
Usage: config neigh-suppress vlan [OPTIONS] <vid> <on|off>
Try "config neigh-suppress vlan -h" for help.

Error: Missing argument "<on|off>".  Choose from:
        on,
        off.
root@as7726-32x-3:/home/admin#  config neigh-suppress vlan 1000 on
Vlan1000 doesn't exist
root@as7726-32x-3:/home/admin#  config vlan add 1000
root@as7726-32x-3:/home/admin#  config neigh-suppress vlan 1000 on
root@as7726-32x-3:/home/admin#  config neigh-suppress vlan 1000 off
root@as7726-32x-3:/home/admin#  config neigh-suppress vlan 1000 on
root@as7726-32x-3:/home/admin# redis-cli
127.0.0.1:6379> select 4
OK
127.0.0.1:6379[4]> keys *SUPPRESS_VLAN_NEIGH*
1) "SUPPRESS_VLAN_NEIGH|Vlan1000"
127.0.0.1:6379[4]> HGETALL "SUPPRESS_VLAN_NEIGH|Vlan1000"
1) "suppress"
2) "on"
127.0.0.1:6379[4]>

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)

iris00522 avatar Dec 15 '21 09:12 iris00522

This pull request introduces 1 alert when merging 119aaeacf3af4fbfe1edb6bc85338c2a2eaf1343 into 96143eef7424f5ce65ff3d6bea8709b3147fe449 - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

lgtm-com[bot] avatar Dec 15 '21 09:12 lgtm-com[bot]

/azpw run

iris00522 avatar Dec 15 '21 13:12 iris00522

/AzurePipelines run

mssonicbld avatar Dec 15 '21 13:12 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Dec 15 '21 13:12 azure-pipelines[bot]