Skip to content

[warm/fast-reboot] Retain TRANSCEIVER_INFO/STATUS tables on deinit#454

Merged
prgeor merged 4 commits intosonic-net:masterfrom
stepanblyschak:retain-xcvr-tables
Apr 5, 2024
Merged

[warm/fast-reboot] Retain TRANSCEIVER_INFO/STATUS tables on deinit#454
prgeor merged 4 commits intosonic-net:masterfrom
stepanblyschak:retain-xcvr-tables

Conversation

@stepanblyschak
Copy link
Copy Markdown
Contributor

@stepanblyschak stepanblyschak commented Mar 25, 2024

Description

Save TRANSCEIVER_INFO/STATUS tables on warm/fast-reboot.

Motivation and Context

The information in TRANSCEIVER_INFO tables is required for orchagent to set SAI_PORT_ATTR_HOST_TX_SIGNAL_ENABLE=true on system startup, otherwise after APPLY_VIEW port becomes down until pmon populates TRANSCEIEVER_INFO table.

How Has This Been Tested?

Run fast/warm-reboot.

Additional Information (Optional)

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
@mihirpat1
Copy link
Copy Markdown
Contributor

@stepanblyschak - Can you please elaborate on the usage of each TRANSCEIVER_INFO and TRANSCEIVER_STATUS tables by orchagent.
Also, please list the in description about the current issue caused by these tables being deleted during warm reboot (i.e. what happens if OA does not set SAI_PORT_ATTR_HOST_TX_SIGNAL_ENABLE=true on system startup).

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
@keboliu
Copy link
Copy Markdown
Collaborator

keboliu commented Apr 21, 2024

@stepanblyschak would you please handle the cherry-pick conflict?

@stepanblyschak
Copy link
Copy Markdown
Contributor Author

@keboliu Cherry-pick #477

lotus-nexthop pushed a commit to lotus-nexthop/sonic-platform-daemons that referenced this pull request Oct 28, 2025
…onic-net#454)

* Added new APIs and enhanced the required APIs in "module_base.py and
chassis_base.py" to support SmartSwitch
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.

5 participants