-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[202012][e1031] add GbE and higig port to E1031 port_config.ini #7224
Conversation
Why I did it These ports are being enumerated by the latest SAI. But they are not defined in port_config.ini. SONiC end up trying to delete these 3 ports and hit SAI error and crash. How I did it Add the GbE and the 2 HiGig ports in the port_config.ini. How to verify it Put the port_config.ini on a device crashing with port deleting. load minigraph and the crash stopped. Signed-off-by: Ying Xie ying.xie@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.
I don't believe the autoneg column exists in 202012
@yxieca it seems not work. I check it and find swss could not start, syslog error info as below root@localhost:/home/admin# show version SONiC Software Version: SONiC.202012.85-e6bbb3c3 Platform: x86_64-cel_e1031-r0 Docker images: |
@119064273 can you file an issue and attach the show techsupport? |
|
@yxieca I double confirm it, it‘s ok now, after generate a new config_db.json , swss could run without crash, old config_db.json do not have port 52-54 so the error is the same, now it's ok , thanks |
Port change in #7208 to 202012 branch
Why I did it
These ports are being enumerated by the latest SAI. But they are not defined in port_config.ini.
SONiC end up trying to delete these 3 ports and hit SAI error and crash.
How I did it
Add the GbE and the 2 HiGig ports in the port_config.ini.
How to verify it
Put the port_config.ini on a device crashing with port deleting. load minigraph and the crash stopped.
Signed-off-by: Ying Xie ying.xie@microsoft.com