SONiC Chassis Subgroup PR Status [7/21/2021]

[Anshu Verma]

Hi Team – please find Chassis subgroup status below:

 Chassis Code PRs:

Feature	PR	Owner	Reviewer  - Status	Action item
Bootstrap service for pluggable fabric card on VOQ chassis	https://github.com/Azure/sonic-buildimage/pull/7477 Ready for Merge	Manju (Nokia)	@Judy- approved @Suvarna- pending @Abhishek - approved @shyam - pending @anand- pending @ngoc-do - pending	7/21: Approved by Abhishek, Judy thanks! Waiting for Cisco/Arista team approval 7/14: Abhishek to review and approve 7/7: Judy provided review feedback, Abhishek to review and approve 6/30: Judy reviewing PR, to be closed asap
Inband Vlan Code PR	https://github.com/Azure/sonic-swss/pull/1555 On hold	Song (Arista)	@Veda Ganesan – approved @Guohan - change requested @Abhishek  - change requested @Prince - Change requested	6/30: PR on hold 6/23: PR on hold
CLI Commands enhancement (VoQ LAG HLD)	https://github.com/Azure/SONiC/pull/802	Veda (Nokia)	Arvind - under review Song - approved	7/21: Veda -please address review comments, 7/14: Arvind to review and approve, comments addressed 7/7: Veda to address review comments address. Approved by Song 6/30: New PR, Arvind to review
CLI Commands enhancement (VoQ Code PR)	https://github.com/Azure/sonic-utilities/pull/1689	Veda (Nokia)	Arvind - under review Judy - under review Song - under review	7/21 Veda - please add unit test for single asic LC and confirm if changes are good for single asic LC 7/14: Arvind to review and approve, comments addressed 7/7: Veda to address review comments address. Song please review, thanks. 6/30: New PR, Arvind/Judy to review
Minigraph fix for inband interface admin_status missing issue	https://github.com/Azure/sonic-buildimage/pull/8050	Veda (Nokia)	Arvind - under review Abhishek- under review Judy - under review Maxime - approved	7/21: Veda - please add unit test for this PR 7/14: Veda - please add unit test for this PR 7/7: Reviewers assigned
Linecard CLI attempting to connect to chassis databases	https://github.com/Azure/sonic-buildimage/pull/8065 https://github.com/Azure/sonic-utilities/pull/1707	Veda (Nokia)	Arvind - under review Judy - under review Maxime - approved	7/21: Veda - please address review comments, PR#1707 on hold to resolve mid-plane communication. 7/14: Under discussion 7/7: Reviewers assigned
Same iBGP router id in all asics issue fix	https://github.com/Azure/sonic-buildimage/pull/8049	Veda (Nokia)	Arvind - under review Judy - under review Abhishek - under review	7/21: Veda - please add unit tests to the PR, Arvind please review. 7/14: Veda - please add unit tests to the PR, Arvind please review. 7/7: Reviewers assigned
PR for add the context_config.json as argument for syncd command	https://github.com/Azure/sonic-sairedis/pull/858	Marty (Nokia)	Kamil - approved	7/14: Ready for merge
[SWSS] VOQ Nexthop for remote VOQ LC should be created on inband OIF	https://github.com/Azure/sonic-swss/pull/1823	Suresh (Nokia)	Abhishek - under review	7/21: Suresh to add unit test for the PR
Fix for sonic-cfggen exception during platform string read	https://github.com/Azure/sonic-buildimage/pull/8229	Suresh (Nokia)	Arvind - under review	7/21: Suresh to address review comments

Chassis Testing Effort:

Functional area	Description	PR#	Owner	Reviewed	PR Status
BGP on Chassis Test plan	Placeholder		Arista		3/20: No new update 3/12: No new update 3/5: Need ETA from Arista team @Eswaran Baskaran
Fabric Test plan	Placeholder		Arista		3/20: No new update 3/12: No new update 3/5: Need ETA from Arista team @Eswaran Baskaran
Everflow support Test plan	Placeholder		Arista		3/20: No new update 3/12: No new update 3/5: Need ETA from Arista team @Eswaran Baskaran
	Gen-mg support for voq in multi-asic line cards		Nokia		7/7: Sandeep to share mini-graph PR 6/30: Sandeep to share by 7/10
