[barefoot][platform] Refactor legacy scripts (#6871)

Removed or adapted obsolete code

Signed-off-by: Volodymyr Boyko <volodymyrx.boiko@intel.com>

platform/barefoot/sonic-platform-modules-bfn-montara/scripts/bf-fancontrol

@@ -0,0 +1,3 @@
+#!/bin/bash
+
+python3 -m sonic_platform.bfn_extensions.platform_fancontrol "$@"

platform/barefoot/sonic-platform-modules-bfn-montara/scripts/bf-sfputil

@@ -1,10 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd sfputil "$@"

platform/barefoot/sonic-platform-modules-bfn-montara/scripts/eeprom

@@ -1,10 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd eeprom "$@"

platform/barefoot/sonic-platform-modules-bfn-montara/scripts/fancontrol

@@ -1,11 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd fancontrol "$@"
-

platform/barefoot/sonic-platform-modules-bfn-montara/scripts/ps_info

@@ -1,10 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd ps_info "$@"

platform/barefoot/sonic-platform-modules-bfn-montara/scripts/sensors

@@ -7,6 +7,5 @@ if [ -t 1 ] ; then
     DOCKER_EXEC_FLAGS+="t"
 fi
 
-docker exec -$DOCKER_EXEC_FLAGS syncd sensors "$@"
+python3 -m sonic_platform.bfn_extensions.platform_sensors "$@"
-
+docker exec -$DOCKER_EXEC_FLAGS pmon sensors "$@"
-

platform/barefoot/sonic-platform-modules-bfn-montara/scripts/test

@@ -1 +0,0 @@
-echo "test"

platform/barefoot/sonic-platform-modules-bfn-montara/setup.py

@@ -12,7 +12,8 @@ setup(
     maintainer_email='',
     packages=[
         'sonic_platform',
-        'sonic_platform/pltfm_mgr_rpc'
+        'sonic_platform/pltfm_mgr_rpc',
+        'sonic_platform/bfn_extensions',
     ],
     package_data = {'sonic_platform':['logging.conf']},
     classifiers=[

platform/barefoot/sonic-platform-modules-bfn-montara/sonic_platform/bfn_extensions/__init__.py

@@ -0,0 +1,4 @@
+__all__ = [
+    'platform_sensors',
+    'platform_fancontrol',
+]

platform/barefoot/sonic-platform-modules-bfn-montara/sonic_platform/bfn_extensions/platform_fancontrol.py

@@ -0,0 +1,59 @@
+import sys
+
+from sonic_platform.platform_thrift_client import thrift_try
+
+_MAX_FAN = 10
+
+def fan_speed_set(fan, percent):
+    def set_fan_speed(client):
+        return client.pltfm_mgr.pltfm_mgr_fan_speed_set(fan, percent)
+    return thrift_try(set_fan_speed)
+
+def fan_speed_info_get():
+    for fan_num in range(1, _MAX_FAN + 1):
+        def get_data(client, fan_num=fan_num):
+            return client.pltfm_mgr.pltfm_mgr_fan_info_get(fan_num)
+        fan_info = thrift_try(get_data)
+        if fan_info.fan_num == fan_num:
+            yield fan_info
+
+if __name__ == '__main__':
+    def print_usage():
+        print("Usage: platform_fancontrol.py <function> <param list>  ", file=sys.stderr)
+        print("           function: fan_speed_set <fan #> <percent>   ", file=sys.stderr)
+        print("                     fan_speed_info_get                ", file=sys.stderr)
+
+    argc = len(sys.argv)
+    if argc == 1:
+      print_usage()
+      exit(0)
+
+    if sys.argv[1] == "fan_speed_set":
+      if argc != 4:
+        print_usage()
+        exit(0)
+
+      fan = int(sys.argv[2])
+      percent = int(sys.argv[3])
+
+      if (fan > _MAX_FAN) | (fan < 0):
+        print("Invalid value for fan #.\n", file=sys.stderr)
+        print_usage()
+        exit(0)
+
+      if (percent > 100) | (percent < 0):
+        print("Invalid value for precent\n", file=sys.stderr)
+        print_usage()
+        exit(0)
+
+      fan_speed_set(fan, percent)
+      exit(0)
+
+    if sys.argv[1] == "fan_speed_info_get":
+      for fan_info in fan_speed_info_get():
+        print("fan number: %d front rpm: %d rear rpm: %d percent: %d%% " %
+            (fan_info.fan_num, fan_info.front_rpm, fan_info.rear_rpm, fan_info.percent))
+
+      exit(0)
+
+    print_usage()

platform/barefoot/sonic-platform-modules-bfn-montara/sonic_platform/bfn_extensions/platform_sensors.py

@@ -0,0 +1,26 @@
+import sys
+import codecs
+import urllib
+
+from sonic_platform.platform_thrift_client import thrift_try
+
+def platform_sensors_get(args):
+    options = ""
+    if len(args)!=0:
+        options = urllib.quote(" ".join(args))
+    def get_data(client):
+        return client.pltfm_mgr.pltfm_mgr_sensor_info_get(options)
+    raw_out = thrift_try(get_data)
+    raw_list = raw_out.split('\"')
+    if len(raw_list) >= 2:
+        sensors_out = raw_list[1]
+        sensors_out = codecs.decode(sensors_out, "unicode_escape")
+        return sensors_out
+    return None
+
+if __name__ == '__main__':
+    data = platform_sensors_get(sys.argv[1:])
+    if data:
+        print(data)
+    else:
+        print("No sensors info available", file=sys.stderr)

platform/barefoot/sonic-platform-modules-bfn-newport/scripts

@@ -0,0 +1 @@
+../sonic-platform-modules-bfn-montara/scripts

platform/barefoot/sonic-platform-modules-bfn-newport/scripts/bf-sfputil

@@ -1,10 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd sfputil "$@"

platform/barefoot/sonic-platform-modules-bfn-newport/scripts/eeprom

@@ -1,10 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd eeprom "$@"

platform/barefoot/sonic-platform-modules-bfn-newport/scripts/fancontrol

@@ -1,11 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd fancontrol "$@"
-

platform/barefoot/sonic-platform-modules-bfn-newport/scripts/ps_info

@@ -1,10 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd ps_info "$@"

platform/barefoot/sonic-platform-modules-bfn-newport/scripts/sensors

@@ -1,12 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd sensors "$@"
-
-

platform/barefoot/sonic-platform-modules-bfn-newport/scripts/test

@@ -1 +0,0 @@
-echo "test"

platform/barefoot/sonic-platform-modules-bfn/scripts

@@ -0,0 +1 @@
+../sonic-platform-modules-bfn-montara/scripts

platform/barefoot/sonic-platform-modules-bfn/scripts/bf-sfputil

@@ -1,10 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd sfputil "$@"

platform/barefoot/sonic-platform-modules-bfn/scripts/eeprom

@@ -1,10 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd eeprom "$@"

platform/barefoot/sonic-platform-modules-bfn/scripts/fancontrol

@@ -1,11 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd fancontrol "$@"
-

platform/barefoot/sonic-platform-modules-bfn/scripts/ps_info

@@ -1,10 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd ps_info "$@"

platform/barefoot/sonic-platform-modules-bfn/scripts/sensors

@@ -1,12 +0,0 @@
-#!/bin/bash
-
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS syncd sensors "$@"
-
-

platform/barefoot/sonic-platform-modules-bfn/scripts/test

@@ -1 +0,0 @@
-echo "test"