Skip to content

[action] [PR:612] [module_base] Change state db connector to be lazy initialized#614

Merged
mssonicbld merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/612
Dec 4, 2025
Merged

[action] [PR:612] [module_base] Change state db connector to be lazy initialized#614
mssonicbld merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/612

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description

Initialize state_db entry only when needed
The module_base implementation for state_db connector is used only in smartswitch platforms. To prevent test failures in platform code (because redis dbs are not initialized for platform tests) The module base implementation is changed to initialize the state_db connector only when needed

Motivation and Context

How Has This Been Tested?

Additional Information (Optional)

<!-- Provide a general summary of your changes in the Title above -->

#### Description
<!--
     Describe your changes in detail
-->
Initialize state_db entry only when needed
The module_base implementation for state_db connector is used only in smartswitch platforms. To prevent test failures in platform code (because redis dbs are not initialized for platform tests) The module base implementation is changed to initialize the state_db connector only when needed

#### Motivation and Context
<!--
     Why is this change required? What problem does it solve?
     If this pull request closes/resolves an open Issue, make sure you
     include the text "fixes #xxxx", "closes #xxxx" or "resolves #xxxx" here
-->

#### How Has This Been Tested?
<!--
     Please describe in detail how you tested your changes.
     Include details of your testing environment, and the tests you ran to
     see how your change affects other areas of the code, etc.
-->

#### Additional Information (Optional)
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #612

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit dae5cfa into sonic-net:202511 Dec 4, 2025
5 of 6 checks passed
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.

1 participant