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

tahan: config: Delete the extra configuration SMB_FRU "3 5 20" and fix NETLAKE "3 1 2" wrong configuration #314

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zhongedward
Copy link
Contributor

@zhongedward zhongedward commented Dec 24, 2024

Description
This PR is for tahan come pvt sensor config file.

Motivation
The sensor config file submitted(#312) last time has errors(The wrong file was uploaded, resulting in an extra SMB_FRU "3 5 20" configuration and a wrong NETLAKE "3 1 2" configuration), in this PR revise it and submit the correct file again.
1.This is the wrong file content:
image

2.This is the file after repair:

image

Test Plan
1.The correctness of the format has been verified on this website https://jsonlint.com/
2.Used jq cmd to pretty the format.
3.Test log as follows:
Tested in tahan come pvt 2nd source machine.

image

tahan_ppvt_sensor_service_test_12_20_log.txt

@zhongedward zhongedward changed the title tahan: config: The sensor config file submitted last time has errors, submit again tahan: config: Fix the extra configuration SMB_FRU "3 5 20" and NETLAKE "3 1 2" wrong configuration Dec 24, 2024
@zhongedward zhongedward changed the title tahan: config: Fix the extra configuration SMB_FRU "3 5 20" and NETLAKE "3 1 2" wrong configuration tahan: config: Delete the extra configuration SMB_FRU "3 5 20" and fix NETLAKE "3 1 2" wrong configuration Dec 24, 2024
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.

2 participants