[Sensors]fix sensors test for DellS6000#458
Closed
maggiemsft wants to merge 1 commit intosonic-net:masterfrom
Closed
[Sensors]fix sensors test for DellS6000#458maggiemsft wants to merge 1 commit intosonic-net:masterfrom
maggiemsft wants to merge 1 commit intosonic-net:masterfrom
Conversation
Contributor
|
should be two separate PR one for mellanox, one for dell s6000. |
Contributor
Author
|
close it submit new PR |
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
Update the sonic-swss-common submodule. The following are the commits in the submodule. f01fede [debian/control] libswsscommon-dev depends on libbost-dev (sonic-net#458) 607a8ce Convert return value of get_all function in SonicV2Connector to dict (sonic-net#462)
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
…ules (sonic-net#2569) PR#2538 cannot merge due to master branch status. It has been tested against 201811 branch. Submodule src/sonic-sairedis 21f4a49..d57222a: > Add more specific logic for ingress ACL and buffer profile (sonic-net#421) > Move warm_restart enable/disable config to stateDB WARM_RESTART_ENABLE_TABLE (sonic-net#418) > Add support for vlan tagged frames in virtual switch (sonic-net#417) Submodule src/sonic-swss 1590030..584490c: > Move warm_restart enable/disable config to stateDB WARM_RESTART_ENABLE_TABLE (sonic-net#786) > [vstest]: Potential fix for timing issue in warm_reboot's routing UT (sonic-net#788) Submodule src/sonic-swss-common 594f4e8..286ef34: > Move warm_restart enable/disable config to stateDB WARM_RESTART_ENABLE_TABLE (sonic-net#260) Submodule src/sonic-utilities c6666e2..b44b462: > Move warm_restart enable/disable config to stateDB WARM_RESTART_ENABL… (sonic-net#458) > [aclshow] output only counters per table/rule (sonic-net#442) Signed-off-by: Ying Xie <[email protected]> [PR 2538] Move warm_restart enable/disable config to stateDB WARM_RESTART_ENABLE_TABLE Signed-off-by: Jipan Yang <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of PR
Fixes sensors test
Type of change
Approach
How did you do it?
DellS6000 sensors test fail due to wrong module. remove the test check module of the wrong module name
Also, Mellanox the same hardware has two sku name, change sensors test yaml file to avoid adding new sensors data for new sku.
How did you verify/test it?
tested in my local testbed
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation