[V1] tests/telemetry: update file_prefix according to dpdk changes

Message ID 20230210065125.9955-1-linglix.chen@intel.com (mailing list archive)
State Accepted
Headers
Series [V1] tests/telemetry: update file_prefix according to dpdk changes |

Checks

Context Check Description
ci/Intel-dts-format-test success Testing OK
ci/Intel-dts-pylama-test success Testing OK
ci/Intel-dts-suite-test success Testing OK

Commit Message

Lingli Chen Feb. 10, 2023, 6:51 a.m. UTC
  according to dpdk commit a15fc792cf (usertools/telemetry: add file prefix argument)
update file_prefix in telemetry client

Signed-off-by: Lingli Chen <linglix.chen@intel.com>
---
 tests/TestSuite_telemetry.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

Comments

Tu, Lijuan March 7, 2023, 3:36 a.m. UTC | #1
On Fri, 10 Feb 2023 01:51:25 -0500, Lingli Chen <linglix.chen@intel.com> wrote:
> according to dpdk commit a15fc792cf (usertools/telemetry: add file prefix argument)
> update file_prefix in telemetry client
> 
> Signed-off-by: Lingli Chen <linglix.chen@intel.com>


Applied, thanks
  
David Marchand March 7, 2023, 8:37 a.m. UTC | #2
On Fri, Feb 10, 2023 at 8:55 AM Lingli Chen <linglix.chen@intel.com> wrote:
>
> according to dpdk commit a15fc792cf (usertools/telemetry: add file prefix argument)
> update file_prefix in telemetry client
>
> Signed-off-by: Lingli Chen <linglix.chen@intel.com>
> ---
>  tests/TestSuite_telemetry.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/TestSuite_telemetry.py b/tests/TestSuite_telemetry.py
> index 736ae7ed..ca2d74f5 100644
> --- a/tests/TestSuite_telemetry.py
> +++ b/tests/TestSuite_telemetry.py
> @@ -420,11 +420,11 @@ class TestTelemetry(TestCase):
>      def change_run_fileprefix(self, out):
>          self.get_file_prefix(out)
>          if self.file_prefix:
> -            cmd1 = 'sed -i \'s/self.socket.send_fd.connect("\/var\/run\/dpdk\/.*\/telemetry")/self.socket.send_fd.connect("\/var\/run\/dpdk\/{0}\/telemetry")/g\' {1}'.format(
> +            cmd1 = "sed -i 's/self.socket.send_fd.connect(self.run_path)/self.socket.send_fd.connect(\"\/var\/run\/dpdk\/{0}\/telemetry\")/g' {1}".format(
>                  self.file_prefix,
>                  os.path.join(self.target_dir, "usertools/dpdk-telemetry-client.py"),
>              )
> -            cmd2 = 'sed -i \'s/self.socket.send_fd.connect("\/var\/run\/dpdk\/.*\/telemetry")/self.socket.send_fd.connect("\/var\/run\/dpdk\/{0}\/telemetry")/g\' {1}'.format(
> +            cmd2 = "sed -i 's/self.socket.send_fd.connect(self.run_path)/self.socket.send_fd.connect(\"\/var\/run\/dpdk\/{0}\/telemetry\")/g' {1}".format(
>                  self.file_prefix,
>                  os.path.join(self.target_dir, "dpdk_telemetry_client.py"),
>              )

Please don't change DPDK sources.
If something is missing in DPDK, please talk to the maintainers.
  

Patch

diff --git a/tests/TestSuite_telemetry.py b/tests/TestSuite_telemetry.py
index 736ae7ed..ca2d74f5 100644
--- a/tests/TestSuite_telemetry.py
+++ b/tests/TestSuite_telemetry.py
@@ -420,11 +420,11 @@  class TestTelemetry(TestCase):
     def change_run_fileprefix(self, out):
         self.get_file_prefix(out)
         if self.file_prefix:
-            cmd1 = 'sed -i \'s/self.socket.send_fd.connect("\/var\/run\/dpdk\/.*\/telemetry")/self.socket.send_fd.connect("\/var\/run\/dpdk\/{0}\/telemetry")/g\' {1}'.format(
+            cmd1 = "sed -i 's/self.socket.send_fd.connect(self.run_path)/self.socket.send_fd.connect(\"\/var\/run\/dpdk\/{0}\/telemetry\")/g' {1}".format(
                 self.file_prefix,
                 os.path.join(self.target_dir, "usertools/dpdk-telemetry-client.py"),
             )
-            cmd2 = 'sed -i \'s/self.socket.send_fd.connect("\/var\/run\/dpdk\/.*\/telemetry")/self.socket.send_fd.connect("\/var\/run\/dpdk\/{0}\/telemetry")/g\' {1}'.format(
+            cmd2 = "sed -i 's/self.socket.send_fd.connect(self.run_path)/self.socket.send_fd.connect(\"\/var\/run\/dpdk\/{0}\/telemetry\")/g' {1}".format(
                 self.file_prefix,
                 os.path.join(self.target_dir, "dpdk_telemetry_client.py"),
             )