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

[202305][Ufispace][PDDF] Add PDDF support on Ufispace platforms. #17625

Closed
wants to merge 17 commits into from

Conversation

nonodark
Copy link
Contributor

Why I did it

Add PDDF support on following Ufispace platforms.

  • Broadcom ASIC
    • S9300-32D
    • S9301-32D
    • S9301-32DB
    • S9110-32X
    • S8901-54XC
    • S7801-54XS
    • S6301-56ST
  • Barefoot ASIC
    • S9180-32X

How I did it

Add PDDF configuration files, scripts and python files

How to verify it

Run pddf commands and show commands.

S9300-32D
root@sonic:~# pddf_psuutil status
PSU    Status
-----  -----------
PSU1   NOT PRESENT
PSU2   OK
root@sonic:~# pddf_thermalutil gettemp
Temp Sensor       Value
----------------  -------
Temp_CPU_PECI     temp1  +43.0 C (high = +95.0 C, crit = +100.0 C)
Temp_CPU_ENV      temp1  +41.0 C (high = +85.0 C, crit = +90.0 C)
Temp_CPU_ENV2     temp1  +34.0 C (high = +70.0 C, crit = +80.0 C)
Temp_MAC_ENV      temp1  +35.0 C (high = +85.0 C, crit = +90.0 C)
Temp_MAC_DIE      temp1  +40.0 C (high = +100.0 C, crit = +105.0 C)
Temp_CAGE         temp1  +30.0 C (high = +85.0 C, crit = +90.0 C)
Temp_PSU_CONNTOR  temp1  +32.0 C (high = +65.0 C, crit = +70.0 C)
root@sonic:~# pddf_fanutil status
FAN         Status
----------  --------
Fantray1_1  OK
Fantray1_2  OK
Fantray2_1  OK
Fantray2_2  OK
Fantray3_1  OK
Fantray3_2  OK
Fantray4_1  OK
Fantray4_2  OK
Fantray5_1  OK
Fantray5_2  OK
Fantray6_1  OK
Fantray6_2  OK
root@sonic:~# pddf_ledutil getstatusled SYS_LED
green
root@sonic:~# show platform fan
  Drawer    LED         FAN    Speed    Direction     Presence    Status          Timestamp
--------  -----  ----------  -------  -----------  -----------  --------  -----------------
Fantray1  green  Fantray1_1      40%      EXHAUST      Present        OK  20231227 07:25:14
Fantray1  green  Fantray1_2      39%      EXHAUST      Present        OK  20231227 07:25:14
Fantray2  green  Fantray2_1      39%      EXHAUST      Present        OK  20231227 07:25:14
Fantray2  green  Fantray2_2      40%      EXHAUST      Present        OK  20231227 07:25:14
Fantray3  green  Fantray3_1      40%      EXHAUST      Present        OK  20231227 07:25:14
Fantray3  green  Fantray3_2      40%      EXHAUST      Present        OK  20231227 07:25:14
Fantray4  green  Fantray4_1      39%      EXHAUST      Present        OK  20231227 07:25:15
Fantray4  green  Fantray4_2      39%      EXHAUST      Present        OK  20231227 07:25:15
Fantray5  green  Fantray5_1      39%      EXHAUST      Present        OK  20231227 07:25:15
Fantray5  green  Fantray5_2      39%      EXHAUST      Present        OK  20231227 07:25:15
Fantray6  green  Fantray6_1      40%      EXHAUST      Present        OK  20231227 07:25:15
Fantray6  green  Fantray6_2      39%      EXHAUST      Present        OK  20231227 07:25:15
     N/A    N/A   PSU1_FAN1      N/A          N/A  Not Present       N/A  20231227 07:25:18
     N/A    N/A   PSU2_FAN1      45%      EXHAUST      Present        OK  20231227 07:25:21
root@sonic:~# sfputil show presence 
Port         Presence
-----------  -----------
Ethernet0    Not present
Ethernet8    Not present
Ethernet16   Present
Ethernet24   Present
Ethernet32   Not present
Ethernet40   Not present
Ethernet48   Not present
Ethernet56   Not present
Ethernet64   Not present
Ethernet72   Not present
Ethernet80   Not present
Ethernet88   Not present
Ethernet96   Not present
Ethernet104  Not present
Ethernet112  Not present
Ethernet120  Not present
Ethernet128  Not present
Ethernet136  Not present
Ethernet144  Not present
Ethernet152  Not present
Ethernet160  Not present
Ethernet168  Not present
Ethernet176  Not present
Ethernet184  Not present
Ethernet192  Not present
Ethernet200  Not present
Ethernet208  Present
Ethernet216  Present
Ethernet224  Not present
Ethernet232  Not present
Ethernet240  Not present
Ethernet248  Not present
Ethernet256  Present
Ethernet257  Not present
S9301-32D
root@sonic:~# pddf_psuutil status
PSU    Status
-----  --------
PSU1   OK
PSU2   NOT OK
root@sonic:~# pddf_thermalutil gettemp
Temp Sensor       Value
----------------  -------
Temp_CPU_PECI     temp1  +47.0 C (high = +95.0 C, crit = +100.0 C)
Temp_CPU_ENV      temp1  +43.0 C (high = +85.0 C, crit = +90.0 C)
Temp_CPU_ENV2     temp1  +35.0 C (high = +70.0 C, crit = +80.0 C)
Temp_MAC_ENV      temp1  +39.0 C (high = +85.0 C, crit = +90.0 C)
Temp_MAC_DIE      temp1  +45.0 C (high = +100.0 C, crit = +105.0 C)
Temp_CAGE         temp1  +31.0 C (high = +85.0 C, crit = +90.0 C)
Temp_PSU_CONNTOR  temp1  +33.0 C (high = +65.0 C, crit = +70.0 C)
root@sonic:~# pddf_fanutil status
FAN         Status
----------  --------
Fantray1_1  OK
Fantray1_2  OK
Fantray2_1  OK
Fantray2_2  OK
Fantray3_1  OK
Fantray3_2  OK
Fantray4_1  OK
Fantray4_2  OK
Fantray5_1  OK
Fantray5_2  OK
Fantray6_1  OK
Fantray6_2  OK
root@sonic:~# pddf_ledutil getstatusled SYS_LED
green
root@sonic:~# show interfaces transceiver presence
Port         Presence
-----------  -----------
Ethernet0    Not present
Ethernet8    Not present
Ethernet16   Present
Ethernet24   Present
Ethernet32   Not present
Ethernet40   Not present
Ethernet48   Present
Ethernet56   Present
Ethernet64   Not present
Ethernet72   Not present
Ethernet80   Not present
Ethernet88   Not present
Ethernet96   Not present
Ethernet104  Not present
Ethernet112  Present
Ethernet120  Present
Ethernet128  Not present
Ethernet136  Not present
Ethernet144  Not present
Ethernet152  Not present
Ethernet160  Not present
Ethernet168  Not present
Ethernet176  Not present
Ethernet184  Not present
Ethernet192  Not present
Ethernet200  Not present
Ethernet208  Not present
Ethernet216  Not present
Ethernet224  Not present
Ethernet232  Not present
Ethernet240  Not present
Ethernet248  Not present
Ethernet256  Not present
Ethernet257  Not present
root@sonic:~# show platform fan
  Drawer    LED         FAN    Speed    Direction    Presence    Status          Timestamp
--------  -----  ----------  -------  -----------  ----------  --------  -----------------
Fantray1  green  Fantray1_1      55%      EXHAUST     Present        OK  20230821 09:34:36
Fantray1  green  Fantray1_2      56%      EXHAUST     Present        OK  20230821 09:34:36
Fantray2  green  Fantray2_1      55%      EXHAUST     Present        OK  20230821 09:34:36
Fantray2  green  Fantray2_2      56%      EXHAUST     Present        OK  20230821 09:34:36
Fantray3  green  Fantray3_1      55%      EXHAUST     Present        OK  20230821 09:34:36
Fantray3  green  Fantray3_2      56%      EXHAUST     Present        OK  20230821 09:34:36
Fantray4  green  Fantray4_1      56%      EXHAUST     Present        OK  20230821 09:34:36
Fantray4  green  Fantray4_2      56%      EXHAUST     Present        OK  20230821 09:34:36
Fantray5  green  Fantray5_1      55%      EXHAUST     Present        OK  20230821 09:34:37
Fantray5  green  Fantray5_2      56%      EXHAUST     Present        OK  20230821 09:34:37
Fantray6  green  Fantray6_1      56%      EXHAUST     Present        OK  20230821 09:34:37
Fantray6  green  Fantray6_2      56%      EXHAUST     Present        OK  20230821 09:34:37
     N/A    N/A   PSU1_FAN1      45%      EXHAUST     Present        OK  20230821 09:34:37
     N/A    N/A   PSU2_FAN1       0%      EXHAUST     Present    Not OK  20230821 09:34:37
