X-Git-Url: https://gerrit.fd.io/r/gitweb?a=blobdiff_plain;f=resources%2Flibraries%2Fpython%2FTrace.py;h=f54ae1060ec5ea04bae372d2642711c208cb4e40;hb=a275fa0062158d712152f542b7bc9ec40b5c5f31;hp=27cc30476e005f8443eef865669ddd0f02e4f80e;hpb=8b1bead7b4b70e3ff4e7d4cb82940695d763ed2d;p=csit.git

diff --git a/resources/libraries/python/Trace.py b/resources/libraries/python/Trace.py
index 27cc30476e..f54ae1060e 100644
--- a/resources/libraries/python/Trace.py
+++ b/resources/libraries/python/Trace.py
@@ -1,4 +1,4 @@
-# Copyright (c) 2019 Cisco and/or its affiliates.
+# Copyright (c) 2021 Cisco and/or its affiliates.
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at:
@@ -17,7 +17,7 @@ from resources.libraries.python.PapiExecutor import PapiSocketExecutor
 from resources.libraries.python.topology import NodeType
 
 
-class Trace(object):
+class Trace:
     """This class provides methods to manipulate the VPP packet trace."""
 
     @staticmethod
@@ -29,13 +29,11 @@ class Trace(object):
         :type nodes: dict
         :type maximum: int
         """
-        maximum = "max {count}".format(count=maximum) if maximum is not None\
-            else ""
-
+        max_opt = f"" if maximum is None else f" max {maximum}"
         for node in nodes.values():
-            if node['type'] == NodeType.DUT:
-                PapiSocketExecutor.run_cli_cmd(
-                    node, "show trace {max}".format(max=maximum))
+            if node[u"type"] == NodeType.DUT:
+                PapiSocketExecutor.run_cli_cmd_on_all_sockets(
+                    node, f"show trace{max_opt}")
 
     @staticmethod
     def clear_packet_trace_on_all_duts(nodes):
@@ -45,5 +43,6 @@ class Trace(object):
         :type nodes: dict
         """
         for node in nodes.values():
-            if node['type'] == NodeType.DUT:
-                PapiSocketExecutor.run_cli_cmd(node, "clear trace")
+            if node[u"type"] == NodeType.DUT:
+                PapiSocketExecutor.run_cli_cmd_on_all_sockets(
+                    node, u"clear trace")