Skip to content

[Nokia][asic-sensors] Modify the platform.json to enable the asic_sensors polling on Nokia platforms#20979

Merged
rlhui merged 1 commit intosonic-net:masterfrom
mlok-nokia:nokia-asic-sensors-polling
Dec 4, 2024
Merged

[Nokia][asic-sensors] Modify the platform.json to enable the asic_sensors polling on Nokia platforms#20979
rlhui merged 1 commit intosonic-net:masterfrom
mlok-nokia:nokia-asic-sensors-polling

Conversation

@mlok-nokia
Copy link
Copy Markdown
Contributor

Why I did it

Enable and use the asic_sensors polling on the Nokia platforms:
arm64-nokia_ixs7215_52xb-r0
x86_64-nokia_ixr7220_h4-r0
x86_64-nokia_ixr7220_h4_32d-r0
x86_64-nokia_ixr7220_h5_64d-r0
Fixes https://github.com/Nokia-ION/ndk/issues/36

Work item tracking
  • Microsoft ADO (number only):

How I did it

Modify the platform.json files with the below definition no Nokia platforms to enable the asic_sensors polling.
"asic_sensors": {
"polling_interval": "10",
"poll_admin_status": "enable"
}
This PR works with PR #20826

How to verify it

execute CLI "show platform temperature". The output will display the asic_sensors temperature.

admin@ixre-egl-board25:~$ show platform temperature 
         Sensor    Temperature    High TH    Low TH    Crit High TH    Crit Low TH    Warning          Timestamp
---------------  -------------  ---------  --------  --------------  -------------  ---------  -----------------
  ASIC0_0--FAB0             47        100        10           100                1      False  20241201 04:06:38
  ASIC0_1--FAB1             46        100        10           100                1      False  20241201 04:06:38
  ASIC0_2--FAB2             47        100        10           100                1      False  20241201 04:06:38
  ASIC0_3--FAB3             48        100        10           100                1      False  20241201 04:06:39
  ASIC0_4--NIF0             47        100        10           100                1      False  20241201 04:06:39
  ASIC0_5--NIF1             47        100        10           100                1      False  20241201 04:06:39
   ASIC0_6--PRM             46        100        10           100                1      False  20241201 04:06:39
  ASIC0_7--EMI0             44        100        10           100                1      False  20241201 04:06:39
  ASIC0_8--EMI1             44        100        10           100                1      False  20241201 04:06:40
 ASIC0_9--DRAM0             39        100        10           100                1      False  20241201 04:06:40
ASIC0_10--DRAM1             40        100        10           100                1      False  20241201 04:06:38
  ASIC1_0--FAB0             49        100        10           100                1      False  20241201 04:06:40
  ASIC1_1--FAB1             49        100        10           100                1      False  20241201 04:06:40
  ASIC1_2--FAB2             49        100        10           100                1      False  20241201 04:06:41
  ASIC1_3--FAB3             48        100        10           100                1      False  20241201 04:06:41
  ASIC1_4--NIF0             51        100        10           100                1      False  20241201 04:06:41
  ASIC1_5--NIF1             51        100        10           100                1      False  20241201 04:06:41
   ASIC1_6--PRM             50        100        10           100                1      False  20241201 04:06:42
  ASIC1_7--EMI0             47        100        10           100                1      False  20241201 04:06:42
  ASIC1_8--EMI1             47        100        10           100                1      False  20241201 04:06:42
 ASIC1_9--DRAM0             45        100        10           100                1      False  20241201 04:06:42
ASIC1_10--DRAM1             43        100        10           100                1      False  20241201 04:06:41
    temp_1(fan)             50        100        10           100                1      False  20241201 04:06:36
    temp_2(fan)             43         78        10            85.8              1      False  20241201 04:06:36
    temp_3(fan)             52         85        10            93.5              1      False  20241201 04:06:36
    temp_4(fan)             51         99        10           100                1      False  20241201 04:06:36
         temp_5             44         85        10            93.5              1      False  20241201 04:06:36
         temp_6             45         85        10            93.5              1      False  20241201 04:06:37
    temp_7(fan)             48        100        10           100                1      False  20241201 04:06:37
    temp_8(fan)             32         68        10            74.8              1      False  20241201 04:06:37
    temp_9(fan)             30         68        10            74.8              1      False  20241201 04:06:37
   temp_10(fan)             38         68        10            74.8              1      False  20241201 04:06:37
   temp_11(fan)             46         99        10           100                1      False  20241201 04:06:38

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305
  • [ x] 202405

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@mlok-nokia
Copy link
Copy Markdown
Contributor Author

@arlakshm @judyjoseph This PR is the asic_sensors Nokia platform related change. Please review it. Thanks

@mlok-nokia mlok-nokia changed the title [Nokia]{asic-sensors] Modify the platform.json to enable the asic_sensors polling on Nokia plaftorms [Nokia][asic-sensors] Modify the platform.json to enable the asic_sensors polling on Nokia platforms Dec 2, 2024
@rlhui rlhui merged commit 8df0930 into sonic-net:master Dec 4, 2024
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Dec 11, 2024
…sors polling on Nokia Platforms (sonic-net#20979)

Enable and use the asic_sensors polling on the Nokia platforms:
arm64-nokia_ixs7215_52xb-r0
x86_64-nokia_ixr7220_h4-r0
x86_64-nokia_ixr7220_h4_32d-r0
x86_64-nokia_ixr7220_h5_64d-r0
Fixes Nokia-ION/ndk#36

Signed-off-by: mlok <[email protected]>
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202405: #21138

mssonicbld pushed a commit that referenced this pull request Dec 12, 2024
…sors polling on Nokia Platforms (#20979)

Enable and use the asic_sensors polling on the Nokia platforms:
arm64-nokia_ixs7215_52xb-r0
x86_64-nokia_ixr7220_h4-r0
x86_64-nokia_ixr7220_h4_32d-r0
x86_64-nokia_ixr7220_h5_64d-r0
Fixes Nokia-ION/ndk#36

Signed-off-by: mlok <[email protected]>
VladimirKuk pushed a commit to Marvell-switching/sonic-buildimage that referenced this pull request Jan 21, 2025
…sors polling on Nokia Platforms (sonic-net#20979)

Enable and use the asic_sensors polling on the Nokia platforms:
arm64-nokia_ixs7215_52xb-r0
x86_64-nokia_ixr7220_h4-r0
x86_64-nokia_ixr7220_h4_32d-r0
x86_64-nokia_ixr7220_h5_64d-r0
Fixes Nokia-ION/ndk#36

Signed-off-by: mlok <[email protected]>
@mlok-nokia
Copy link
Copy Markdown
Contributor Author

@arlakshm and @judyjoseph This PR is needed by 202411. Please help to cherry-pick it. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

7 participants