-
Notifications
You must be signed in to change notification settings - Fork 543
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in 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