S9301-32DB
root@sonic:~# pddf_psuutil status
PSU    Status
-----  -----------
PSU1   NOT PRESENT
PSU2   OK
root@sonic:~# pddf_thermalutil gettemp
Temp Sensor       Value
----------------  -------
Temp_CPU_PECI     temp1  +41.0 C (high = +95.0 C, crit = +100.0 C)
Temp_CPU_ENV      temp1  +38.0 C (high = +85.0 C, crit = +90.0 C)
Temp_CPU_ENV2     temp1  +33.0 C (high = +70.0 C, crit = +80.0 C)
Temp_MAC_ENV      temp1  +32.0 C (high = +85.0 C, crit = +90.0 C)
Temp_MAC_DIE      temp1  +36.0 C (high = +100.0 C, crit = +105.0 C)
Temp_CAGE         temp1  +31.0 C (high = +85.0 C, crit = +90.0 C)
Temp_PSU_CONNTOR  temp1  +33.0 C (high = +65.0 C, crit = +70.0 C)
root@sonic:~# pddf_fanutil status
FAN         Status
----------  --------
Fantray1_1  OK
Fantray1_2  OK
Fantray2_1  OK
Fantray2_2  OK
Fantray3_1  OK
Fantray3_2  OK
Fantray4_1  OK
Fantray4_2  OK
Fantray5_1  OK
Fantray5_2  OK
Fantray6_1  OK
Fantray6_2  OK
root@sonic:~# pddf_ledutil getstatusled SYS_LED
green
root@sonic:~# show platform fan
  Drawer    LED         FAN    Speed    Direction     Presence    Status          Timestamp
--------  -----  ----------  -------  -----------  -----------  --------  -----------------
Fantray1  green  Fantray1_1      56%      EXHAUST      Present        OK  20230717 15:51:49
Fantray1  green  Fantray1_2      56%      EXHAUST      Present        OK  20230717 15:51:49
Fantray2  green  Fantray2_1      55%      EXHAUST      Present        OK  20230717 15:51:52
Fantray2  green  Fantray2_2      56%      EXHAUST      Present        OK  20230717 15:51:59
Fantray3  green  Fantray3_1      55%      EXHAUST      Present        OK  20230717 15:51:59
Fantray3  green  Fantray3_2      56%      EXHAUST      Present        OK  20230717 15:51:59
Fantray4  green  Fantray4_1      56%      EXHAUST      Present        OK  20230717 15:51:59
Fantray4  green  Fantray4_2      58%      EXHAUST      Present        OK  20230717 15:51:59
Fantray5  green  Fantray5_1      55%      EXHAUST      Present        OK  20230717 15:51:59
Fantray5  green  Fantray5_2      56%      EXHAUST      Present        OK  20230717 15:51:59
Fantray6  green  Fantray6_1      56%      EXHAUST      Present        OK  20230717 15:51:59
Fantray6  green  Fantray6_2      57%      EXHAUST      Present        OK  20230717 15:51:59
     N/A    N/A   PSU1_FAN1      N/A          N/A  Not Present       N/A  20230717 15:51:59
     N/A    N/A   PSU2_FAN1      44%      EXHAUST      Present        OK  20230717 15:52:00
root@sonic:~# sfputil show presence 
Port         Presence
-----------  -----------
Ethernet0    Not present
Ethernet8    Not present
Ethernet16   Not present
Ethernet24   Not present
Ethernet32   Not present
Ethernet40   Not present
Ethernet48   Not present
Ethernet56   Not present
Ethernet64   Not present
Ethernet72   Not present
Ethernet80   Present
Ethernet88   Not present
Ethernet96   Present
Ethernet104  Not present
Ethernet112  Not present
Ethernet120  Not present
Ethernet128  Not present
Ethernet136  Not present
Ethernet144  Not present
Ethernet152  Present
Ethernet160  Not present
Ethernet168  Not present
Ethernet176  Not present
Ethernet184  Present
Ethernet192  Not present
Ethernet200  Not present
Ethernet208  Not present
Ethernet216  Not present
Ethernet224  Not present
Ethernet232  Not present
Ethernet240  Not present
Ethernet248  Not present
S9110-32X
root@sonic:~# pddf_psuutil status
PSU    Status
-----  --------
PSU1   NOT OK
PSU2   OK
root@sonic:~# pddf_thermalutil gettemp
Temp Sensor       Value
----------------  -------
TEMP_MAC          temp1  +32.0 C (high = +95.0 C, crit = +105.0 C)
TEMP_ENV_MACCASE  temp1  +29.0 C (high = +75.0 C, crit = +85.0 C)
TEMP_ENV_SSDCASE  temp1  +29.0 C (high = +75.0 C, crit = +80.0 C)
TEMP_ENV_PSUCASE  temp1  +24.0 C (high = +70.0 C, crit = +75.0 C)
TEMP_ENV_BMC      temp1  +26.0 C (high = +80.0 C, crit = +85.0 C)
TEMP_PSU0_TEMP1   temp1  +24.0 C (high = +65.0 C, crit = +70.0 C)
TEMP_PSU1_TEMP1   temp1  +25.0 C (high = +65.0 C, crit = +70.0 C)
root@sonic:~# pddf_fanutil status
FAN         Status
----------  --------
Fantray1_1  OK
Fantray1_2  OK
Fantray2_1  OK
Fantray2_2  OK
Fantray3_1  OK
Fantray3_2  OK
Fantray4_1  OK
Fantray4_2  OK
root@sonic:~# pddf_ledutil getstatusled SYS_LED
green
root@sonic:~# show interfaces transceiver presence
Port         Presence
-----------  -----------
Ethernet0    Present
Ethernet4    Not present
Ethernet8    Not present
Ethernet12   Not present
Ethernet16   Not present
Ethernet20   Not present
Ethernet24   Not present
Ethernet28   Present
Ethernet32   Not present
Ethernet36   Not present
Ethernet40   Not present
Ethernet44   Not present
Ethernet48   Not present
Ethernet52   Not present
Ethernet56   Not present
Ethernet60   Not present
Ethernet64   Not present
Ethernet68   Not present
Ethernet72   Not present
Ethernet76   Not present
Ethernet80   Not present
Ethernet84   Not present
Ethernet88   Not present
Ethernet92   Not present
Ethernet96   Not present
Ethernet100  Not present
Ethernet104  Not present
Ethernet108  Not present
Ethernet112  Not present
Ethernet116  Not present
Ethernet120  Not present
Ethernet124  Not present
Ethernet128  Not present
root@sonic:~# show platform fan
  Drawer    LED         FAN    Speed    Direction    Presence    Status          Timestamp
--------  -----  ----------  -------  -----------  ----------  --------  -----------------
Fantray1  green  Fantray1_1      56%      EXHAUST     Present        OK  20231227 04:56:46
Fantray1  green  Fantray1_2      58%      EXHAUST     Present        OK  20231227 04:56:47
Fantray2  green  Fantray2_1      57%      EXHAUST     Present        OK  20231227 04:56:47
Fantray2  green  Fantray2_2      58%      EXHAUST     Present        OK  20231227 04:56:47
Fantray3  green  Fantray3_1      57%      EXHAUST     Present        OK  20231227 04:56:47
Fantray3  green  Fantray3_2      56%      EXHAUST     Present        OK  20231227 04:56:47
Fantray4  green  Fantray4_1      58%      EXHAUST     Present        OK  20231227 04:56:47
Fantray4  green  Fantray4_2      57%      EXHAUST     Present        OK  20231227 04:56:47
     N/A    N/A   PSU1_FAN1       0%      EXHAUST     Present    Not OK  20231227 04:56:48
     N/A    N/A   PSU2_FAN1      26%      EXHAUST     Present        OK  20231227 04:56:48
S8901-54XC
PSU    Status
-----  --------
PSU1   OK
PSU2   OK
Temp Sensor       Value
----------------  -------
TEMP_MAC          temp1	 +39.0 C (high = +95.0 C, crit = +105.0 C)
TEMP_ENV_MACCASE  temp1	 +32.0 C (high = +80.0 C, crit = +85.0 C)
TEMP_ENV_PSUCASE  temp1	 +28.0 C (high = +75.0 C, crit = +80.0 C)
TEMP_ENV_FANCONN  temp1	 +31.0 C (high = +75.0 C, crit = +80.0 C)
TEMP_ENV_FANCARD  temp1	 +30.0 C (high = +75.0 C, crit = +80.0 C)
TEMP_ENV_BMC      temp1	 +33.0 C (high = +80.0 C, crit = +85.0 C)
TEMP_PSU0_TEMP1   temp1	 +34.0 C (high = +65.0 C, crit = +70.0 C)
TEMP_PSU1_TEMP1   temp1	 +32.0 C (high = +65.0 C, crit = +70.0 C)
FAN         Status
----------  --------
Fantray1_1  OK
Fantray2_1  OK
Fantray3_1  OK
Fantray4_1  OK
Fantray5_1  OK
green
Port        Presence
----------  -----------
Ethernet0   Present
Ethernet1   Present
Ethernet2   Present
Ethernet3   Present
Ethernet4   Not present
Ethernet5   Not present
Ethernet6   Not present
Ethernet7   Not present
Ethernet8   Not present
Ethernet9   Not present
Ethernet10  Not present
Ethernet11  Not present
Ethernet12  Not present
Ethernet13  Not present
Ethernet14  Not present
Ethernet15  Not present
Ethernet16  Not present
Ethernet17  Not present
Ethernet18  Not present
Ethernet19  Not present
Ethernet20  Not present
Ethernet21  Not present
Ethernet22  Not present
Ethernet23  Not present
Ethernet24  Not present
Ethernet25  Not present
Ethernet26  Not present
Ethernet27  Not present
Ethernet28  Not present
Ethernet29  Not present
Ethernet30  Not present
Ethernet31  Not present
Ethernet32  Not present
Ethernet33  Not present
Ethernet34  Not present
Ethernet35  Not present
Ethernet36  Not present
Ethernet37  Not present
Ethernet38  Not present
Ethernet39  Not present
Ethernet40  Not present
Ethernet41  Not present
Ethernet42  Not present
Ethernet43  Not present
Ethernet44  Not present
Ethernet45  Not present
Ethernet46  Not present
Ethernet47  Not present
Ethernet48  Present
Ethernet52  Present
Ethernet56  Present
Ethernet60  Present
Ethernet64  Present
Ethernet68  Present
  Drawer    LED         FAN    Speed    Direction    Presence    Status          Timestamp
--------  -----  ----------  -------  -----------  ----------  --------  -----------------
Fantray1  green  Fantray1_1      35%      EXHAUST     Present        OK  20231222 07:34:16
Fantray2  green  Fantray2_1      35%      EXHAUST     Present        OK  20231222 07:34:16
Fantray3  green  Fantray3_1      36%      EXHAUST     Present        OK  20231222 07:34:17
Fantray4  green  Fantray4_1      35%      EXHAUST     Present        OK  20231222 07:34:17
Fantray5  green  Fantray5_1      35%      EXHAUST     Present        OK  20231222 07:34:18
     N/A    N/A   PSU1_FAN1      58%      EXHAUST     Present        OK  20231222 07:34:19
     N/A    N/A   PSU2_FAN1      57%      EXHAUST     Present        OK  20231222 07:34:19

S7801-54XS
PSU    Status
-----  --------
PSU1   OK
PSU2   OK
Temp Sensor       Value
----------------  -------
TEMP_MAC          temp1	 +35.0 C (high = +95.0 C, crit = +105.0 C)
TEMP_ENV_MACCASE  temp1	 +30.0 C (high = +80.0 C, crit = +85.0 C)
TEMP_ENV_PSUCASE  temp1	 +28.0 C (high = +75.0 C, crit = +80.0 C)
TEMP_ENV_FANCONN  temp1	 +29.0 C (high = +75.0 C, crit = +80.0 C)
TEMP_ENV_FANCARD  temp1	 +29.0 C (high = +75.0 C, crit = +80.0 C)
TEMP_ENV_BMC      temp1	 +30.0 C (high = +80.0 C, crit = +85.0 C)
TEMP_PSU0_TEMP1   temp1	 +32.0 C (high = +65.0 C, crit = +70.0 C)
TEMP_PSU1_TEMP1   temp1	 +32.0 C (high = +65.0 C, crit = +70.0 C)
FAN         Status
----------  --------
Fantray1_1  OK
Fantray2_1  OK
Fantray3_1  OK
Fantray4_1  OK
Fantray5_1  OK
green
Port        Presence
----------  -----------
Ethernet0   Present
Ethernet1   Present
Ethernet2   Not present
Ethernet3   Not present
Ethernet4   Not present
Ethernet5   Not present
Ethernet6   Not present
Ethernet7   Not present
Ethernet8   Not present
Ethernet9   Not present
Ethernet10  Not present
Ethernet11  Not present
Ethernet12  Not present
Ethernet13  Not present
Ethernet14  Not present
Ethernet15  Not present
Ethernet16  Not present
Ethernet17  Not present
Ethernet18  Not present
Ethernet19  Not present
Ethernet20  Not present
Ethernet21  Not present
Ethernet22  Not present
Ethernet23  Not present
Ethernet24  Not present
Ethernet25  Not present
Ethernet26  Not present
Ethernet27  Not present
Ethernet28  Not present
Ethernet29  Not present
Ethernet30  Not present
Ethernet31  Not present
Ethernet32  Not present
Ethernet33  Not present
Ethernet34  Not present
Ethernet35  Not present
Ethernet36  Not present
Ethernet37  Not present
Ethernet38  Not present
Ethernet39  Not present
Ethernet40  Not present
Ethernet41  Not present
Ethernet42  Not present
Ethernet43  Not present
Ethernet44  Not present
Ethernet45  Not present
Ethernet46  Not present
Ethernet47  Not present
Ethernet48  Present
Ethernet52  Present
Ethernet56  Not present
Ethernet60  Not present
Ethernet64  Not present
Ethernet68  Not present
  Drawer    LED         FAN    Speed    Direction    Presence    Status          Timestamp
--------  -----  ----------  -------  -----------  ----------  --------  -----------------
Fantray1  green  Fantray1_1      68%      EXHAUST     Present        OK  20231222 15:46:40
Fantray2  green  Fantray2_1      68%      EXHAUST     Present        OK  20231222 15:46:40
Fantray3  green  Fantray3_1      68%      EXHAUST     Present        OK  20231222 15:46:40
Fantray4  green  Fantray4_1      68%      EXHAUST     Present        OK  20231222 15:46:40
Fantray5  green  Fantray5_1      68%      EXHAUST     Present        OK  20231222 15:46:40
     N/A    N/A   PSU1_FAN1      41%      EXHAUST     Present        OK  20231222 15:46:41
     N/A    N/A   PSU2_FAN1      43%      EXHAUST     Present        OK  20231222 15:46:41

S6301-56ST
root@sonic:~# pddf_psuutil status
PSU    Status
-----  --------
PSU1   OK
PSU2   OK
root@sonic:~# pddf_thermalutil gettemp
Temp Sensor    Label              Value
-------------  -----------------  -------
Temp_FAN1      tmp75-i2c-3-49     temp1  +30.4 C (high = +80.0 C)
Temp_FAN2      tmp75-i2c-3-4a     temp1  +30.0 C (high = +80.0 C)
Temp_PSUDB     ucd90124-i2c-5-34  temp1  +33.4 C (high = +55.0 C)
Temp_MAC       ucd90124-i2c-5-34  temp1  +42.2 C (high = +85.0 C)
Temp_INLET     ucd90124-i2c-5-34  temp1  +34.1 C (high = +65.0 C)
root@sonic:~# pddf_fanutil status
FAN         Status
----------  --------
Fantray1_1  OK
Fantray2_1  OK
root@sonic:~# pddf_ledutil getstatusled SYS_LED
green_blink
root@sonic:~# show interfaces transceiver presence
Port        Presence
----------  -----------
Ethernet0   Not present
Ethernet1   Not present
Ethernet2   Not present
Ethernet3   Not present
Ethernet4   Not present
Ethernet5   Not present
Ethernet6   Not present
Ethernet7   Not present
Ethernet8   Not present
Ethernet9   Not present
Ethernet10  Not present
Ethernet11  Not present
Ethernet12  Not present
Ethernet13  Not present
Ethernet14  Not present
Ethernet15  Not present
Ethernet16  Not present
Ethernet17  Not present
Ethernet18  Not present
Ethernet19  Not present
Ethernet20  Not present
Ethernet21  Not present
Ethernet22  Not present
Ethernet23  Not present
Ethernet24  Not present
Ethernet25  Not present
Ethernet26  Not present
Ethernet27  Not present
Ethernet28  Not present
Ethernet29  Not present
Ethernet30  Not present
Ethernet31  Not present
Ethernet32  Not present
Ethernet33  Not present
Ethernet34  Not present
Ethernet35  Not present
Ethernet36  Not present
Ethernet37  Not present
Ethernet38  Not present
Ethernet39  Not present
Ethernet40  Not present
Ethernet41  Not present
Ethernet42  Not present
Ethernet43  Not present
Ethernet44  Not present
Ethernet45  Not present
Ethernet46  Not present
Ethernet47  Not present
Ethernet48  Present
Ethernet49  Not present
Ethernet50  Present
Ethernet51  Not present
Ethernet52  Present
Ethernet53  Not present
Ethernet54  Not present
Ethernet55  Present
root@sonic:~# show platform fan
  Drawer    LED         FAN    Speed    Direction    Presence    Status          Timestamp
--------  -----  ----------  -------  -----------  ----------  --------  -----------------
Fantray1  green  Fantray1_1      21%      EXHAUST     Present        OK  20231227 07:24:44
Fantray2  green  Fantray2_1      20%      EXHAUST     Present        OK  20231227 07:24:44
     N/A    N/A   PSU1_FAN1      41%       INTAKE     Present        OK  20231227 07:24:44
     N/A    N/A   PSU2_FAN1      44%      EXHAUST     Present        OK  20231227 07:24:44
S9180-32X
PSU    Status
-----  --------
PSU1   OK
PSU2   OK
                Temp Sensor     Value
--------------  --------------  -------
Temp_CPU_BOARD  tmp75-i2c-0-4f  temp1    +36.2 C (crit = +70.0 C)
Temp_BMC        temp1    +31.0 C (high = +65.0 C, crit = +75.0 C)
Temp_MAC        temp1    +52.0 C (high = +95.0 C, crit = +105.0 C)
Temp_MAC_Front  temp1    +34.0 C (high = +65.0 C, crit = +75.0 C)
Temp_MAC_Rear   temp1    +36.0 C (high = +65.0 C, crit = +75.0 C)
Temp_PSU1_AMB   temp1    +36.0 C (high = +60.0 C, crit = +65.0 C)
Temp_PSU1_HS    temp1    +41.0 C (high = +95.0 C, crit = +100.0 C)
Temp_PSU2_AMB   temp1    +41.0 C (high = +60.0 C, crit = +65.0 C)
Temp_PSU2_HS    temp1    +39.0 C (high = +95.0 C, crit = +100.0 C)
FAN         Status
----------  --------
Fantray1_1  OK
Fantray1_2  OK
Fantray2_1  OK
Fantray2_2  OK
Fantray3_1  OK
Fantray3_2  OK
Fantray4_1  OK
Fantray4_2  OK
green
Port         Presence
-----------  -----------
Ethernet0    Present
Ethernet4    Present
Ethernet8    Present
Ethernet12   Not present
Ethernet16   Present
Ethernet20   Present
Ethernet24   Not present
Ethernet28   Not present
Ethernet32   Not present
Ethernet36   Not present
Ethernet40   Not present
Ethernet44   Not present
Ethernet48   Not present
Ethernet52   Not present
Ethernet56   Not present
Ethernet60   Not present
Ethernet64   Not present
Ethernet68   Not present
Ethernet72   Not present
Ethernet76   Not present
Ethernet80   Not present
Ethernet84   Not present
Ethernet88   Not present
Ethernet92   Not present
Ethernet96   Not present
Ethernet100  Not present
Ethernet104  Not present
Ethernet108  Not present
Ethernet112  Not present
Ethernet116  Not present
Ethernet120  Not present
Ethernet124  Not present
Ethernet128  Present
Ethernet129  Present
  Drawer    LED         FAN    Speed    Direction    Presence    Status          Timestamp
--------  -----  ----------  -------  -----------  ----------  --------  -----------------
Fantray1  green  Fantray1_1      50%      exhaust     Present        OK  20231226 06:13:11
Fantray1  green  Fantray1_2      50%      exhaust     Present        OK  20231226 06:13:11
Fantray2  green  Fantray2_1      50%      exhaust     Present        OK  20231226 06:13:11
Fantray2  green  Fantray2_2      50%      exhaust     Present        OK  20231226 06:13:11
Fantray3  green  Fantray3_1      50%      exhaust     Present        OK  20231226 06:13:11
Fantray3  green  Fantray3_2      51%      exhaust     Present        OK  20231226 06:13:11
Fantray4  green  Fantray4_1      50%      exhaust     Present        OK  20231226 06:13:15
Fantray4  green  Fantray4_2      51%      exhaust     Present        OK  20231226 06:13:15
     N/A    N/A   PSU1_FAN1      13%      exhaust     Present        OK  20231226 06:13:15
     N/A    N/A   PSU2_FAN1      27%      exhaust     Present        OK  20231226 06:13:15

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

  • 202305

Description for the changelog

Link to config_db schema for YANG module changes

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

@nonodark nonodark changed the title [Ufispace][PDDF] Add PDDF support on Ufispace platforms. [202305][Ufispace][PDF] Add PDDF support on Ufispace platforms. Dec 27, 2023
@nonodark nonodark changed the title [202305][Ufispace][PDF] Add PDDF support on Ufispace platforms. [202305][Ufispace][PDDF] Add PDDF support on Ufispace platforms. Dec 27, 2023
* S9300-32D
* S9301-32D
* S9301-32DB
* S9110-32X
* S8901-54XC
* S7801-54XS
* S6301-56ST
* S9180-32X

Signed-off-by: nonodark <ef67891@yahoo.com.tw>
@nonodark nonodark marked this pull request as ready for review January 2, 2024 00:13
@nonodark nonodark requested a review from lguohan as a code owner January 2, 2024 00:13
@nonodark
Copy link
Contributor Author

nonodark commented Jan 2, 2024

@FuzailBrcm pls help review, thanks

…code.

* Replace <stdbool.h> with <linux/types.h> in kernel module code.

Signed-off-by: nonodark <ef67891@yahoo.com.tw>
The return type of the remove method of i2c drivers changed from int to
void in 6.0.

Signed-off-by: nonodark <ef67891@yahoo.com.tw>
@FuzailBrcm
Copy link
Contributor

@geans-pin
Please review these changes from Ufispace

Correct the incorrect kernel version from 6.0 to 6.1.

Signed-off-by: nonodark <ef67891@yahoo.com.tw>
@geans-pin
Copy link
Contributor

Can you provide the PDE UT log ? You can reference the #11196

…1-56ST.

Signed-off-by: nonodark <ef67891@yahoo.com.tw>
Signed-off-by: nonodark <ef67891@yahoo.com.tw>
Change to get platform name from eeprom for chassis name in the following platform.
* s6301-56st
* s9300-32d
* s9301-32d
* s9301-32db

Signed-off-by: nonodark <ef67891@yahoo.com.tw>
…l in s9301-32db

* Remove blank line in port_config.ini to avoid pde test fail in s9301-32db

Signed-off-by: nonodark <ef67891@yahoo.com.tw>
Signed-off-by: nonodark <ef67891@yahoo.com.tw>
…301-32db

Signed-off-by: nonodark <ef67891@yahoo.com.tw>
@mssonicbld
Copy link
Collaborator

@StormLiangMS PR: #17625 is conflict with MS internal repo
Please complete the following PR by pushing fix commit to sonicbld/conflict_prefix/17625-fix
https://msazure.visualstudio.com/One/_git/Networking-acs-buildimage/pullrequest/9367118
Then comment "/azpw ms_conflict" to rerun PR checker.

Signed-off-by: nonodark <ef67891@yahoo.com.tw>

Squashed commit of the following:

commit 335ebaa
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Wed Jan 24 16:35:00 2024 +0800

    [submodule] Update submodule sonic-platform-daemons to the latest HEAD automatically (sonic-net#17891)
    src/sonic-platform-daemons
    ```
    * 824c20a - (HEAD -> 202305, origin/202305) Support 800G ifname in xcvrd (sonic-net#420) (20 hours ago) [Anoop Kamath]
    ```

commit 78e211c
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Sun Jan 21 16:32:35 2024 +0800

    [submodule] Update submodule sonic-utilities to the latest HEAD automatically (sonic-net#17868)
    src/sonic-utilities
    ```
    * 83a548de - (HEAD -> 202305, origin/202305) Disable Key Validation feature during sonic-installation for Cisco Platforms (sonic-net#3115) (22 hours ago) [selvipal]
    ```

commit 5d30151
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Sat Jan 20 16:32:45 2024 +0800

    [submodule] Update submodule sonic-snmpagent to the latest HEAD automatically (sonic-net#17863)
    src/sonic-snmpagent
    ```
    * 6f59d29 - (HEAD -> 202305, origin/202305) Fix SNMP dropping some of the queue counter when create_only_config_db_buffers is set to true (sonic-net#303) (sonic-net#309) (33 minutes ago) [mssonicbld]
    ```

commit 208c170
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Sat Jan 20 04:32:37 2024 +0800

    [submodule] Update submodule sonic-snmpagent to the latest HEAD automatically (sonic-net#17859)
    src/sonic-snmpagent
    ```
    * 2efaf2e - (HEAD -> 202305, origin/202305) Revert "[action] [PR:303] Fix SNMP dropping some of the queue counter when create_only_config_db_buffers is set to true (sonic-net#303)" (sonic-net#308) (4 minutes ago) [StormLiangMS]
    ```

commit 306175b
Author: Saikrishna Arcot <sarcot@microsoft.com>
Date:   Fri Jan 19 03:16:04 2024 -0800

    dhcrelay: Don't look up the ifindex for the fallback interface (sonic-net#17797) (sonic-net#17840)

    Currently, whenever isc-dhcp-relay forwards a packet upstream,
    internally, it will try to send it on a "fallback" interface. My
    understanding is that this isn't meant to be a real interface, but
    instead is basically saying to use Linux's regular routing stack to
    route the packet appropriately (rather than having isc-dhcp-relay
    specify specifically which interface to use).

    The problem is that on systems with a weak CPU, a large number of
    interfaces, and many upstream servers specified, this can introduce a
    noticeable delay in packets getting sent. The delay comes from trying to
    get the ifindex of the fallback interface. In one test case, it got to
    the point that only 2 packets could be processed per second. Because of
    this, dhcrelay will easily get backlogged and likely get to a point
    where packets get dropped in the kernel.

    Fix this by adding a check saying if we're using the fallback interface,
    then don't try to get the ifindex of this interface. We're never going
    to have an interface named this in SONiC.

    Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>

commit fc7a1ac
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Fri Jan 19 18:36:00 2024 +0800

    [submodule] Update submodule sonic-snmpagent to the latest HEAD automatically (sonic-net#17853)
    src/sonic-snmpagent
    ```
    * b0a4bcc - (HEAD -> 202305, origin/202305) Set the execute bit on sysDescr_pass.py (sonic-net#306) (22 hours ago) [Andre Kostur]
    ```

commit 30b1f05
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Fri Jan 19 16:34:55 2024 +0800

    [submodule] Update submodule linkmgrd to the latest HEAD automatically (sonic-net#17851)
    src/linkmgrd
    ```
    * f5e9b54 - (HEAD -> 202305, origin/202305) [CodeQL] fix unmet build dependency (sonic-net#222) (10 hours ago) [Jing Zhang]
    * 2282cc5 - [active-standby] Probe the link in suspend timeout (sonic-net#235) (22 hours ago) [Longxiang Lyu]
    ```

commit 66ce739
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Fri Jan 19 16:34:46 2024 +0800

    [submodule] Update submodule sonic-utilities to the latest HEAD automatically (sonic-net#17855)
    src/sonic-utilities
    ```
    * 93c42272 - (HEAD -> 202305, origin/202305) [chassis]: Support show ip bgp summary to display without error when no external neighbors are configured on chassis LC (sonic-net#3099) (22 hours ago) [Arvindsrinivasan Lakshmi Narasimhan]
    ```

commit 076b6c3
Author: zitingguo-ms <zitingguo@microsoft.com>
Date:   Fri Jan 19 14:56:01 2024 +0800

    upgrade xgs SAI to 8.4.39.2 (sonic-net#17842)

    Why I did it
    Upgrade the xgs SAI version to 8.4.39.2 to include the following fix:

    8.4.36.0: [submodule upgrade] [SAI_BRANCH rel_ocp_sai_8_4] SID: SDK-381039 Cosq control dynamic type changes
    8.4.37.0: SID: MMU cosq control configuration with Dynamic Type Check
    8.4.38.0: [sbumodule upgrade] [CSP 0001232212][SAI_BRANCH rel_ocp_sai_8_4]back-porting SONIC-82415 to SAI 8.4
    8.4.39.0: [CSP CS00012320979] Port SONIC-81867 sai spec compliance for get SAI_SWITCH_ATTR_SWITCH_HARDWARE_INFO
    8.4.39.1: changes for phy-re-init of 40G ports for TH platforms CS00012327470
    8.4.39.2: fix capability for Hostif queue by change SET operation of SAI_HOSTIF_ATTR_QUEUE to be true
    Work item tracking
    Microsoft ADO (number only): 26491005
    How I did it
    Upgrade xgs SAI version in sai.mk file.

    How to verify it
    Run basic SONiC test using SAI release pipeline, all cases passed.
    https://dev.azure.com/mssonic/internal/_build/results?buildId=457869&view=results

commit 6d767e5
Author: abdosi <58047199+abdosi@users.noreply.github.com>
Date:   Thu Jan 18 20:53:26 2024 -0800

    [chassis] Support advertisement of Loopback0 of all LC's across all e-BGP peers in TSA mode (sonic-net#16714) (sonic-net#17837)

    What I did:
    In Chassis TSA mode Loopback0 Ip's of each LC's should be advertise through e-BGP peers of each remote LC's

    How I did:

    - Route-map policy to Advertise own/self Loopback IP to other internal iBGP peers with a community internal_community as define in constants.yml
    - Route-map policy to match on above internal_community when route is received from internal iBGP peers and set a internal tag as define in constants.yml and also delete the internal_community so we don't send to any of e-BGP peers
    - In TSA new route-map match on above internal tag and permit the route (Loopback0 IP's of remote LC's) and set the community to traffic_shift_community.
    - In TSB delete the above new route-map.

    How I verify:

    Manual Verification

    UT updated.
    sonic-mgmt PR: sonic-net/sonic-mgmt#10239

    Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>

commit 898f126
Author: Baorong Liu <baorliu@cisco.com>
Date:   Thu Jan 18 04:08:29 2024 -0800

    [staticroutebfd]double commit PR-16450, change bfd to singlehop (sonic-net#17549)

    Why I did it
    double commit PR-16450 because of cherry pick conflict for PR#202305

    Work item tracking
    Microsoft ADO (number only):
    How I did it
    How to verify it

commit d81780b
Author: ganglv <88995770+ganglyu@users.noreply.github.com>
Date:   Thu Jan 18 20:07:40 2024 +0800

    Fix host service for 202305 branch (sonic-net#17781)

    Why I did it
    When we disable telemetry.service, sonic-hostservice will not start. And root cause is sonic-hostservice is only wanted by telemetry.service.

    Work item tracking
    Microsoft ADO (number only):
    How I did it
    Add dependency for gnmi.service.

    How to verify it
    Disable telemetry.service and build new image, and then check sonic-hostservice with new image.

commit b99c46e
Author: Liu Shilong <shilongliu@microsoft.com>
Date:   Thu Jan 18 20:05:04 2024 +0800

    [build] Add gpg keys for sonic-slave-bullseye in arm64 cross build on amd64. (sonic-net#17182) (sonic-net#17828)

    Fix sonic-net#16204

    Microsoft ADO (number only): 25746782

    How I did it
    multiarch/debian-debootstrap:arm64-bullseye is too old.
    It needs to add some gpg keys before 'apt-get update'

commit f460347
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Thu Jan 18 16:34:00 2024 +0800

    [submodule] Update submodule sonic-swss to the latest HEAD automatically (sonic-net#17821)
    src/sonic-swss
    ```
    * ac94f0b7 - (HEAD -> 202305, origin/202305) [202305][routeorch] Fixing bug with multiple routes pointing to nhg (sonic-net#3002) (2 hours ago) [Nikola Dancejic]
    ```

commit ed71c12
Author: abdosi <58047199+abdosi@users.noreply.github.com>
Date:   Fri Jul 7 11:10:35 2023 -0700

    Enable BFD for Static Route for chassis-packet. (sonic-net#15383)

    *What I did:
    Enable BFD for Static Route for chassis-packet. This will trigger the use of the feature as defined in here: sonic-net#13789

    Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>

commit aa01630
Author: abdosi <58047199+abdosi@users.noreply.github.com>
Date:   Tue Aug 29 13:41:01 2023 -0700

    [build]: Added flag in sonic_version.yml to see if image is secured or non-secured (sonic-net#16191)

    What I did:

    Added flag in sonic_version.yml to see if compiled image is secured or non-secured. This is done using build/compile time environmental variable SECURE_UPGRADE_MODE as define in HLD: https://github.com/sonic-net/SONiC/blob/master/doc/secure_boot/hld_secure_boot.md

    This flag does not provide the runtime status of whether the image has booted securely or not. It's possible that compile time signed image (secured image) can boot on non secure platform.

    Why I did:
    Flag can be used for manual check or by the test case.

    ADO: 24319390

    How I verify:
    Manual Verification

    ---
    build_version: 'master-16191.346262-cdc5e72a3'
    debian_version: '11.7'
    kernel_version: '5.10.0-18-2-amd64'
    asic_type: broadcom
    asic_subtype: 'broadcom'
    commit_id: 'cdc5e72a3'
    branch: 'master-16191'
    release: 'none'
    build_date: Fri Aug 25 03:15:45 UTC 2023
    build_number: 346262
    built_by: AzDevOps@vmss-soni001UR5
    libswsscommon: 1.0.0
    sonic_utilities: 1.2
    sonic_os_version: 11
    secure_boot_image: 'no'

    Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>

commit 9f81d9d
Author: snider-nokia <76123698+snider-nokia@users.noreply.github.com>
Date:   Mon Jan 8 14:38:46 2024 -0500

    [Nokia][sonic-platform] Update Nokia sonic-platform submodule and device data (sonic-net#17378)

    These changes, in conjunction with NDK version >= 22.9.17 address the thermal logging issues discussed at Nokia-ION/ndk#27. While the changes contained at this PR do not require coupling to NDK version >= 22.9.17, thermal logging enhancements will not be available without updated NDK >= 22.9.17. Thus, coupling with NDK >=22.9.17 is preferred and recommended.

    Why I did it
    To address thermal logging deficiencies.

    Work item tracking
    Microsoft ADO (number only): 26365734
    How I did it
    The following changes are included:

    Threshold configuration values are provided in the associated device data .json files. There is also a change included to better handle the condition where an SFP module read fails.

    Modify the module.py reboot to support reboot linecard from Supervisor

     - Modify reboot to call _reboot_imm for single IMM card reboot
     - Add log to the ndk_cmd to log the operation of "reboot-linecard" and "shutdown/satrtup the sfm"
    Add new nokia_cmd set command and modify show ndk-status output

     - Add a new function reboot_imm() to nokia_common.py to support reboot a single IMM slot from CPM
     - Added new command: nokia_cmd set reboot-linecard <slot> [forece] for CPM
     - Append a new column "RebootStatus" at the end of output of "nokia_cmd show ndk-status"
     - Provide ability for IMM to disable all transceiver module TX at reboot time
     - Remove defunct xcvr-resync service

commit 2f8630c
Author: Marty Y. Lok <76118573+mlok-nokia@users.noreply.github.com>
Date:   Mon Jan 8 14:39:30 2024 -0500

    [Nokia-IXR7250E] Modify the platform_reboot on the IXR7250E for PMON API reboot and Disable all SFPs (sonic-net#17483)

    Why I did it
    When Supervisor card is rebooted by using PMON API, it takes about 90 seconds to trigger the shutdown in down path. At this time linecards have been up. This delays linecards database initialization which is trying to PING/PONG the database-chassis. To address this issue, we modified the NDK to use the system call with "sudo reboot" when the request is from PMON API on Supervisor case. The NDK version is 22.9.20 and greater. This new NDK requires this modifcaiton of platform_reboot to work with.

    Work item tracking
    Microsoft ADO (number only): 26365734
    How I did it
    Modify the platform_reboot In Supervisor not to reboot all IMMs since it has been done in the function reboot() in module.py. Also handle the reboot-cause.txt for on the Supervisor when the reboot is request from PMON API.
    Modify the Nokia platform specific platform_reboot in linecard to disable all SPFs.
    This PR works with NDK version 22.9.20 and above

    Signed-off-by: mlok <marty.lok@nokia.com>

commit 6926171
Author: Liu Shilong <shilongliu@microsoft.com>
Date:   Mon Jan 15 14:59:21 2024 +0800

    [build] Fix a bash script some times called by sh issue. (sonic-net#17761)

    Why I did it
    Fix a bug that sometimes the script runs in sh not bash.

    Work item tracking
    Microsoft ADO (number only): 26297955
    How I did it

commit a262306
Author: vdahiya12 <67608553+vdahiya12@users.noreply.github.com>
Date:   Tue Jan 16 11:34:19 2024 -0800

    [Arista] Update config.bcm of 7060_cx32s for handling 40g optics with unreliable los settings (sonic-net#17768)

    For 40G optics there is SAI handling of T0 facing ports to be set with SR4 type and unreliable los set for a fixed set of ports. For this property to be invoked the requirement is set
    phy_unlos_msft=1 in config.bcm.
    This change is to meet the requirement and once this property is set, the los/interface type settings is applied by SAI on the required ports.

    Why I did it
    For Arista-7060CX-32S-Q32 T1, 40G ports RX_ERR minimalization during connected device reboot
    can be achieved by turning on Unreliable LOS and SR4 media_type for all ports which are connected to T0.

    The property phy_unlos_msft=1 is to exclusively enable this property.

    Microsoft ADO: 25941176

    How I did it
    Changes in SAI and turning on property

    How to verify it
    Ran the changes on a testbed and verified configurations are as intended.

    with property

    admin@sonic2:~$ bcmcmd "phy diag xe8 dsc config" | grep -C 2 "LOS"
    Brdfe_on                    = 0
    Media Type                  = 2
    Unreliable LOS              = 1
    Scrambling Disable          = 0
    Lane Config from PCS        = 0

    without property

    admin@sonic:~$ bcmcmd "phy diag xe8 dsc config" | grep -C 2 "LOS"
    Brdfe_on                    = 0
    Media Type                  = 0
    Unreliable LOS              = 0
    Scrambling Disable          = 0
    Lane Config from PCS        = 0

    Signed-off-by: vaibhav-dahiya <vdahiya@microsoft.com>

commit 245f337
Author: Feng-msft <fenpan@microsoft.com>
Date:   Fri Jan 12 10:51:48 2024 +0800

    Fix dialout build flag issue. (sonic-net#17715)
    Fix ENABLE_DIALOUT flag issue.
    - Microsoft ADO **(number only)**: 21326000
    Update Makefile.work and add debug string.
    ![image](https://github.com/sonic-net/sonic-buildimage/assets/97083744/960d75d1-618c-4734-acb5-7a32a28c262b)

commit b53a890
Author: Liping Xu <108326363+lipxu@users.noreply.github.com>
Date:   Fri Jan 12 15:26:06 2024 +0800

    disable restapi for leafRouter in slim image (sonic-net#17713)

    Why I did it
    For some devices with small memory, after upgrading to the latest image, the available memory is not enough.

    Work item tracking
    Microsoft ADO (number only):
    26324242
    How I did it
    Disable restapi feature for LeafRouter which with slim image.

    How to verify it
    verified on 7050qx T1 (slim image), restapi disabled
    verified on 7050qx T0 (slim image), restapi enabled
    verified on 7260 T1 (normal image), restapi enabled

commit 8561fa6
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Fri Jan 12 16:34:35 2024 +0800

    [submodule] Update submodule sonic-utilities to the latest HEAD automatically (sonic-net#17757)
    src/sonic-utilities
    ```
    * 651a80b1 - (HEAD -> 202305, origin/202305) Modify teamd retry count script to base BGP status on default BGP status (sonic-net#3069) (22 hours ago) [Saikrishna Arcot]
    ```

commit 9f66a7d
Author: Lawrence Lee <lawlee@microsoft.com>
Date:   Wed Jan 10 14:40:15 2024 -0800

    add timeout to ping6 command (sonic-net#17729)

    Signed-off-by: Lawrence Lee <lawlee@microsoft.com>

commit 13aa19a
Author: Nazarii Hnydyn <nazariig@nvidia.com>
Date:   Wed Jan 10 15:15:55 2024 +0200

    [swss/syncd]: Remove dependency on interfaces-config.service (sonic-net#17649)

    Signed-off-by: Nazarii Hnydyn nazariig@nvidia.com

    This improvement does not bring any warm-reboot degradation, since the database availability (tcp/ip access over the loopback interface) was fixed by these PRs:

    Re-add 127.0.0.1/8 when bringing down the interfaces sonic-net#15080
    Fix potentially not having any loopback address on lo interface sonic-net#16490
    Why I did it
    Removed dependency on interfaces-config.service to speed up the boot, because interfaces-config.service takes a lot of time on init
    Work item tracking
    N/A
    How I did it
    Changed service files for swss/syncd
    How to verify it
    Boot and check swss/syncd start time comparing to interfaces-config

commit f4c15b8
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Wed Jan 10 16:34:10 2024 +0800

    [submodule] Update submodule linkmgrd to the latest HEAD automatically (sonic-net#17731)
    src/linkmgrd
    ```
    * 2f5971f - (HEAD -> 202305, origin/202305) [warmboot] use config_db connector to update mux mode config instead of CLI (sonic-net#223) (4 hours ago) [Jing Zhang]
    ```

commit a3b5ea9
Author: Nazarii Hnydyn <nazariig@nvidia.com>
Date:   Wed Jan 10 02:56:20 2024 +0200

    [sonic-cfggen]: Optimize template rendering and database access. (sonic-net#17650)

    Signed-off-by: Nazarii Hnydyn nazariig@nvidia.com

    Why I did it
    Improved switch init time
    Work item tracking
    N/A
    How I did it
    Replaced: sonic-cfggen -> sonic-db-cli
    Aggregated template list for sonic-cfggen
    How to verify it
    Run warm-reboot

commit 44bc291
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Wed Jan 10 04:32:40 2024 +0800

    [submodule] Update submodule linkmgrd to the latest HEAD automatically (sonic-net#17721)
    src/linkmgrd
    ```
    * 2089ab6 - (HEAD -> 202305, origin/202305) Exclude DbInterface in PR coverage check (sonic-net#224) (3 hours ago) [Jing Zhang]
    ```

commit 1337597
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Wed Jan 10 02:32:19 2024 +0800

    [submodule] Update submodule sonic-snmpagent to the latest HEAD automatically (sonic-net#17722)
    src/sonic-snmpagent
    ```
    * e5fd192 - (HEAD -> 202305, origin/202305) Fix SNMP dropping some of the queue counter when create_only_config_db_buffers is set to true (sonic-net#303) (9 hours ago) [DavidZagury]
    ```

commit bb23f7b
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Sat Jan 6 16:42:00 2024 +0800

    [submodule] Update submodule sonic-gnmi to the latest HEAD automatically (sonic-net#17696)

commit 7ccef97
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Sat Jan 6 03:37:17 2024 +0800

    [YANG] Enable Yang model for BGP_BBR config entry (sonic-net#17622) (sonic-net#17692)

commit 8defdfe
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Sat Jan 6 01:15:01 2024 +0800

    [submodule] Update submodule sonic-sairedis to the latest HEAD automatically (sonic-net#17691)

commit b549962
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Sat Jan 6 01:00:55 2024 +0800

    [submodule] Update submodule sonic-gnmi to the latest HEAD automatically (sonic-net#17690)

commit 02dc49a
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Fri Jan 5 15:14:43 2024 +0800

    [submodule] Update submodule linkmgrd to the latest HEAD automatically (sonic-net#17687)

commit 0d31ee8
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Fri Jan 5 02:39:03 2024 +0800

    Update TELEMETRY_CLIENT YANG model (sonic-net#16861) (sonic-net#17679)

commit df22ca9
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Fri Jan 5 01:04:39 2024 +0800

    [Arista] Remove aggregate port config files for multi-asic devices (sonic-net#16923) (sonic-net#17678)

commit ca2695f
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Fri Jan 5 00:44:19 2024 +0800

    password-hardening: Add support to disable expiration date like in Linux (PAM) (sonic-net#17426) (sonic-net#17674)

commit 388457a
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Fri Jan 5 00:35:07 2024 +0800

    [arp_update]: Flush neighbors with incorrect MAC info (sonic-net#17238) (sonic-net#17677)

commit 38c5e68
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Fri Jan 5 00:32:38 2024 +0800

    Fix can't access IPV6 address via management interface because 'default' route table does not add to route lookup issue. (sonic-net#17281) (sonic-net#17676)

commit 76d8ff0
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Thu Jan 4 23:10:28 2024 +0800

    Update backend_acl.py to specify ACL table name (sonic-net#17553) (sonic-net#17673)

commit b72a740
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Thu Jan 4 22:37:26 2024 +0800

    [docker_image_ctl.j2]: swss docker initialization improvements (sonic-net#17628) (sonic-net#17672)

commit 609b3a7
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Thu Jan 4 22:29:00 2024 +0800

    [image_config]: Update DHCP rate-limit for mgmt TOR devices (sonic-net#17630) (sonic-net#17671)

commit 8362c09
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Tue Jan 2 15:13:53 2024 +0800

    [submodule] Update submodule sonic-swss to the latest HEAD automatically (sonic-net#17642)

commit 3cb6343
Author: Stepan Blyshchak <38952541+stepanblyschak@users.noreply.github.com>
Date:   Tue Jan 2 02:59:17 2024 +0200

    [202305] Revert bgp suppress fib pending (sonic-net#17578)

    DEPENDS ON: sonic-net/sonic-swss#2997 sonic-net/sonic-utilities#3093

    What I did

    Revert the feature.

    Why I did it

    Revert bgp suppress FIB functionality due to found FRR memory consumption issues and bugs.

    How I verified it

    Basic sanity check on t1-lag, regression in progress.

commit 64ed3d5
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Mon Jan 1 10:31:33 2024 +0800

    Fix system-health hardware_checker to consume fan tolerance details (sonic-net#16689) (sonic-net#17640)

commit 98ad3b0
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Sun Dec 31 17:01:53 2023 +0800

    [submodule] Update submodule sonic-utilities to the latest HEAD automatically (sonic-net#17643)

commit 1398daa
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Sun Dec 31 15:20:25 2023 +0800

    [submodule] Update submodule sonic-platform-daemons to the latest HEAD automatically (sonic-net#17641)

commit fb83d4d
Author: mssonicbld <79238446+mssonicbld@users.noreply.github.com>
Date:   Fri Dec 29 01:07:45 2023 +0800

    [Mellanox] wait until hw-management watchdog files ready (sonic-net#17618) (sonic-net#17633)

commit a8ad19d
Author: Junchao-Mellanox <57339448+Junchao-Mellanox@users.noreply.github.com>
Date:   Thu Dec 28 20:53:49 2023 +0800

    [202305] Optimize syslog rate limit feature for fast and warm boot (sonic-net#17478)

    Backport PR sonic-net#17458 due to conflict.

    Why I did it
    Optimize syslog rate limit feature for fast and warm boot

    Work item tracking
    Microsoft ADO (number only):
    How I did it
    Optimize redis start time
    Don't render rsyslog.conf in container startup script
    Disable containercfgd by default. There is a new CLI to enable it (in another PR)
    How to verify it
    Manual test
    Regression test
@mssonicbld
Copy link
Collaborator

@StormLiangMS PR: #17625 is conflict with MS internal repo
Please complete the following PR by pushing fix commit to sonicbld/conflict_prefix/17625-fix
https://msazure.visualstudio.com/One/_git/Networking-acs-buildimage/pullrequest/9367118
Then comment "/azpw ms_conflict" to rerun PR checker.

@nonodark
Copy link
Contributor Author

@geans-pin The attachments are the PDE test logs.
The PDE does not seem fully supported in Barefoot ASIC, so I cannot provide the s9180 PDE test log.
Btw, there is a code conflict, as mentioned by @mssonicbld , which is from the MS internal repo, so I can't fix it.

@StormLiangMS , please help fix the MS code conflict issue as mentioned by @mssonicbld

Thanks

s6301-56st-202305-pde.txt
s7801-54xs-202305-pde.txt
s8901-54xc-202305-pde.txt
s9110-32x-202305-pde.txt
s9300-32d-202305-pde.txt
s9301-32d-202305-pde.txt
s9301-32db-202305-pde.txt

Copy link
Contributor

@geans-pin geans-pin left a comment

Choose a reason for hiding this comment

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

Why there are lots of other changes which are not related with Ufispace ? I thought you should only commit the changes which are related with Ufispace BSP only ..

Geans

@geans-pin
Copy link
Contributor

geans-pin commented Jan 29, 2024

@geans-pin The attachments are the PDE test logs. The PDE does not seem fully supported in Barefoot ASIC, so I cannot provide the s9180 PDE test log. Btw, there is a code conflict, as mentioned by @mssonicbld , which is from the MS internal repo, so I can't fix it.

@StormLiangMS , please help fix the MS code conflict issue as mentioned by @mssonicbld

Thanks

s6301-56st-202305-pde.txt s7801-54xs-202305-pde.txt s8901-54xc-202305-pde.txt s9110-32x-202305-pde.txt s9300-32d-202305-pde.txt s9301-32d-202305-pde.txt s9301-32db-202305-pde.txt

I see lots of PDE Test ERR from S9110 UT ... Is there any missing ?
Can you reference the UT from this PR ? => #11196

========================================================================================================== FAILURES ===========================================================================================================
_____________________________________________________________________________________________________ test_for_eeprom_sn ______________________________________________________________________________________________________

json_config_data = {'PLATFORM': {'drivers': {'CPLD': {'driver_info': {'support': 'false'}}, 'EEPROM': {'driver_info': {'name': 'x86_64_uf...pport': 'true'}, 'PSU': {'support': 'true'}, 'SYSLED': {'support': 'false'}, ...}, 'num_cplds': 3, 'num_fans': 6, ...}}
json_test_data = {'PLATFORM': {'CONFIG': {'notpermitted': {'config.bcm': []}, 'required': {'config.bcm': []}}, 'CPLD': {'CPLD1': {'vers...st'}, 'FAN2': {'direction': 'exhaust'}, 'FAN3': {'direction': 'exhaust'}, 'FAN4': {'direction': 'exhaust'}, ...}, ...}}

def test_for_eeprom_sn(json_config_data,json_test_data):
    """Test Purpose:  Verify that the SERIAL read from EEPROM is matching with the
                      test config JSON setting

       Args:
           arg1 (json): platform-<sonic_platform>-config.json
           arg2 (json): test-<sonic_platform>-config.json

       Example:
            For a system that physically supports 1 syseeprom, the SERIAL number is
            configured in the test-<sonic_platform>-config.json

           test-<sonic_platform>-config.json
           "EEPROM": {
                "mac": "00:11:22:33:44:55",
                "ser": "AABBCCDDEEFF",
                "model": "AAAA"
            },
    """
    _wrapper_init()
    for key in json_config_data:
        tst = json_test_data[key]['EEPROM']['ser']

        if platform_chassis is not None:
         val = platform_chassis.get_serial_number()

E AttributeError: 'Chassis' object has no attribute 'get_serial_number'

test_eeprom.py:150: AttributeError
______________________________________________________________________________________________ test_for_switch_port_enumeration _______________________________________________________________________________________________

def test_for_switch_port_enumeration():
    """
    Test Purpose:
        Verify the SONiC port enumeration to ensure all the SONiC ports
        are mapped to SAI ports with valid object ids

    Args:
        None
    """
  rv = load_platform_portdict()

test_sai.py:112:


def load_platform_portdict():
    global port_dict

    if port_dict is not None:
        return port_dict

    port_dict = {}
  file = open(HWSKU_PATH + '/port_config.ini', 'r')

E FileNotFoundError: [Errno 2] No such file or directory: '/usr/share/sonic/hwsku/port_config.ini'

test_sai.py:55: FileNotFoundError
_____________________________________________________________________________________________ test_for_switch_media_setting_json ______________________________________________________________________________________________

def test_for_switch_media_setting_json():
    """
    Test Purpose:  Verify preemphasis media setting json

    Args:
        None
    """
    try:
      with open(PLATFORM_PATH +'/media_settings.json') as data:

E FileNotFoundError: [Errno 2] No such file or directory: '/usr/share/sonic/platform/media_settings.json'

test_sai.py:262: FileNotFoundError

During handling of the above exception, another exception occurred:

def test_for_switch_media_setting_json():
    """
    Test Purpose:  Verify preemphasis media setting json

    Args:
        None
    """
    try:
        with open(PLATFORM_PATH +'/media_settings.json') as data:
            media_dict = json.load(data)
    except:
      assert False, "Unable to open media_settings.json"

E AssertionError: Unable to open media_settings.json
E assert False

test_sai.py:265: AssertionError
_____________________________________________________________________________________________________ test_for_sfp_number _____________________________________________________________________________________________________

json_config_data = {'PLATFORM': {'drivers': {'CPLD': {'driver_info': {'support': 'false'}}, 'EEPROM': {'driver_info': {'name': 'x86_64_uf...pport': 'true'}, 'PSU': {'support': 'true'}, 'SYSLED': {'support': 'false'}, ...}, 'num_cplds': 3, 'num_fans': 6, ...}}

def test_for_sfp_number(json_config_data):
    """Test Purpose:  Verify that the numer of SFPs reported as supported by the SFP plugin matches what the platform supports.

    Args:
        arg1 (json): platform-<sonic_platform>-config.json

    Example:
        For a system that physically supports 32 (Q)SFP transceivers

        platform-<sonic_platform>-config.json
        {
            "PLATFORM": {
                "num_sfps": 32
            }
        }
        """
    for plat in json_config_data:
      assert _wrapper_get_num_sfps() == int(json_config_data[plat]["num_sfps"])

test_sfp.py:171:


test_sfp.py:115: in _wrapper_get_num_sfps
load_platform_portdict()


def load_platform_portdict():
    global port_dict

    if port_dict is not None:
        return

    port_dict = {}
    idx = 0
  file = open(get_hwsku_path() + "/port_config.ini", "r")

E FileNotFoundError: [Errno 2] No such file or directory: '/usr/share/sonic/platform/UFISPACE-S9110-32X/port_config.ini'

test_sfp.py:62: FileNotFoundError
______________________________________________________________________________________________________ test_for_usb_ntfs ______________________________________________________________________________________________________

def test_for_usb_ntfs():
    """Test Purpose:  Verify that if the NTFS filesystem support is enable in the kernel

    Args:
        None
        """
    cmd = "cat /boot/config-* | grep ^CONFIG_NTFS_FS="
    ret = subprocess.call(cmd + " > /dev/null 2>&1", shell=True)
  assert ret == 0, "NTFS support is disabled in the kernel"

E AssertionError: NTFS support is disabled in the kernel
E assert 1 == 0

test_usb.py:52: AssertionError
____________________________________________________________________________________________________ test_for_arm_watchdog ____________________________________________________________________________________________________

json_config_data = {'PLATFORM': {'drivers': {'CPLD': {'driver_info': {'support': 'false'}}, 'EEPROM': {'driver_info': {'name': 'x86_64_uf...pport': 'true'}, 'PSU': {'support': 'true'}, 'SYSLED': {'support': 'false'}, ...}, 'num_cplds': 3, 'num_fans': 6, ...}}
json_test_data = {'PLATFORM': {'CONFIG': {'notpermitted': {'config.bcm': []}, 'required': {'config.bcm': []}}, 'CPLD': {'CPLD1': {'vers...st'}, 'FAN2': {'direction': 'exhaust'}, 'FAN3': {'direction': 'exhaust'}, 'FAN4': {'direction': 'exhaust'}, ...}, ...}}

def test_for_arm_watchdog(json_config_data, json_test_data):
    """
    Test Purpose:  Verify the watchdog can be armed

    Args:
        arg1 (json): platform-<sonic_platform>-config.json
        arg2 (json): test-<sonic_platform>-config.json

    Example:
        Test config JSON setting for Watchdog arm Timeout configured

          "WDT":
          {
            "arm":{
              "timeout":190
            }
          }
    """
    sec = json_test_data['PLATFORM']['WDT']['arm']['timeout']
    if _wrapper_arm_watchdog(sec) != WDT_COMMON_ERROR:
     assert _wrapper_watchdog_is_armed() == True, "watchdog is not active"

E AssertionError: watchdog is not active
E assert None == True
E + where None = _wrapper_watchdog_is_armed()

test_watchdog.py:103: AssertionError
_______________________________________________________________________________________________ test_for_remaining_time_changed _______________________________________________________________________________________________

def test_for_remaining_time_changed():
    """
      Test Purpose:  Verify the watchdog remaining time is keep changing ( Couting down )

      Args:
        None
    """
    Time1 = _wrapper_get_watchdog_remaining_time();
    time.sleep(5)
    Time2 = _wrapper_get_watchdog_remaining_time();
  assert Time1 != Time2, "Watchdog remaining time is the same without changed"

E AssertionError: Watchdog remaining time is the same without changed
E assert None != None

test_watchdog.py:124: AssertionError

@nonodark
Copy link
Contributor Author

Why there are lots of other changes which are not related with Ufispace ? I thought you should only commit the changes which are related with Ufispace BSP only ..

Geans

@geans-pin
I reverted c3b93: 'Sync the latest code from the 202305 branch' and merged the 202305 branch again to fix the code sync issue.

test_for_eeprom_sn

We use pddf_chassis.py and it doesn't implement get_serial_number.
Should we need to implement this function?

test_for_switch_port_enumeration/test_for_sfp_number

According to Port Guide, port_config.ini is deprecated in favor of the use of platform.json and hwsku.json

test_for_arm_watchdog/test_for_remaining_time_changed

PDDF doesn't implement the watchdog feature. Should we need to implement this function?

@liushilongbuaa
Copy link
Contributor

Hi @nonodark , you changed code in platform/barefoot. Currently barefoot's build fails on sai package.
Maybe it's better to fix barefoot sai package first.

@nonodark
Copy link
Contributor Author

nonodark commented Feb 5, 2024

Hi @nonodark , you changed code in platform/barefoot. Currently barefoot's build fails on sai package.
Maybe it's better to fix barefoot sai package first.

@liushilongbuaa Do you mean that if Barefoot's build fails on the SAI package, it's an issue with Barefoot, and we should wait for them to fix it before submitting a Barefoot platform pull request? Or do you mean that if Barefoot's build fails on the SAI package, it's an issue on the Ufispace side, and we need to try to fix it?

@liushilongbuaa
Copy link
Contributor

Hi @nonodark , you changed code in platform/barefoot. Currently barefoot's build fails on sai package.
Maybe it's better to fix barefoot sai package first.

@liushilongbuaa Do you mean that if Barefoot's build fails on the SAI package, it's an issue with Barefoot, and we should wait for them to fix it before submitting a Barefoot platform pull request? Or do you mean that if Barefoot's build fails on the SAI package, it's an issue on the Ufispace side, and we need to try to fix it?

Sorry. I thought maybe you can fix barefoot sai. It's my mistake. But currently we can't verify if it works on barefoot platform. Please take care.

@nonodark
Copy link
Contributor Author

nonodark commented Feb 5, 2024

Hi @nonodark , you changed code in platform/barefoot. Currently barefoot's build fails on sai package.
Maybe it's better to fix barefoot sai package first.

@liushilongbuaa Do you mean that if Barefoot's build fails on the SAI package, it's an issue with Barefoot, and we should wait for them to fix it before submitting a Barefoot platform pull request? Or do you mean that if Barefoot's build fails on the SAI package, it's an issue on the Ufispace side, and we need to try to fix it?

Sorry. I thought maybe you can fix barefoot sai. It's my mistake. But currently we can't verify if it works on barefoot platform. Please take care.

@liushilongbuaa It builds successfully on my end currently. Could you please provide the build failure log for us?

@liushilongbuaa
Copy link
Contributor

liushilongbuaa commented Feb 5, 2024

Hi @nonodark , you changed code in platform/barefoot. Currently barefoot's build fails on sai package.
Maybe it's better to fix barefoot sai package first.

@liushilongbuaa Do you mean that if Barefoot's build fails on the SAI package, it's an issue with Barefoot, and we should wait for them to fix it before submitting a Barefoot platform pull request? Or do you mean that if Barefoot's build fails on the SAI package, it's an issue on the Ufispace side, and we need to try to fix it?

Sorry. I thought maybe you can fix barefoot sai. It's my mistake. But currently we can't verify if it works on barefoot platform. Please take care.

@liushilongbuaa It builds successfully on my end currently. Could you please provide the build failure log for us?

https://dev.azure.com/mssonic/build/_build/results?buildId=454731&view=results
configure:18037: error: "SAI library libsai.so does not have sai_query_api_version API which is required"

@liushilongbuaa
Copy link
Contributor

/azpw ms_conflict

@nonodark
Copy link
Contributor Author

nonodark commented Feb 15, 2024

@geans-pin Happy Chinese New Year! Any comments? If everything is okay, please help me approve the PR and merge it.

@@ -0,0 +1,3 @@
depmod -a
systemctl enable pddf-platform-init.service
Copy link
Contributor

Choose a reason for hiding this comment

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

For every .postinst file, there should be a corresponding .prerm file which would 'stop' and 'disable' the services enabled in the .postinst file. The effect is particularly seen when .deb pkg installation and removal is tested.

e.g. for this .postinst file, the .prerm file should have following,

# cat broadcom/sonic-platform-modules-ufispace/debian/sonic-platform-ufispace-s9300-32d.prerm
systemctl stop pddf-platform-init.service
systemctl disable pddf-platform-init.service
# 

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@FuzailBrcm I appreciate your reminder. I have updated the .prerm file. Please help me review the code. Thanks.

Add the .prerm file to the ufispace platform below:
* S9300-32D
* S9301-32D
* S9301-32DB
* S9110-32X
* S8901-54XC
* S7801-54XS
* S6301-56ST
* S9180-32X
@mssonicbld
Copy link
Collaborator

@StormLiangMS PR: #17625 is conflict with MS internal repo
Please complete the following PR by pushing fix commit to sonicbld/conflict_prefix/17625-fix
https://msazure.visualstudio.com/One/_git/Networking-acs-buildimage/pullrequest/9537732
Then comment "/azpw ms_conflict" to rerun PR checker.

@nonodark
Copy link
Contributor Author

/azpw ms_conflict

@mssonicbld
Copy link
Collaborator

@StormLiangMS PR: #17625 is conflict with MS internal repo
Please complete the following PR by pushing fix commit to sonicbld/conflict_prefix/17625-fix
https://msazure.visualstudio.com/One/_git/Networking-acs-buildimage/pullrequest/9537732
Then comment "/azpw ms_conflict" to rerun PR checker.

@nonodark
Copy link
Contributor Author

@liushilongbuaa @StormLiangMS could you please help me resolve the MS conflict again? All reviews are done. Could you please assist me in merging it?

@liushilongbuaa
Copy link
Contributor

/azpw ms_conflict

@nonodark
Copy link
Contributor Author

@yxieca All reviews have been completed. Could you please assist me in merging it?

@nonodark
Copy link
Contributor Author

@yxieca @StormLiangMS @lguohan All reviews have been completed. If everything is fine, please assist me in merging it.

@yxieca
Copy link
Contributor

yxieca commented Feb 22, 2024

@prgeor please review this change. Thanks.

@yxieca yxieca requested a review from prgeor February 22, 2024 16:22
@nonodark
Copy link
Contributor Author

@prgeor , please help me review the code. Thanks.

1 similar comment
@nonodark
Copy link
Contributor Author

nonodark commented Mar 6, 2024

@prgeor , please help me review the code. Thanks.

@nonodark
Copy link
Contributor Author

@prgeor , please help me review the code. Thanks.

1 similar comment
@nonodark
Copy link
Contributor Author

@prgeor , please help me review the code. Thanks.

@nonodark
Copy link
Contributor Author

@yxieca , It has been one month since, and @prgeor , perhaps too busy to review, has not provided any feedback.
Please assign a new reviewer or help me merge the code. Thanks.

@nonodark
Copy link
Contributor Author

nonodark commented Apr 2, 2024

@yxieca please assign another reviewer or help me merge the code. Thanks.

@yxieca
Copy link
Contributor

yxieca commented Apr 2, 2024

@yxieca please assign another reviewer or help me merge the code. Thanks.

@nonodark , sorry that it took so long. I'll ping Prince offline. However, is this a bug fix? It looks like a feature to me. If it is a feature, then we should not take it at this stage when qualification is almost done. @StormLiangMS is the 202305 branch manager. Add him for viz.

@nonodark
Copy link
Contributor Author

nonodark commented Apr 8, 2024

@yxieca and @StormLiangMS ,it's not a feature, but rather the porting of SONiC to support Ufispace hardware platforms. This will not affect existing functionality.

@nonodark
Copy link
Contributor Author

I have merged this into the master branch, so I'll close it.

@nonodark nonodark closed this Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants