Skip to content

[Command-Reference] Add CLI docs for IP interface loopback action#2216

Closed
liorghub wants to merge 2 commits intosonic-net:masterfrom
liorghub:ip_interface_loopback_doc
Closed

[Command-Reference] Add CLI docs for IP interface loopback action#2216
liorghub wants to merge 2 commits intosonic-net:masterfrom
liorghub:ip_interface_loopback_doc

Conversation

@liorghub
Copy link
Contributor

@liorghub liorghub commented Jun 14, 2022

HLD: sonic-net/SONiC#1006

What I did

Add CLIs document for IP intefeature, CLI PR: #2192

How I did it

How to verify it

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)

Config command

config interface loopback-action <interface-name> <action>
config interface loopback-action Ethernet0 drop

Show command

root@sonic:~# show ip interfaces loopback-action
Interface     Action      
------------  ----------  
Ethernet232   drop
Vlan100       forward    

@liorghub liorghub force-pushed the ip_interface_loopback_doc branch from 52d235c to 4398db8 Compare June 14, 2022 13:17

- Example:
```
admin@sonic:~$ config interface loopback-action Ethernet0 drop
Copy link
Contributor

Choose a reason for hiding this comment

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

Since 'show' is part of 'ip' sub-command, should we have this as 'config interface ip loopback-action ...'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. I am closing his PR, those changes will be part of PR #2192

@liat-grozovik
Copy link
Collaborator

liat-grozovik commented Jun 28, 2022

@liorghub i suggest that the command reference will be update on the same PR of the code changes. So, lets close this one and add the .md file changes to #2216

@liorghub liorghub closed this Jun 29, 2022
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.

3 participants