Skip to content
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

[7050CX3-32S-C32] Add the 10G ports with updated speed #6699

Merged
merged 1 commit into from
Feb 6, 2021

Conversation

vaibhavhd
Copy link
Contributor

- Why I did it
#6638 brought in changes which made ProtOrch crash with below error:

Feb  5 19:30:55.923878 str2-7050cx3-acs-01 ERR syncd#syncd: [none] SAI_API_PORT:brcm_sai_flex_port_oper:10451 port resource multi set failed with error Invalid parameter (0xfffffffc).
Feb  5 19:30:55.924407 str2-7050cx3-acs-01 INFO syncd#/supervisord: syncd 0:_bcmi_xgs5_port_flexible_validate: FlexPort operation is not enabled on physical port 128#015
Feb  5 19:30:55.924612 str2-7050cx3-acs-01 ERR syncd#syncd: :- sendApiResponse: api SAI_COMMON_API_REMOVE failed in syncd mode: SAI_STATUS_FAILURE
Feb  5 19:30:55.924787 str2-7050cx3-acs-01 ERR swss#orchagent: :- remove: remove status: SAI_STATUS_FAILURE
Feb  5 19:30:55.926302 str2-7050cx3-acs-01 INFO swss#/supervisord: orchagent terminate called after throwing an instance of 'std::runtime_error'
Feb  5 19:30:55.926521 str2-7050cx3-acs-01 INFO swss#/supervisord: orchagent   what():  PortsOrch initialization failure.
Feb  5 19:30:56.119747 str2-7050cx3-acs-01 INFO swss#supervisord 2021-02-05 19:30:56,119 INFO exited: orchagent (terminated by SIGABRT (core dumped); not expected)

The reason seems be due to BRCM file still has the port https://github.com/Azure/sonic-buildimage/blob/580666a406fa2ea70972fca6d64e21732095bb38/device/arista/x86_64-arista_7050cx3_32s/Arista-7050CX3-32S-C32/td3-a7050cx3-32s-32x100G.config.bcm

- How I did it
Two ports are added to back with speed 10G. This is so that port_config.ini is in sync with config.bcm file.

Note: the lanes are still looking incorrect. This probably needs a fix from Arista.

- How to verify it
Tested warm-reboot, the PortsOrch initialization failure. issue is not seen. However, there is another port warm-reboot issue that is seen:

Feb  5 22:09:44.804084 str2-7050cx3-acs-01 NOTICE admin: Rebooting with /sbin/kexec -e to SONiC-OS-HEAD.350-ca355037 ...

Feb  5 22:10:28.487262 str2-7050cx3-acs-01 NOTICE swss#orchagent: :- notifySyncd: sending syncd: INIT_VIEW
Feb  5 22:11:28.545452 str2-7050cx3-acs-01 ERR swss#orchagent: :- wait: SELECT operation result: TIMEOUT on notify
Feb  5 22:11:28.545452 str2-7050cx3-acs-01 ERR swss#orchagent: :- wait: failed to get response for notify
Feb  5 22:11:28.545982 str2-7050cx3-acs-01 ERR swss#orchagent: :- initSaiRedis: Failed to notify syncd INIT_VIEW, rv:-1 gSwitchId 0
Feb  5 22:11:28.545982 str2-7050cx3-acs-01 WARNING swss#orchagent: :- meta_sai_on_switch_shutdown_request: switch_id oid:0x0 is of type SAI_OBJECT_TYPE_NULL, but expected SAI_OBJECT_TYPE_SWITCH

Feb  5 22:11:28.546022 str2-7050cx3-acs-01 ERR swss#orchagent: :- operator(): not handled: SAI_OBJECT_TYPE_NULL
Feb  5 22:11:28.546183 str2-7050cx3-acs-01 NOTICE swss#orchagent: :- uninitialize: begin

- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@vaibhavhd vaibhavhd merged commit a7ba1b8 into sonic-net:master Feb 6, 2021
@vaibhavhd vaibhavhd deleted the fix-7050cx-portconfig branch February 6, 2021 03:36
lguohan pushed a commit that referenced this pull request Feb 7, 2021
Port_config update for hwsku 7050CX3-32S-C3 - add two 10G ports.
This change is added to fix issue of "PortsOrch initialization failure" seen by previous removal of these 10G ports.
Tested on the device with new minigraph, and the PortsOrch initialization failure is not seen.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants