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

[PikeZ] Correct port index in Arista-720DT-48S/phy24_config.json #11699

Merged
merged 1 commit into from
Aug 17, 2022

Conversation

jimmyzhai
Copy link
Contributor

@jimmyzhai jimmyzhai commented Aug 11, 2022

Why I did it

Port index 22 is associated with phy23_config.json, then same port index 22 in phy24_config.json may cause gearbox port creation error. Port Ethernet22 maps to index 23.

The related error messages:

Aug 12 01:59:09.077562 bjw-pikez-112 NOTICE swss#orchagent: :- initGearboxPort: BOX: port_id:0x1000000000018 index:23 alias:Ethernet22
Aug 12 01:59:09.077562 bjw-pikez-112 NOTICE swss#orchagent: :- initGearboxPort: BOX: Gearbox port Ethernet22 assigned phyOid 0x1621010000000016
Aug 12 01:59:09.078782 bjw-pikez-112 ERR swss#orchagent: :- meta_generic_validation_create: SAI_PORT_ATTR_SPEED:SAI_ATTR_VALUE_TYPE_UINT32 attribute is mandatory but not passed in attr list
Aug 12 01:59:09.078782 bjw-pikez-112 ERR swss#orchagent: :- initGearboxPort: BOX: Failed to create Gearbox system-side port for alias:Ethernet22 port_id:0x1000000000018 index:23 status:-14
Aug 12 01:59:09.078782 bjw-pikez-112 ERR swss#orchagent: :- handleSaiCreateStatus: Encountered failure in create operation, exiting orchagent, SAI API: SAI_API_PORT, status: SAI_STATUS_MANDATORY_ATTRIBUTE_MISSING

How I did it

Correct port index from 22 to 23 in phy24_config.json.

How to verify it

Gearbox port Ethernet22 is created successfully.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205

Description for the changelog

Link to config_db schema for YANG module changes

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

@Blueve
Copy link
Contributor

Blueve commented Aug 15, 2022

@andywongarista for awareness

@Blueve
Copy link
Contributor

Blueve commented Aug 15, 2022

Remove the backporting tag due to previous PR is not backported yet due to conflict

@yxieca
Copy link
Contributor

yxieca commented Aug 26, 2022

@jimmyzhai this change cannot be cherry-picked to 202205 cleanly, please create separate PR.

yxieca pushed a commit that referenced this pull request Sep 1, 2022
Port index 22 is associated with phy23_config.json, then same port index 22 in phy24_config.json may cause gearbox port creation error. Port Ethernet22 maps to index 23.
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