Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Thermalctld] Update thermal info to CHASSIS_STATE_DB #101

Merged
merged 2 commits into from
Nov 11, 2020

Conversation

mprabhu-nokia
Copy link
Contributor

@mprabhu-nokia mprabhu-nokia commented Oct 14, 2020

Enhance thermalctld to write to chassis state-DB on a modular chassis

HLD: sonic-net/SONiC#646

In a modular chassis, the thermal information from all line-cards
will be updated to the chassis state-DB in the control-card.

Additionally, minimum and maximum temperatures will be recorded.
The fan control algorithm used by certain vendors will require
this information.

In a modular chassis, the thermal information from all line-cards
will be updated to the global-state-db in the control-card.

Additionally, minimum and maximum temperatures will be recorded.
The fan control algorithm used by certain vendors will require
this information.
@anshuv-mfst
Copy link

@judyjoseph - could you please review.
@shyam77git - please approve if comments are met.

@mprabhu-nokia mprabhu-nokia changed the title Thermalctld update thermal info to global-state-db Thermalctld update thermal info to CHASSIS_STATE_DB Nov 6, 2020
@jleveque
Copy link
Contributor

jleveque commented Nov 6, 2020

@Junchao-Mellanox: Please review, also.

Copy link
Contributor

@judyjoseph judyjoseph left a comment

Choose a reason for hiding this comment

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

LGTM

@jleveque jleveque changed the title Thermalctld update thermal info to CHASSIS_STATE_DB [Thermalctld] Update thermal info to CHASSIS_STATE_DB Nov 11, 2020
@jleveque jleveque merged commit 8c2a5cc into sonic-net:master Nov 11, 2020
@mprabhu-nokia mprabhu-nokia deleted the nokia-thermal-pr branch November 12, 2020 21:45
yxieca pushed a commit to sonic-net/sonic-mgmt that referenced this pull request Mar 19, 2021
#3117)

Approach
What is the motivation for this PR?
Add new attribute modular_chassis which will be set true if DUT is modular chassis and False for other
Add support to run tests on tests/platform_tests/test_cpu_memory_usage.py, tests/platform_tests/test_port_toggle.py on T2 chassis, change test_show_chassis_module to run duts per hwsku.
Add new tests to verify thermal local and global state db on VoQ chassis introduced by PRs sonic-net/sonic-swss-common#395 and sonic-net/sonic-platform-daemons#101

How did you do it?
modifications to current tests and new test file as below:

tests/common/devices/sonic.py
Add new attribute modular_chassis which can be used to skip tests only for modular chassis if DUT is not chassis
possible values for attribute are True if DUT is part of modular chassis and False if not part of modular chassis

tests/platform_tests/cli/test_show_chassis_module.py

change test select one dut per hwsku instead of running on all DUTs
tests/platform_tests/test_cpu_memory_usage.py

change rand_one_dut_hostname to enum_rand_one_per_hwsku_hostname to test per hwsku.
tests/platform_tests/test_port_toggle.py

change rand_one_dut_hostname to enum_rand_one_per_hwsku_hostname to test per hwsku.
tests/platform_tests/test_thermal_state_db.py

Add new test for t2 chassis to verify thermal data in local state db for all thermal sensors
Verify global state db in supervisor has data for all sensors from all modules in chassis
How did you verify/test it?
Validated the modified tests against a chassis .
vmittal-msft pushed a commit to vmittal-msft/sonic-mgmt that referenced this pull request Sep 28, 2021
sonic-net#3117)

Approach
What is the motivation for this PR?
Add new attribute modular_chassis which will be set true if DUT is modular chassis and False for other
Add support to run tests on tests/platform_tests/test_cpu_memory_usage.py, tests/platform_tests/test_port_toggle.py on T2 chassis, change test_show_chassis_module to run duts per hwsku.
Add new tests to verify thermal local and global state db on VoQ chassis introduced by PRs sonic-net/sonic-swss-common#395 and sonic-net/sonic-platform-daemons#101

How did you do it?
modifications to current tests and new test file as below:

tests/common/devices/sonic.py
Add new attribute modular_chassis which can be used to skip tests only for modular chassis if DUT is not chassis
possible values for attribute are True if DUT is part of modular chassis and False if not part of modular chassis

tests/platform_tests/cli/test_show_chassis_module.py

change test select one dut per hwsku instead of running on all DUTs
tests/platform_tests/test_cpu_memory_usage.py

change rand_one_dut_hostname to enum_rand_one_per_hwsku_hostname to test per hwsku.
tests/platform_tests/test_port_toggle.py

change rand_one_dut_hostname to enum_rand_one_per_hwsku_hostname to test per hwsku.
tests/platform_tests/test_thermal_state_db.py

Add new test for t2 chassis to verify thermal data in local state db for all thermal sensors
Verify global state db in supervisor has data for all sensors from all modules in chassis
How did you verify/test it?
Validated the modified tests against a chassis .
vdahiya12 pushed a commit to vdahiya12/sonic-platform-daemons that referenced this pull request Apr 4, 2022
…QSFP-DD codes (sonic-net#101)

- Add parser for QSFP-DD cable type.
- Add dictionaries for QSFP-DD codes.

Signed-off-by: Shlomi Bitton <shlomibi@mellanox.com>
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.

6 participants