VoQ Chassis	ARP: Modified '/arp/test_neighbor_mac_noptf.py' testcase to skip 'bgp shutdown all' for modular chassis	3788	Nokia		7/21: Under review by Abhishek
	[rsyslog] Fixing pretest and posttest checks for multiasic boards	3764	Nokia		7/21: Thomas to confirm db check, Abhishek provided comments
	Changes test_nbr_health test	3763	Nokia		7/21: assign to chassis review group
	Changing show chassis-module cli command as per PR 805	3762	Nokia		7/21: Arvind to review
	Modified iface_namingmode testcases to get spine ports for T2 topology	3758	Nokia		7/21: assign to chassis review group
	Modified iface_namingmode testcases to get internal ports on voq system	3757	Nokia		7/21: assign to chassis review group
	Enhancements platform api tests and change in psustatus_json to use enum_supervisor_dut_hostname	3756	Nokia		7/21: Reviewed in subgroup, Guohan to re-assign
	Modified VOQ portchannel testcases to get asic with portchannels configured	3754	Nokia		7/21: Judy to review
	[voq] VoQ multiasic fixes and minigraph support	3752	Nokia		7/21: Arvind to review 7/14: Under review by Ying, Arvind to review
	Support for gen-mg for multi-asic linecards in VoQ chassis	3746	Nokia		7/21: Arvind to review 7/14: Arvind to review
	[arp] Allow use of discontiguous ports with ARP tests	3776	Nokia
	7/21: Sandeep to share sample inventory file for VOQ chassis tests
	Revert https://github.com/Azure/sonic-mgmt/pull/3643				7/21: Arvind to review and revert the PR.

  

 

 

Merged last week:

 

CLI Commands enhancement (VoQ HLD)	https://github.com/Azure/SONiC/pull/801 Merged	Veda (Nokia)	Arvind - under review Song - approved	7/20: Approved & merged, thanks Arvind! 7/14: Arvind to review and approve, comments addressed 7/7: Veda to address review comments address. Approved by Song 6/30: New PR, Arvind to review
CLI Commands enhancement (PMON HLD)	https://github.com/Azure/SONiC/pull/805 Merged	Veda (Nokia)	Arvind - approved Judy - under review Song - approved	7/20: Approved & merged, thanks Arvind! 7/14: Arvind to review and approve, comments addressed 7/7: Judy to review and approve, approved by Arvind & Song. 6/30: New PR, Arvind /Judy to review
Change the Opnensl service dependency for DNX module      Closed	https://github.com/Azure/saibcm-modules/pull/1	Marty (Nokia)	Judy - under review	7/20: Closed

 

Test PRs:

VoQ Chassis	Changing ipv6 loopback addresses on the linecards to be on different subnets and adding t2 as a supported topology in veos Merged	3797	Nokia		7/21: Merged, thanks Abhishek!
VOQ Chassis	VOQ multi-asic Merged	3463	Nokia		7/21: merged, thanks Arvind! 6/30: Arvind to review 6/23: Review comments addressed, Arvind to review and approve 6/18: Arvind provided review comments, Tcusto to address/add input 6/9: Arvind provided review comments, Tcusto to address/add input 6/2: Arvind please review and approve 5/26: Arvind please review and approve

 

Thank You,

Anshu

 

[Manjunath Prabhu]

Hi Anshu,

A few more bug fix PRs for modular chassis. Could you please add it to the list of items to track?

https://github.com/Azure/sonic-platform-daemons/pull/203 - Arista

https://github.com/Azure/sonic-platform-daemons/pull/206  - Nokia

https://github.com/Azure/sonic-utilities/pull/1656/ - Nokia

Thanks,

Manju

--
You received this message because you are subscribed to the Google Groups "sonic-chassis-subgroup" group.
To unsubscribe from this group and stop receiving emails from it, send an email to sonic-chassis-sub...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/sonic-chassis-subgroup/BYAPR21MB1335135042DE8E733AFC191FB1E39%40BYAPR21MB1335.namprd21.prod.outlook.com.