From 0fa024883d205767493a5082c44882beda7bfae9 Mon Sep 17 00:00:00 2001 From: Volodymyr Boiko <66446128+vboykox@users.noreply.github.com> Date: Wed, 26 May 2021 22:02:46 +0300 Subject: [PATCH] [platform][barefoot] Add Psu.get_name (#7717) #### Why I did it To fix the following: ``` # psuutil status Traceback (most recent call last): File "/usr/local/bin/psuutil", line 8, in sys.exit(cli()) File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 764, in __call__ return self.main(*args, **kwargs) File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 717, in main rv = self.invoke(ctx) File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1137, in invoke return _process_result(sub_ctx.command.invoke(sub_ctx)) File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 956, in invoke return ctx.invoke(self.callback, **ctx.params) File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 555, in invoke return callback(*args, **kwargs) File "/usr/local/lib/python3.7/dist-packages/psuutil/main.py", line 93, in status psu_name = psu.get_name() File "/usr/local/lib/python3.7/dist-packages/sonic_platform_base/device_base.py", line 28, in get_name raise NotImplementedError NotImplementedError ``` #### How I did it Implemented get_name Signed-off-by: Volodymyr Boyko --- .../sonic-platform-modules-bfn-montara/sonic_platform/psu.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/platform/barefoot/sonic-platform-modules-bfn-montara/sonic_platform/psu.py b/platform/barefoot/sonic-platform-modules-bfn-montara/sonic_platform/psu.py index d46cbcca54..cf9a84b2a6 100644 --- a/platform/barefoot/sonic-platform-modules-bfn-montara/sonic_platform/psu.py +++ b/platform/barefoot/sonic-platform-modules-bfn-montara/sonic_platform/psu.py @@ -41,6 +41,9 @@ class Psu(PsuBase): """ return 2 + def get_name(self): + return f"psu-{self.__index}" + def get_powergood_status(self): """ Retrieves the oprational status of power supply unit (PSU) defined