Updated sensors test for x86_64-arista_7170_64c platform due to updates in coretemp module#2255
Merged
yxieca merged 1 commit intosonic-net:masterfrom Sep 24, 2020
Merged
Conversation
Signed-off-by: Nazar Tkachuk <nazarx.tkachuk@intel.com>
yxieca
approved these changes
Sep 23, 2020
SavchukRomanLv
approved these changes
Sep 23, 2020
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
``` 39cdb49c7 [202012][show] Add bgpraw to show run all (sonic-net#2639) b3ebba2ca [202012][show] add new CLI to show tunnel route objects sonic-net#2255 (sonic-net#2659) d08f59b9f Fixed a bug in "show vnet routes all" causing screen overrun. (sonic-net#2644) (sonic-net#2654) a996abdb5 [202012][show] show logging CLI support for logs stored in tmpfs (sonic-net#2652) c60f771c0 [202012][show_bfd] add local discriminator in show bfd command (sonic-net#2616) ```
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
…t#11384) utilities: * 9f496a0 2022-07-06 | Update load minigraph to load backend acl (sonic-net#2236) (HEAD -> 202205, github/202205) [Neetha John] * ea938e3 2022-07-07 | Add support for IP interface loopback action (sonic-net#2192) [Lior Avramov] * a0b04de 2022-07-07 | [show] add new CLI to show tunnel route objects (sonic-net#2255) [Jing Zhang] platform-daemon: * e03d6a9 2022-07-06 | [ycabled][grpc] Fix some time interval,options for grpc keep alive to work (sonic-net#271) (HEAD -> 202205, github/202205) [vdahiya12] Signed-off-by: Ying Xie <ying.xie@microsoft.com>
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
3 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.
Signed-off-by: Nazar Tkachuk nazarx.tkachuk@intel.com
Description of PR
Summary:
Test sensors is failed on x86_64-arista_7170_64c platform with error
Path coretemp-isa-0000/Physical id 0/temp1_crit_alarm is not exist. This issue is observed on SONiC master.The root cause of failure: command
sensorsreturnsPackage id 0instead ofPhysical id 0as it was on SONiC 201911.Today SONiC master is based on kernel
4.19which containscoretempsmodule updates (see torvalds/linux@7126684).Based on these updates in
coretempsmodule output is correct(meansPackage id 0)Type of change
Approach
What is the motivation for this PR?
Fix test
sensorsfor x86_64-arista_7170_64c platformHow did you do it?
Updated approach for getting sensors date with regex to support backward compatibility (SONiC master and SONiC 201911)
How did you verify/test it?
Run
sensorstests on x86_64-arista_7170_64c platform with latest SONiC master and SONiC 201911Any platform specific information?
Fixed and verified only for x86_64-arista_7170_64c platform
Supported testbed topology if it's a new test case?
Documentation