Skip to content

Commit

Permalink
Sub port interface high level design (#420)
Browse files Browse the repository at this point in the history
* Initial sub port interface high level design

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Address comments

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Rename DOT1Q_INTERFACE table to SUB_INTERFACE table

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Make mtu not configurable in the current design

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Add more comments to using sub port as a bridge port, which is out of the scope of
this design

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Address comment

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Rename SUB_INTERFACE to VLAN_SUB_INTERFACE

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Clarify the use of iproute2

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Remove the need to create an object id for vlan

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Set admin status of a sub port interface

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Config ip address on a sub port interface using existing `config
interface ip` command

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Clarify the use of iproute2 package is internal to SONiC

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Add Appendix, difference between sub port interface and vlan interface

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Add miss policy as open questions

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* List the attributes that can be changed at rum time in SAI spec

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* No oper status in show command

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Update table of contents; add acknowledgment

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Add implementation reference

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Amend references

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Update sub port interface removal event flow

* Remove definite article

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Correct typo

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Update table of contents; update references

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Change wording

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Update usage examples

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Add CLI design for changing admin status

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Add an APPL_DB entry example for IPv6

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Add a STATE_DB entry example for IPv6

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Amend references

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Amend reference on add/del ip address CLI

Signed-off-by: Wenda Ni <wenni@microsoft.com>
  • Loading branch information
wendani authored and lguohan committed Oct 9, 2019
1 parent df91b66 commit 832fafd
Show file tree
Hide file tree
Showing 8 changed files with 596 additions and 0 deletions.
596 changes: 596 additions & 0 deletions doc/sonic-sub-port-intf-hld.md

Large diffs are not rendered by default.

Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added images/sub_interface_hld/sub_intf_rif.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added images/sub_interface_hld/vlan_intf_rif.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit 832fafd

Please sign in to comment.