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

[device/Seastone] Update sensors conf on DX010 #1430

Merged
merged 1 commit into from Mar 23, 2018
Merged

[device/Seastone] Update sensors conf on DX010 #1430

merged 1 commit into from Mar 23, 2018

Conversation

pphuchar
Copy link
Contributor

Update sensors configuration file. Add threshold to PSU sensors values.

What I did

  • Update the sensors configuration file for Celestica Seastone DX010.
  • Update power supply voltage constrain in sensors.conf.

How to verify it

  • In cli run sensors to check the constrain of PSU and compare to configuration file.

Update sensors configuration file. Add threshold to PSU sensors values.
Copy link
Collaborator

@zhenggen-xu zhenggen-xu left a comment

Choose a reason for hiding this comment

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

Can you add threshold settings for 6 temperature sensors ("Power Supply 1/2 temp sensor 1/2/3" ) as well? Thanks!

@pphuchar
Copy link
Contributor Author

pphuchar commented Feb 28, 2018

Yes, I will add the threshold for those sensors.

@lguohan
Copy link
Collaborator

lguohan commented Mar 14, 2018

is this closed?

@mudsut4ke
Copy link
Contributor

After update settings on threshold for 6 temperature sensors ("Power Supply 1/2 temp sensor 1/2/3" ) , It doesn't have effect on machine.

@zhenggen-xu
Copy link
Collaborator

OK, let's push the changes for now, and we will address the temperature sensor issues when we found the solution and generate a new PR.

@lguohan lguohan merged commit c083532 into sonic-net:master Mar 23, 2018
@pphuchar pphuchar deleted the update-celestica-dx010-sensors branch August 1, 2018 08:11
abdosi added a commit that referenced this pull request Sep 19, 2020
Avoid adding loopback interface (ip link add) when setting nat zone on
loopback interface (#1434)
    [acl] Remove Ethertype from L3V6 qualifiers (#1433)
    Sflow fixes during DEL processing (#1427)
    Fix #3971 by skipping create-only SAI attributes when modifying
    buffer pools or profiles in orchagent (#1430)
    Fix issue: bufferorch only pass the first attribute to sai when
    setting attribute (#1442)

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-buildimage that referenced this pull request Jan 14, 2021
…ing buffer pools or profiles in orchagent (sonic-net#1430)

* Skip create-only fields when modify buffer pool or profile in orchagent

This PR also fixes the issue sonic-net#3971 redis sends unchanged field

Signed-off-by: Stephen Sun <stephens@mellanox.com>

* Fix review comments

Signed-off-by: Stephen Sun <stephens@nvidia.com>

Co-authored-by: Stephen Sun <stephens@mellanox.com>
lguohan pushed a commit that referenced this pull request Feb 14, 2021
Commits include:

* src/sonic-utilities c7e46c9...42cab68 (3):
  > [consutil] Look for udevprefix.conf file under platform dir, not plugins (#1431)
  > [ci]: download from sonic-buildimage.vs artifact (#1428)
  > [storyteller] sort output by time and improve lag support (#1430)
stepanblyschak pushed a commit to stepanblyschak/sonic-buildimage that referenced this pull request May 10, 2021
…1430)

What I did
sort output by time
improve lag support according to event format changes
add an option to sort the log files by a field of file name (after log files got moved or copied, sorting by timestamp is no longer reliable).
Signed-off-by: Ying Xie ying.xie@microsoft.com

How to verify it
run 'sudo storyteller --since "2021-2-10" -c lag' on a dut
theasianpianist pushed a commit to theasianpianist/sonic-buildimage that referenced this pull request Feb 5, 2022
…ing buffer pools or profiles in orchagent (sonic-net#1430)

* Skip create-only fields when modify buffer pool or profile in orchagent

This PR also fixes the issue sonic-net#3971 redis sends unchanged field

Signed-off-by: Stephen Sun <stephens@mellanox.com>

* Fix review comments

Signed-off-by: Stephen Sun <stephens@nvidia.com>

Co-authored-by: Stephen Sun <stephens@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.

None yet

5 participants