sonic-buildimage
sonic-buildimage copied to clipboard
sonic-net
[Nokia-7220-H3] Add support for NOKIA 7220 IXR-H3 platform
Why I did it
Add support for NOKIA 7220 IXR-H3:
Platform: x86_64-nokia_ixr7220_h3-r0 HwSKU: Nokia-IXR7220-H3 ASIC: Broadcom Port Config: 32x400G + 2x10G
Work item tracking
- Microsoft ADO (number only):
How I did it
- Added module x86_64-nokia_ixr7220_h3-r0 ../device/nokia directory.
- Modified the platform/broadcom/one-image.mk to include NOKIA_IXR7220H3_PLATFORM_MODULE
- Modified the platform/broadcom/ platform-modules-nokia.mk to include NOKIA_IXR7220H3_PLATFORM_MODULE
- Add platform related directory ixr7220h3 in platform/Broadcom/sonic-platform-modules-nokia
- Modified related build scripts in platform/Broadcom/sonic-platform-modules-nokia/debian
How to verify it
- Make sure the sonic-buildimage is successful
- Run this image on x86_64-nokia_ixr7220_h3-r0 and verify all dockers are up and test basic commands like: o show version o show platform summary o show platform syseeprom o show platform fan o show platform psustatus o show platform firmware status o show platform temperature o sudo show system-health detail o show interface status
- Run OC test, for T0/T1 topology and have 100% pass on platform.
- Run this image on x86_64-nokia_ixr7250e_sup-r0 and x86_64-nokia_ixr7250e_36x400g-r0, make sure all functions run correctly.
Which release branch to backport (provide reason below if selected)
- [ ] 201811
- [ ] 201911
- [ ] 202006
- [ ] 202012
- [ ] 202106
- [ ] 202111
- [ ] 202205
- [ ] 202211
- [ ] 202305
Tested branch (Please provide the tested image version)
- [ ]
- [ ]
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)
@StormLiangMS,@liushilongbuaa PR: https://github.com/sonic-net/sonic-buildimage/pull/18497 is conflict with MS internal repo
Please push fix commit to sonicbld/precheck/head/18497 and approve
https://msazure.visualstudio.com/One/_git/Networking-acs-buildimage/pullrequest/9811366
After ms PR is merged, comment "/azpw ms_conflict" to rerun PR checker.
@StormLiangMS,@liushilongbuaa PR: #18497 is conflict with MS internal repoPlease push fix commit to sonicbld/precheck/head/18497 and approvehttps://msazure.visualstudio.com/One/_git/Networking-acs-buildimage/pullrequest/9811366After ms PR is merged, comment "/azpw ms_conflict" to rerun PR checker.
@liushilongbuaa @StormLiangMS Can you please take a look at this
@StormLiangMS,@liushilongbuaa PR: https://github.com/sonic-net/sonic-buildimage/pull/18497 is conflict with MS internal repo
Please push fix commit to sonicbld/precheck/head/18497 and approve
https://msazure.visualstudio.com/One/_git/Networking-acs-buildimage/pullrequest/9852302
After ms PR is merged, comment "/azpw ms_conflict" to rerun PR checker.
@liushilongbuaa @mssonicbld How can we help resolve this "internal conflict" issue and get this PR merged.
@liushilongbuaa @mssonicbld How can we help resolve this "internal conflict" issue and get this PR merged.
It is resolved last week.
@liushilongbuaa @mssonicbld How can we help resolve this "internal conflict" issue and get this PR merged.
It is resolved last week.
I still see that the corresponding PR check is in pending state and has not completed. Do you recommend we restart the checks?
![azure-pipelines[bot] avatar](https://avatars.githubusercontent.com/in/9426?v=4 "Published by a pub.dev verified publisher"){kind=small}
Run this image on x86_64-nokia_ixr7250e_sup-r0 and x86_64-nokia_ixr7250e_36x400g-r0, make sure all functions run correctly.
Why you state that you tested on 7250?
Run this image on x86_64-nokia_ixr7250e_sup-r0 and x86_64-nokia_ixr7250e_36x400g-r0, make sure all functions run correctly.
Why you state that you tested on 7250?
The H3 and 7250 share build files, so we have tested and made sure that our changes have not caused any regression to already existing platforms.
@liushilongbuaa After we rebased the PR to latest Master the internal conflict (ms_conflict) has appeared again. Could you please help resolve.
@liushilongbuaa After we rebased the PR to latest Master the internal conflict (ms_conflict) has appeared again. Could you please help resolve.
We are fixing the conflict. It still needs about 12 hours for the automation.
@liushilongbuaa After we rebased the PR to latest Master the internal conflict (ms_conflict) has appeared again. Could you please help resolve.
We are fixing the conflict. It still needs about 12 hours for the automation.
@liushilongbuaa Thank you for your help with this. Are we all set to retry the PR checks?