Skip to content

Conversation

@YairRaviv
Copy link

@YairRaviv YairRaviv commented Jan 14, 2026

Description

  • Modified is_syncd_warm_restore_complete() to accept namespace parameter to connect to the correct STATE_DB for each ASIC
  • Added get_namespace_from_asic_id() helper function to convert ASIC ID to namespace string
  • Updated _post_port_sfp_info_and_dom_thr_to_db_once() to pre-fetch warm start status for all namespaces and check per-ASIC during port processing
  • Updated deinit() to check warm/fast reboot status per-ASIC namespace
  • Added unit tests for namespace-aware warm restore checking

Motivation and Context

HLD - Warm-reboot multi-ASIC HLD

How Has This Been Tested?

  1. Manual testing on Multi ASIC platform simulation running SONiC
  2. Automated warm-boot and cold-boot tests on real hardware Single ASIC device

Additional Information (Optional)

- Modified is_syncd_warm_restore_complete() to accept namespace parameter
  to connect to the correct STATE_DB for each ASIC
- Added get_namespace_from_asic_id() helper function to convert ASIC ID
  to namespace string
- Updated _post_port_sfp_info_and_dom_thr_to_db_once() to pre-fetch warm
  start status for all namespaces and check per-ASIC during port processing
- Updated deinit() to check warm/fast reboot status per-ASIC namespace
- Added unit tests for namespace-aware warm restore checking

Signed-off-by: Stepan Blyschak <[email protected]>
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prgeor
Copy link
Collaborator

prgeor commented Jan 25, 2026

@YairRaviv please populate this section
How Has This Been Tested? for

  1. platform with mult-asic
  2. platform with single asic
  3. warm boot for 1 and 2
  4. cold boot for 1 and 2

@YairRaviv
Copy link
Author

@prgeor Added tests section to the description

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants