06e1a0bc14
Dependency: [PR (#12065)](https://github.com/sonic-net/sonic-buildimage/pull/12065) needs to merge first. #### Why I did it `commands` module is not protected against malicious input `getstatusoutput` is detected without a static string, uses `shell=True` #### How I did it Eliminate the use of `commands` Use `subprocess.run()`, commands in `subprorcess.run()` are totally static Fix indentation #### How to verify it Tested on DUT [dell_log.txt](https://github.com/sonic-net/sonic-buildimage/files/9561332/dell_log.txt)
100 lines
2.9 KiB
Python
100 lines
2.9 KiB
Python
#
|
|
# psuutil.py
|
|
# Platform-specific PSU status interface for SONiC
|
|
#
|
|
|
|
|
|
import logging
|
|
import sys
|
|
from sonic_py_common.general import getstatusoutput_noshell, getstatusoutput_noshell_pipe
|
|
|
|
|
|
S5296F_MAX_PSUS = 2
|
|
IPMI_PSU_DATA = ["docker", "exec", "-it", "pmon", "ipmitool", "sdr", "list"]
|
|
IPMI_PSU_DATA_DOCKER = ["ipmitool", "sdr", "list"]
|
|
PSU_PRESENCE = "PSU{0}_stat"
|
|
# Use this for older firmware
|
|
# PSU_PRESENCE="PSU{0}_prsnt"
|
|
ipmi_sdr_list = ""
|
|
|
|
|
|
try:
|
|
from sonic_psu.psu_base import PsuBase
|
|
except ImportError as e:
|
|
raise ImportError(str(e) + "- required module not found")
|
|
|
|
|
|
class PsuUtil(PsuBase):
|
|
"""Platform-specific PSUutil class"""
|
|
|
|
def __init__(self):
|
|
PsuBase.__init__(self)
|
|
|
|
def isDockerEnv(self):
|
|
num_docker = open('/proc/self/cgroup', 'r').read().count(":/docker")
|
|
if num_docker > 0:
|
|
return True
|
|
else:
|
|
return False
|
|
|
|
# Fetch a BMC register
|
|
def get_pmc_register(self, reg_name):
|
|
|
|
global ipmi_sdr_list
|
|
ipmi_cmd = IPMI_PSU_DATA
|
|
dockerenv = self.isDockerEnv()
|
|
if dockerenv == True:
|
|
ipmi_cmd = IPMI_PSU_DATA_DOCKER
|
|
|
|
status, ipmi_sdr_list = getstatusoutput_noshell(ipmi_cmd)
|
|
|
|
if status:
|
|
logging.error('Failed to execute:' + ipmi_sdr_list)
|
|
sys.exit(0)
|
|
|
|
for item in ipmi_sdr_list.split("\n"):
|
|
if reg_name in item:
|
|
output = item.strip()
|
|
|
|
if not output:
|
|
print('\nFailed to fetch: ' + reg_name + ' sensor ')
|
|
sys.exit(0)
|
|
|
|
output = output.split('|')[1]
|
|
|
|
logging.basicConfig(level=logging.DEBUG)
|
|
return output
|
|
|
|
def get_num_psus(self):
|
|
"""
|
|
Retrieves the number of PSUs available on the device
|
|
:return: An integer, the number of PSUs available on the device
|
|
"""
|
|
S5296F_MAX_PSUS = 2
|
|
return S5296F_MAX_PSUS
|
|
|
|
def get_psu_status(self, index):
|
|
"""
|
|
Retrieves the oprational status of power supply unit (PSU) defined
|
|
by index <index>
|
|
:param index: An integer, index of the PSU of which to query status
|
|
:return: Boolean, True if PSU is operating properly, False if PSU is\
|
|
faulty
|
|
"""
|
|
# Until psu_status is implemented this is hardcoded temporarily
|
|
|
|
status = 1
|
|
return status
|
|
|
|
def get_psu_presence(self, index):
|
|
"""
|
|
Retrieves the presence status of power supply unit (PSU) defined
|
|
by index <index>
|
|
:param index: An integer, index of the PSU of which to query status
|
|
:return: Boolean, True if PSU is plugged, False if not
|
|
"""
|
|
ipmi_cmd = ["ipmitool", "raw", "0x04", "0x2d", hex(0x30 + index)]
|
|
awk_cmd = ["awk", "{print substr($0,9,1)}"]
|
|
cmd_status, psu_status = getstatusoutput_noshell_pipe(ipmi_cmd, awk_cmd)
|
|
return 1 if psu_status == '1' else 0
|