-
Notifications
You must be signed in to change notification settings - Fork 558
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Sub port interface implementation #969
Conversation
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Add function declaration Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
…face Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
…u change to sub interface Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
…terface rif to flex counter at creation Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
… specified Signed-off-by: Wenda Ni <wenni@microsoft.com>
retest this please |
please have high level description for this PR? what are changes you have made to help reviewer to understand this PR. |
dcf12d2
to
0ceb374
Compare
Signed-off-by: Wenda Ni <wenni@microsoft.com>
retest this please |
Signed-off-by: Wenda Ni <wenni@microsoft.com>
retest this please |
1 similar comment
retest this please |
Signed-off-by: Wenda Ni <wenni@microsoft.com>
retest this please |
2 similar comments
retest this please |
retest this please |
✔️ |
level Signed-off-by: Wenda Ni <wenni@microsoft.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check others' review comments
ad89e1a
to
90b12e1
Compare
Signed-off-by: Wenda Ni <wenni@microsoft.com>
90b12e1
to
c4bd373
Compare
sub interface ip addresses Signed-off-by: Wenda Ni <wenni@microsoft.com>
retest this please |
1 similar comment
retest this please |
Signed-off-by: Wenda Ni <wenni@microsoft.com>
retest this please |
* Suppport to verify aboot swi image for secure boot * Simplify the code * Fix not return value bug * Add m2crypto to setup.py * Change to only verify the image signed by a correct certificate
What I did
Implement HLD: sonic-net/SONiC#420
IntfMgrd:
Listen to CONFIG_DB VLAN_SUB_INTERFACE table.
Process host sub port interface: create, set (admin_status), and remove using
ip link
andip address
system call, and update the STATE_DBRelay the VLAN_SUB_INTERFACE table operation and field-value content to APPL_DB INTF_TABLE.
IntfOrch:
Listen to APPL_DB INTF_TABLE.
Process sub port interface on physical port: create (a Port object for sub port interface), set (admin_status), and remove.
PortsOrch:
Add member function to create a Port object for sub port interface
vs unit test:
sub port interface creation
sub port interface add IP addresses
sub port interface admin status change
sub port interface remove IP addresses
sub port interface removal
Why I did it
How I verified it
Tested on mlnx dut
Details if related
Depend on:
sonic-net/sonic-swss-common#284
Scenario 1:
Packets egress out of front-panel port from control plane Linux. What is received at ptf interface
tcpdump -ni eth16 -vxe
Scenario 2:
Send ARP request from ptf
What is received and replied at host sub port interface on DUT control plane Linux
sudo tcpdump -ni Ethernet64.10 -vxe
. Packets received and sent from sub port interface is stripped of 802.1q tag.Packets ptf sees
tcpdump -ni eth16 -vxe
. They are 802.1q tagged.vs test:
Test on mlnx dut
master d5af096
Before applying VLAN_SUB_INTERFACE config to CONFIG_DB
Apply VLAN_SUB_INTERFACE config to CONFIG_DB