RE: SONIC on Chassis Subgroup - Weekly PR Tracking [3/12/2021]
Anshu Verma
Hi All – Please find status of Chassis PRs below, action items highlighted in “Yellow”.
|
Feature |
PR |
Owner |
Reviewer - Status |
Action item |
|
PMON HLD |
Nokia |
@Judy Joseph - under review |
3/12: No new update. @Judy - could you please review and approve
3/5: No new update 2/24: No new update 2/18: No new update 2/8: No new update 1/27: @Judy Joseph, could you please review and approve |
|
|
Inband Interface (port type) Code PR |
https://github.com/Azure/sonic-swss/pull/1602 Ready for merge |
Veda (Nokia) |
@song - approved Ready for merge |
3/12: @Abhishek - please help with Merge.
3/5: PR, approved, Thanks Song! Ready for Merge. 2/24: @Abhishek to help with test rerun 2/18: Song (Arista) provided review comments, thanks 2/8: No new update 1/27: @eswaran, @Abhishek - could you please review and approve |
|
|
Veda (Nokia) |
@Abhishek pending @mlorrillere - approved
|
3/12: @Abhishek -thanks for test re-run. #159, #1363 dependent on this PR, please review and approve. Approved by Maxime (Arista), thanks!
3/5: No new update, : 2/24: @Abhishek to help with test rerun 2/18: No new update 2/8: No new update 1/27: @eswaran, @Abhishek - could you please review and approve |
|
|
|
https://github.com/Azure/sonic-platform-common/pull/159 Ready for merge |
Veda (Nokia) |
@ngoc-do - pending @Abhishek - approved Ready for merge |
3/12: @Ngoc - please re-approve. @Abhishek - please help with Merge. PR approved by Abhishek, thanks!
3/5: No new update, : @Abhishek please help with test rerun. PR was approved by Ngoc. 2/24: @Abhishek to help with test rerun 2/18: @Rita, @Guohan - could you please help with Merge. 2/8: PR approved by Ngoc (Arista), thanks! |
|
|
https://github.com/Azure/sonic-platform-daemons/pull/145 Ready for merge |
Veda (Nokia) |
@ngoc-do - pending @Abhishek - approved Ready for merge |
3/12: @Ngoc - please re-approve. @Abhishek - please help with Merge. PR approved by Abhishek, thanks!
3/5: No new update, : @Abhishek please help with test rerun. PR was approved by Ngoc. 2/24: @Abhishek to help with test rerun 2/18: @Rita, @Guohan - could you please help with Merge. 2/8: PR approved by Ngoc (Arista), thanks! Do we need MSFT approval? @Abhishek please confirm 1/27: @eswaran, @Abhishek - could you please review and approve |
|
|
https://github.com/Azure/sonic-utilities/pull/1363 Ready for merge |
Veda (Nokia) |
@ngoc-do - pending @Abhishek - approved Ready for merge |
3/12: @Ngoc - please re-approve. @Abhishek - please help with Merge. PR approved by Abhishek, thanks!
3/5: No new update, : @Abhishek please help with test rerun. PR was approved by Ngoc. 2/24: @Abhishek to help with test rerun 2/18: @Rita, @Guohan - could you please help with Merge. 2/8: PR approved by Ngoc (Arista), thanks! Do we need MSFT approval? @Abhishek please confirm 1/27: @eswaran, @Abhishek- could you please review and approve |
|
LAG Code PRs |
Veda (Nokia) |
@Judy Joseph – under review @Guohan Lu - pending |
3/12: Under review by Judy.
3/5: No new update, @Judy- could you please review and approve 2:24: No new update 2/18: No new update 2/8: PR under review and comments provided by Judy, thanks! 1/27: @Judy- could you please review and approve |
|
|
|
Veda (Nokia) |
@ysmanman- approved @Judy Joseph – under review |
3/12: Under review by Judy.
3/5: No new update, @Judy- could you please review and approve 2/24: PR approved by @ysmanman, thanks! 2/18: Veda added test case for lag id allocator table full error. 2/8: PR under review and comments provided by Judy and Song, thanks! 1/27: Song - could you please review & approve. |
|
|
Recycle port support HLD |
Song (Arista) |
@eswaran - pending @Rita – under review @Arvindsrinivasan - pending @Abhishek - change requested @Srikanth -approved |
3/12: Approved by Srikanth, thanks! Updates requested by Rita and Abhishek.
3/5: Review in progress by Rita. @eswaran, @Arvindsrinivasan - could you please review and approve. 2/24: No new update 2/18: No new update 2/8: @eswaran, @Rita Hui, @Arvindsrinivasan - could you please review and approve 1/27: Need Nokia & MSFT reviewers assigned |
|
|
Routing/BGP support for Chassis HLD |
jmmikkel (Arista)
|
@eswaran - approved @Rita - pending @Guohan Lu- approved @minionatwork - approved |
3/12: Approved by Eswaran, thanks! @Rita could you please approve and merge.
3/5: Approved by Guohan, Suresh Thanks! @eswaran, @Rita- could you please review and approve 2/24: No new update 2/18: No new update 2/8: No new update 1/27: @eswaran, @Rita Hui- could you please review and approve |
|
|
Routing/BGP support for Chassis Code PR |
https://github.com/Azure/sonic-buildimage/pull/5622
|
jmmikkel (Arista) |
@vganesan-nokia - approved @minionatwork - pending @rlhui - pending @Arvindsrinivasan - under review @eswaran - pending @venkat (Dell) - pending @Guohan Lu- pending
|
3/12: Feedback provided by Jmmikkel, @eswaran, @Rita Hui, @Duraisamy, Sureshkannan, @Arvindsrinivasan - could you please review and approve
3/5: Review in progress by Arvind. 2/24: PR approved by Veda, thanks! 2/18: Review comments provided by Veda (Nokia), thanks! 2/8: No new update 1/27: Veda, Suresh, @Rita Hui, @Arvindsrinivasan could you please review and approve |
|
|
jmmikkel (Arista) |
@Arvindsrinivasan - under review @Guohan Lu- pending |
3/12: @Jmmikkel - please fix conflicts
3/5: Review in progress by Arvind. 2/24: No new update 2/18: No new update 2/8: No new update 1/27: @Arvindsrinivasan - could you please review and approve |
|
|
Fabric port support Code PR |
Ngoc (Arista) |
@minionatwork - pending @rlhui – pending @eswaran - pending @Abhishek - change requested @Guohan Lu- change requested
|
3/12: Review in progress, feedback provided by Ngoc.
3/5: Review in progress by Abhishek 2/24: No new update 2/18: No new update 2/8:Vs test cases added, Suresh, @Rita, @Guohan, @Abhishek- could you please review and approve 1/27: Suresh, @Rita- could you please review and approve |
|
|
|
Ngoc (Arista) |
@Danny Allen – approved @Guohan Lu- pending @Joe LeVeque - approved @Srikanth -pending |
3/12: @Srikanth , @Guohan - please review and approve.
3/5: @Srikanth , @Guohan - please review and approve. 2/24: PR approved by Joe, thanks! 2/18: No new update 2/8: New PR, @Guohan, @Joe – could you please review and approve. |
|
|
Everflow support HLD |
Eswaran (Arista) |
@abdosi - change requested @minionatwork - approved |
3/12: Approved by Suresh, thanks! @Eswaran - please add the case where monitor port is the local port the behavior in both the scenarios.
3/5: Review in progress by Abhishek 2/24: No new update 2/18: No new update 2/8: No new update 1/27: @Abhishek- could you please review and approve |
|
|
Everflow support Code PR |
Song (Arista) |
@abdosi - pending @prsunny- pending @daall- pending @zhenggen-xu - pending @Guohan - under review |
3/12: No new update 3/5: No new update 2/24: No new update 2/18: Review comments provided by Guohan, thanks! 2/8: No new update 1/27: @Abhishek, @Prince, @Danny, @zhenggen-xu - could you please review and approve |
|
|
Inband Vlan |
Song (Arista) |
@Veda Ganesan – pending @Guohan - change requested |
3/12: @Guohan - could you please review and approve, @Veda Ganesan - please re-approve. 3/5: @Guohan - could you please review and approve. 2/24: PR approved by Veda, thanks! 2/18: Review comments provided by Veda, thanks! 2/8: New PR, @Veda Ganesan – could you please review and approve. |
|
|
Minigraph support |
Maxime (Arista) |
@Arvindsrinivasan – under review @Veda Ganesan – under review @Guohan - pending @Srikanth -pending |
3/12: No new update 3/5: No new update 2/24: Under review 2/18: No new update 2/8: New PR, under review by @Arvindsrinivasan , @Veda Ganesan |
|
|
|
Maxime (Arista) |
@Arvindsrinivasan – change requested @sandeep (Nokia) - pending @Veda Ganesan – under review |
3/12: Under review 3/5: Change requested by Arvind. @Malhotra, Sandeep (Nokia - US/Westford) - please review & approve. 2/24: No new update 2/18: No new update 2/8: New PR, under review by @Arvindsrinivasan , thanks! |
Code & HLD Merged PRs
|
Feature |
PR |
Owner |
Reviewer - Status |
Update |
|
Chassis sample test conversion to multi-DUT |
Shuba (Nokia) |
2/8: Merged, thanks Ying!
1/27: Need Arista & MSFT reviewers assigned @Eswaran Baskaran, @Ying Xie– could you please assign reviewers to the PR, thanks. |
||
|
LAG Code PR |
https://github.com/Azure/sonic-swss-common/pull/447 Merged |
Veda (Nokia) |
|
2/18: Merged, thanks Judy, Guohan! 2/8: PR under review and comments provided by Judy, thanks. 1/27: @Judy Joseph - could you please review and approve |
|
LAG Code PR |
https://github.com/Azure/sonic-swss-common/pull/448 Merged |
Veda (Nokia) |
|
2/18: Merged, thanks Judy! 2/8: PR approved. 1/27: @Judy Joseph - could you please review and approve |
|
Inband Interface support |
https://github.com/Azure/sonic-swss/pull/1568 Merged |
Veda (Nokia) |
3/5: Merged, thanks Prince! 2/24: Azure pipeline run requested by Prince and tests passed. 2/18: @Rita, @Guohan - could you please help with Merge. 2/8: PR approved, ready for merge. Thanks Prince! 1/27: @eswaran, @Abhishek- could you please review and approve |
|
|
Fabric port support for Chassis HLD |
https://github.com/Azure/SONiC/pull/668 Merged |
Eswaran (Arista) |
|
3/5: Approved and Merged, thanks Rita! 2/24: No new update 2/18: No new update 2/8: No new update 1/27: Suresh, @Rita, @Guohan, @Abhishek- could you please review and approve |
|
LAG Code PR |
https://github.com/Azure/sonic-swss/pull/1603 Merged |
Veda (Nokia) |
|
3/12: Merged, thanks Judy! 3/5: PR approved by Judy, thanks! Ready for Merge 2/24: Retest requested Judy 2/18: No new update 2/8: PR under review and comments provided by Judy, thanks! 1/27: @Judy Joseph - could you please review and approve |
Chassis Testing Effort:
|
Functional area |
Description |
PR# |
Owner |
Reviewed |
PR Status |
|
Adapt testcases for SONiC Chassis |
Chassis Test Infrastructure |
Nokia |
Yes |
Merged |
|
|
|
Implementation of multi-DUT and multi-ASIC as per PR 2347 |
Nokia |
Yes |
Merged |
|
|
|
[multi-dut] - Sanity checks for multi-duts |
Nokia |
Yes |
Merged |
|
|
|
Config_facts enhancements for multi-asic and support in SonicAsic |
Nokia |
Yes |
Merged |
|
|
|
Test bgp facts - skip for supervisor card in a SONiC chassis |
Nokia |
Yes |
Merged |
|
|
|
Adding module_utils to allow for common utilites that can be shared among ansible |
Nokia |
Yes |
Merged |
|
|
|
Support for not all DUT ports connected to a fanout switch |
Nokia |
Yes |
Merged |
|
|
|
Creategraph - Support comments and empty lines in the lab_devices and lab links csv files |
Nokia |
Yes |
Merged |
|
|
|
Introduce basic T2 topology |
Nokia |
Yes |
Merged |
|
|
|
Additional fixtures to filter DUTs as per specific test case needs |
Nokia |
Yes |
Merged |
|
|
|
Support for gen-mg to work with multi-dut where VMs connect to only a single host and supervisor card |
Nokia |
Yes |
Merged |
|
|
|
Optimizing dut selection fixtures in pytest_generate_tests |
Nokia |
Yes |
Merged |
|
|
|
Making log analyzer fixture work with multiple DUTs in multi-dut testbed |
Nokia |
Yes |
Merged |
|
|
VOQ test plan |
VOQ test plan |
Nokia |
Yes |
Merged |
|
|
|
System Initialization & Neighbor Lifecycle functional |
Nokia |
Yes |
Merged |
|
|
|
Neighbor & Router Interface lifecycle events |
|
Nokia |
|
|
|
|
Neighbor and Host IP Forwarding testcases and helpers |
Nokia |
|
3/5: please assign reviewers @Ying, @Eswaran Baskaran |
|
|
PMON Test plan |
PMON Test plan |
Nokia |
Yes |
Merged (Thanks Xin, Ying!) |
|
|
|
Framework for selecting pluggable components for sonic modular chassis, and CLI Tests |
Nokia |
Yes |
Merged |
|
|
|
New tests for APIs added for Chassis and convert existing APIs to run on Chassis |
Nokia |
|
3/12: Comments addressed, please help with approval and Merge @Joe 3/5: Under review (Joe, Sujin, Vaibhav, Ying) |
|
|
|
Thermal CLI test case |
|
Nokia |
|
Need json file for sonic buildimage |
|
LAG Test plan |
LAG on Chassis Test case |
Nokia |
|
3/12: Need reviewers from Arista team @Eswaran Baskaran 3/5: Assigned to Ying, Xin. Need reviewers from Arista team @Eswaran Baskaran |
|
|
BGP on Chassis Test plan |
|
|
Arista |
|
3/12: No new update 3/5: Need ETA from Arista team @Eswaran Baskaran |
|
Fabric Test plan |
|
|
Arista |
|
3/12: No new update 3/5: Need ETA from Arista team @Eswaran Baskaran |
|
Everflow support Test plan |
|
|
Arista |
|
3/12: No new update 3/5: Need ETA from Arista team @Eswaran Baskaran |
|
Minigraph support in sonic-mgmt |
|
|
Arista |
|
3/12: No new update 3/5: Need ETA from Arista team @Eswaran Baskaran |
|
T1 test case conversion |
test_interfaces.py, test_nbr_health.py, iface_namingmode and new DUT selection mechanism, ARP, Route, Syslog, Monit, read_mac, TechSupport, PortStat |
Nokia |
Yes |
Merged |
|
|
|
LLDP, SNMP, PortChannel |
Nokia |
|
|
|
|
|
FIB |
Nokia |
Yes |
Merged (Thanks Joe, Ying! ) |
|
|
|
IpFwd |
Nokia |
|
3/12: Closed, new PR will be created 3/5: Under review by Xin, thanks! |
|
|
|
Drop Packets |
MSFT |
|
Merged |
|
|
|
System Health, Syslog |
|
|
|
|
|
|
TACACS |
|
Nokia |
|
|
|
|
Auto_restart, CRM |
|
Nokia |
|
|
|
|
Added support to get random asic in a testcase using 'enum_rand_one_asic_index' |
Nokia |
|
Merged |
|
|
|
Announce_routes for T2 |
|
|
3/12: @Ying, @Xin, @Rita - Please review |
|
|
|
Thermal CLI test case, test_cpu_memory_usage.py, test_port_toggle.py |
|
|
3/12: Comments addressed, please help with Merge @Joe |
Thanks,
Anshu
From: Anshu Verma
Sent: Friday, March 5, 2021 4:17 PM
To: Guohan Lu <gu...@microsoft.com>; Rita Hui <Rita...@microsoft.com>; Arvindsrinivasan Lakshmi Narasimhan <Arvindsriniv...@microsoft.com>; Abhishek Dosi <Abhish...@microsoft.com>; Prince Sunny <Prince...@microsoft.com>; Eswaran Baskaran
<esw...@arista.com>; Ying Xie <Ying...@microsoft.com>; sonic-chass...@googlegroups.com; Eswaran Baskaran <esw...@arista.com>; Duraisamy, Sureshkannan (Nokia - US/Mountain View) <sureshkanna...@nokia.com>; Ganesan, Vedavinayagam (Nokia -
US/Westford) <vedavinaya...@nokia.com>; Keesara, Srikanth (Nokia - US/Westford) <srikanth...@nokia.com>; Malhotra, Sandeep (Nokia - US/Westford) <sandeep....@nokia.com>; Judy Joseph <Judy....@microsoft.com>; Xin Wang <xiw...@microsoft.com>
Subject: SONIC on Chassis Subgroup - Weekly PR Tracking [3/5/2021]
Hi Team – Please find status of Chassis PRs below, action items highlighted in “Yellow”.
Goal: Team we are targeting 100% approve and merge all PRs by 3/30, appreciate your support for reaching the milestone.
Summary:
- As of 3/5, we are at 68% PR merge completion.
- Total Merged PRs (incl. HLDs, Code, Test, HLDs) = 52
- Open PRs Total = 24
- PRs ready for Merge = 5
- PRs under review this week = 18
- PRs pending review= 1
|
Feature |
PR |
Owner |
Reviewer - Status |
Action item |
|
PMON HLD |
Nokia |
@Judy Joseph - under review |
3/5: No new update. @Judy - could you please review and approve
2/24: No new update 2/18: No new update 2/8: No new update 1/27: @Judy Joseph, could you please review and approve |
|
|
Inband Interface (port type) Code PR |
https://github.com/Azure/sonic-swss/pull/1602 Ready for merge |
Veda (Nokia) |
@song - approved Ready for merge |
3/5: PR, approved, Thanks Song! Ready for Merge.
2/24: @Abhishek to help with test rerun 2/18: Song (Arista) provided review comments, thanks 2/8: No new update 1/27: @eswaran, @Abhishek - could you please review and approve |
|
|
Veda (Nokia) |
@eswaran - pending @abdosi - pending |
3/5: No new update, : @Abhishek please help with test rerun.
2/24: @Abhishek to help with test rerun 2/18: No new update 2/8: No new update 1/27: @eswaran, @Abhishek - could you please review and approve |
|
|
|
https://github.com/Azure/sonic-platform-common/pull/159 Ready for merge |
Veda (Nokia) |
@ngoc-do - approved @Abhishek - pending Ready for merge |
3/5: No new update, : @Abhishek please help with test rerun. PR was approved by Ngoc.
2/24: @Abhishek to help with test rerun 2/18: @Rita, @Guohan - could you please help with Merge. 2/8: PR approved by Ngoc (Arista), thanks! Do we need MSFT approval? @Abhishek please confirm 1/27: @eswaran, @Abhishek - could you please review and approve |
|
|
https://github.com/Azure/sonic-platform-daemons/pull/145 Ready for merge |
Veda (Nokia) |
@ngoc-do - approved @Abhishek - pending Ready for merge |
3/5: No new update, : @Abhishek please help with test rerun. PR was approved by Ngoc.
2/24: @Abhishek to help with test rerun 2/18: @Rita, @Guohan - could you please help with Merge. 2/8: PR approved by Ngoc (Arista), thanks! Do we need MSFT approval? @Abhishek please confirm 1/27: @eswaran, @Abhishek - could you please review and approve |
|
|
https://github.com/Azure/sonic-utilities/pull/1363 Ready for merge |
Veda (Nokia) |
@ngoc-do - approved @Abhishek - pending Ready for merge |
3/5: No new update, : @Abhishek please help with test rerun. PR was approved by Ngoc.
2/24: @Abhishek to help with test rerun 2/18: @Rita, @Guohan - could you please help with Merge. 2/8: PR approved by Ngoc (Arista), thanks! Do we need MSFT approval? @Abhishek please confirm 1/27: @eswaran, @Abhishek- could you please review and approve |
|
LAG Code PR |
https://github.com/Azure/sonic-swss/pull/1603 Ready for merge |
Veda (Nokia) |
@Judy Joseph – approved Ready for merge |
3/5: PR approved by Judy, thanks! Ready for Merge
2/24: Retest requested Judy 2/18: No new update 2/8: PR under review and comments provided by Judy, thanks! 1/27: @Judy Joseph - could you please review and approve |
|
|
Veda (Nokia) |
@Judy Joseph – under review @Guohan Lu - pending |
3/5: No new update, @Judy- could you please review and approve 2:24: No new update 2/18: No new update 2/8: PR under review and comments provided by Judy, thanks! 1/27: @Judy- could you please review and approve |
|
|
|
Veda (Nokia) |
@ysmanman- approved @Judy Joseph – under review |
3/5: No new update, @Judy- could you please review and approve 2/24: PR approved by @ysmanman, thanks! 2/18: Veda added test case for lag id allocator table full error. 2/8: PR under review and comments provided by Judy and Song, thanks! 1/27: Song - could you please review & approve. |
|
|
Recycle port support HLD |
Song (Arista) |
@eswaran - pending @Rita – under review @Arvindsrinivasan - pending @Srikanth -pending |
3/5: Review in progress by Rita. @eswaran, @Arvindsrinivasan - could you please review and approve.
2/24: No new update 2/18: No new update 2/8: @eswaran, @Rita Hui, @Arvindsrinivasan - could you please review and approve 1/27: Need Nokia & MSFT reviewers assigned |
|
|
Routing/BGP support for Chassis HLD |
jmmikkel (Arista)
|
@eswaran - pending @Rita - pending @Guohan Lu- approved @minionatwork - approved |
3/5: Approved by Guohan, Suresh Thanks! @eswaran, @Rita Hui- could you please review and approve
2/24: No new update 2/18: No new update 2/8: No new update 1/27: @eswaran, @Rita Hui- could you please review and approve |
|
|
Routing/BGP support for Chassis Code PR |
https://github.com/Azure/sonic-buildimage/pull/5622
|
jmmikkel (Arista) |
@vganesan-nokia - approved @minionatwork - pending @rlhui - pending @Arvindsrinivasan - under review @eswaran - pending
|
3/5: Review in progress by Arvind.
2/24: PR approved by Veda, thanks! 2/18: Review comments provided by Veda (Nokia), thanks! 2/8: No new update 1/27: Veda, Suresh, @Rita Hui, @Arvindsrinivasan could you please review and approve |
|
|
jmmikkel (Arista) |
@Arvindsrinivasan - under review
|
3/5: Review in progress by Arvind. 2/24: No new update 2/18: No new update 2/8: No new update 1/27: @Arvindsrinivasan - could you please review and approve |
|
|
Fabric port support Code PR |
Ngoc (Arista) |
@minionatwork - pending @rlhui – pending @eswaran - pending @Abhishek - change requested @Guohan Lu- change requested
|
3/5: Review in progress by Abhishek 2/24: No new update 2/18: No new update 2/8:Vs test cases added, Suresh, @Rita, @Guohan, @Abhishek- could you please review and approve 1/27: Suresh, @Rita- could you please review and approve |
|
|
|
Ngoc (Arista) |
@Danny Allen – approved @Guohan Lu- pending @Joe LeVeque - approved @Srikanth -pending |
3/5: @Srikanth , @Guohan - please review and approve. 2/24: PR approved by Joe, thanks! 2/18: No new update 2/8: New PR, @Guohan, @Joe – could you please review and approve. |
|
|
Everflow support HLD |
Eswaran (Arista) |
@abdosi - under review @minionatwork - pending |
3/5: Review in progress by Abhishek 2/24: No new update 2/18: No new update 2/8: No new update 1/27: @Abhishek- could you please review and approve |
|
|
Everflow support Code PR |
Song (Arista) |
@abdosi - pending @prsunny- pending @daall- pending @zhenggen-xu - pending @Guohan - under review |
3/5: No new update 2/24: No new update 2/18: Review comments provided by Guohan, thanks! 2/8: No new update 1/27: @Abhishek, @Prince, @Danny, @zhenggen-xu - could you please review and approve |
|
|
Inband Vlan |
Song (Arista) |
@Veda Ganesan – approved @Guohan - under review |
3/5: @Guohan - could you please review and approve. 2/24: PR approved by Veda, thanks! 2/18: Review comments provided by Veda, thanks! 2/8: New PR, @Veda Ganesan – could you please review and approve. |
|
|
Minigraph support |
Maxime (Arista) |
@Arvindsrinivasan – under review @Veda Ganesan – under review @Guohan - pending @Srikanth -pending |
3/5: No new update 2/24: Under review 2/18: No new update 2/8: New PR, under review by @Arvindsrinivasan , @Veda Ganesan |
|
|
|
Maxime (Arista) |
@Arvindsrinivasan – change requested @sandeep (Nokia) - pending |
3/5: Change requested by Arvind. @Malhotra, Sandeep (Nokia - US/Westford) - please review & approve. 2/24: No new update 2/18: No new update 2/8: New PR, under review by @Arvindsrinivasan , thanks! |
Code & HLD Merged PRs
|
Feature |
PR |
Owner |
Reviewer - Status |
Update |
|
Chassis sample test conversion to multi-DUT |
Shuba (Nokia) |
@Ying Xie- approved and Merged |
2/8: Merged, thanks Ying!
1/27: Need Arista & MSFT reviewers assigned @Eswaran Baskaran, @Ying Xie– could you please assign reviewers to the PR, thanks. |
|
|
LAG Code PR |
https://github.com/Azure/sonic-swss-common/pull/447 Merged |
Veda (Nokia) |
|
2/18: Merged, thanks Judy, Guohan! 2/8: PR under review and comments provided by Judy, thanks. 1/27: @Judy Joseph - could you please review and approve |
|
LAG Code PR |
https://github.com/Azure/sonic-swss-common/pull/448 Merged |
Veda (Nokia) |
|
2/18: Merged, thanks Judy! 2/8: PR approved. 1/27: @Judy Joseph - could you please review and approve |
|
Inband Interface support |
https://github.com/Azure/sonic-swss/pull/1568 Merged |
Veda (Nokia) |
3/5: Merged, thanks Prince! 2/24: Azure pipeline run requested by Prince and tests passed. 2/18: @Rita, @Guohan - could you please help with Merge. 2/8: PR approved, ready for merge. Thanks Prince! 1/27: @eswaran, @Abhishek- could you please review and approve |
|
|
Fabric port support for Chassis HLD |
https://github.com/Azure/SONiC/pull/668 Merged |
Eswaran (Arista) |
|
3/5: Approved and Merged, thanks Rita! 2/24: No new update 2/18: No new update 2/8: No new update 1/27: Suresh, @Rita, @Guohan, @Abhishek- could you please review and approve |
Chassis Testing Effort:
|
Functional area |
Description |
PR# |
Owner |
Reviewed |
PR Status |
|
Adapt testcases for SONiC Chassis |
Chassis Test Infrastructure |
Nokia |
Yes |
Merged |
|
|
|
Implementation of multi-DUT and multi-ASIC as per PR 2347 |
Nokia |
Yes |
Merged |
|
|
|
[multi-dut] - Sanity checks for multi-duts |
Nokia |
Yes |
Merged |
|
|
|
Config_facts enhancements for multi-asic and support in SonicAsic |
Nokia |
Yes |
Merged |
|
|
|
Test bgp facts - skip for supervisor card in a SONiC chassis |
Nokia |
Yes |
Merged |
|
|
|
Adding module_utils to allow for common utilites that can be shared among ansible |
Nokia |
Yes |
Merged |
|
|
|
Support for not all DUT ports connected to a fanout switch |
Nokia |
Yes |
Merged |
|
|
|
Creategraph - Support comments and empty lines in the lab_devices and lab links csv files |
Nokia |
Yes |
Merged |
|
|
|
Introduce basic T2 topology |
Nokia |
Yes |
Merged |
|
|
|
Additional fixtures to filter DUTs as per specific test case needs |
Nokia |
Yes |
Merged |
|
|
|
Support for gen-mg to work with multi-dut where VMs connect to only a single host and supervisor card |
Nokia |
Yes |
Merged |
|
|
|
Optimizing dut selection fixtures in pytest_generate_tests |
Nokia |
Yes |
Merged |
|
|
|
Making log analyzer fixture work with multiple DUTs in multi-dut testbed |
Nokia |
Yes |
Merged |
|
|
VOQ test plan |
VOQ test plan |
Nokia |
Yes |
Merged |
|
|
|
System Initialization & Neighbor Lifecycle functional |
Nokia |
Yes |
Merged |
|
|
|
Neighbor & Router Interface lifecycle events |
|
Nokia |
|
|
|
|
Neighbor and Host IP Forwarding testcases and helpers |
Nokia |
|
3/5: please assign reviewers @Ying, @Eswaran Baskaran |
|
|
PMON Test plan |
PMON Test plan |
Nokia |
Yes |
Merged (Thanks Xin, Ying!) |
|
|
|
Framework for selecting pluggable components for sonic modular chassis, and CLI Tests |
Nokia |
Yes |
Merged |
|
|
|
New tests for APIs added for Chassis and convert existing APIs to run on Chassis |
Nokia |
|
3/5: Under review (Joe, Sujin, Vaibhav, Ying) |
|
|
|
Thermal CLI test case |
|
Nokia |
|
Need json file for sonic buildimage |
|
LAG Test plan |
LAG on Chassis Test case |
Nokia |
|
3/5: Assigned to Ying, Xin. Need reviewers from Arista team @Eswaran Baskaran |
|
|
BGP on Chassis Test plan |
|
|
Arista |
|
3/5: Need ETA from Arista team @Eswaran Baskaran |
|
Fabric Test plan |
|
|
Arista |
|
3/5: Need ETA from Arista team @Eswaran Baskaran |
|
Everflow support Test plan |
|
|
Arista |
|
3/5: Need ETA from Arista team @Eswaran Baskaran |
|
Minigraph support in sonic-mgmt |
|
|
Arista |
|
3/5: Need ETA from Arista team @Eswaran Baskaran |
|
T1 test case conversion |
test_interfaces.py, test_nbr_health.py, iface_namingmode and new DUT selection mechanism, ARP, Route, Syslog, Monit, read_mac, TechSupport, PortStat |
Nokia |
Yes |
Merged |
|
|
|
LLDP |
|
Nokia |
|
|
|
|
FIB |
Nokia |
Yes |
Merged (Thanks Joe, Ying! ) |
|
|
|
IpFwd |
Nokia |
|
3/5: Under review by Xin, thanks! |
|
|
|
Drop Packets |
MSFT |
|
Merged |
|
|
|
System Health, Syslog |
|
|
|
|
|
|
SNMP, TACACS |
|
Nokia |
|
|
|
|
Auto_restart, CRM |
|
Nokia |
|
|
|
|
Platform tests - Legacy CLI and API |
|
Nokia |
|
|
|
|
Added support to get random asic in a testcase using 'enum_rand_one_asic_index' |
Nokia |
|
Merged |
Thank You,
Anshu