Skip to content

Add chassis db.#1085

Merged
qiluo-msft merged 1 commit intosonic-net:masterfrom
dypet:add_chassis_db
Oct 14, 2025
Merged

Add chassis db.#1085
qiluo-msft merged 1 commit intosonic-net:masterfrom
dypet:add_chassis_db

Conversation

@dypet
Copy link
Copy Markdown
Contributor

@dypet dypet commented Oct 6, 2025

Chassis DB were missing in the swss-common-testing crate causing some new UT to fail (sonic-net/sonic-dash-ha#120):

Actor failed to handle message: Connecting to db `CHASSIS_STATE_DB`: [SWSSResult SWSSDBConnector_new_keyed(const char*, uint32_t, uint8_t, const char*, const char*, SWSSDBConnectorOpaque**)] Failed to find CHASSIS_STATE_DB database in : key
handle_swbus_message{actor="unknown.unknown.unknown/hamgrd/0/dpu/switch0_dpu0" id=1759761742796580595}: message handled by actor: Fail Connecting to db `CHASSIS_STATE_DB`: [SWSSResult SWSSDBConnector_new_keyed(const char*, uint32_t, uint8_t, const char*, const char*, SWSSDBConnectorOpaque**)] Failed to find CHASSIS_STATE_DB database in : key

Adding these DB to the redis config to fix the above error.

@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Oct 6, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Oct 6, 2025

Azure Pipelines successfully started running 1 pipeline(s).

@dypet dypet mentioned this pull request Oct 6, 2025
Merged
zjswhhh
zjswhhh approved these changes Oct 10, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@zjswhhh zjswhhh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@zjswhhh
Copy link
Copy Markdown
Contributor

zjswhhh commented Oct 10, 2025

Hi @qiluo-msft - please help merge.

@qiluo-msft qiluo-msft merged commit 130d8b6 into sonic-net:master Oct 14, 2025
18 checks passed
@mssonicbld mssonicbld mentioned this pull request Dec 4, 2025
Merged
@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Dec 4, 2025

Cherry-pick PR to 202511: #1128

@prabhataravind prabhataravind mentioned this pull request Dec 10, 2025
Open
@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Dec 13, 2025

@dypet cherry pick PR didn't pass PR checker. Please check!!!
#1128

1 similar comment
@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Dec 13, 2025

@dypet cherry pick PR didn't pass PR checker. Please check!!!
#1128

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

Reviewers

@qiluo-msft qiluo-msft qiluo-msft approved these changes

@zjswhhh zjswhhh zjswhhh approved these changes

@yue-fred-gao yue-fred-gao yue-fred-gao approved these changes

Assignees

No one assigned

Labels

Approved for msft-202506 Branch Approved for 202511 Branch Cherry Pick Conflict_msft-202506 Created PR to 202511 Branch Request for msft-202506 Branch Request for 202511 Branch

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@dypet @mssonicbld @zjswhhh @qiluo-msft @yue-fred-gao @prsunny @KrisNey-MSFT @vmittal-msft