[PATCH v1 1/2] stats_reporter: properly use iterable in for cycle

[PATCH v1 1/2] stats_reporter: properly use iterable in for cycle

[Juraj Linkeš]

Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
---
 framework/stats_reporter.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/framework/stats_reporter.py b/framework/stats_reporter.py
index 80ac942c..1c1ab3fd 100644
--- a/framework/stats_reporter.py
+++ b/framework/stats_reporter.py
@@ -70,7 +70,7 @@ class StatsReporter(object):
                 "dpdk_version = {}\n".format(self.result.current_dpdk_version(duts[0]))
             )
         else:
-            for dut in duts():
+            for dut in duts:
                 dpdk_version = self.result.current_dpdk_version(dut)
                 self.stats_file.write(
                     "{}.dpdk_version = {}\n".format(dut, dpdk_version)
-- 
2.20.1

[PATCH v1 2/2] python: fix shebangs

[Juraj Linkeš]

Remove shebangs in __init__.py files as they're not supposed to be
executed.
Modify the other shebangs so that virtualenvs can be used.

Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
---
 dep/__init__.py              | 1 -
 framework/__init__.py        | 1 -
 main.py                      | 2 +-
 nics/__init__.py             | 1 -
 tests/__init__.py            | 1 -
 tools/__init__.py            | 1 -
 vm_images/create_vm_image.py | 2 +-
 7 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/dep/__init__.py b/dep/__init__.py
index ae0043b7..412dbab4 100644
--- a/dep/__init__.py
+++ b/dep/__init__.py
@@ -1,4 +1,3 @@
-#!/usr/bin/python3
 # BSD LICENSE
 #
 # Copyright (c) 2021 PANTHEON.tech s.r.o.
diff --git a/framework/__init__.py b/framework/__init__.py
index ae0043b7..412dbab4 100644
--- a/framework/__init__.py
+++ b/framework/__init__.py
@@ -1,4 +1,3 @@
-#!/usr/bin/python3
 # BSD LICENSE
 #
 # Copyright (c) 2021 PANTHEON.tech s.r.o.
diff --git a/main.py b/main.py
index f4f5e03a..b625cecf 100755
--- a/main.py
+++ b/main.py
@@ -1,4 +1,4 @@
-#!/usr/bin/python3
+#!/usr/bin/env python3
 # BSD LICENSE
 #
 # Copyright(c) 2010-2014 Intel Corporation. All rights reserved.
diff --git a/nics/__init__.py b/nics/__init__.py
index ae0043b7..412dbab4 100644
--- a/nics/__init__.py
+++ b/nics/__init__.py
@@ -1,4 +1,3 @@
-#!/usr/bin/python3
 # BSD LICENSE
 #
 # Copyright (c) 2021 PANTHEON.tech s.r.o.
diff --git a/tests/__init__.py b/tests/__init__.py
index ae0043b7..412dbab4 100644
--- a/tests/__init__.py
+++ b/tests/__init__.py
@@ -1,4 +1,3 @@
-#!/usr/bin/python3
 # BSD LICENSE
 #
 # Copyright (c) 2021 PANTHEON.tech s.r.o.
diff --git a/tools/__init__.py b/tools/__init__.py
index ae0043b7..412dbab4 100644
--- a/tools/__init__.py
+++ b/tools/__init__.py
@@ -1,4 +1,3 @@
-#!/usr/bin/python3
 # BSD LICENSE
 #
 # Copyright (c) 2021 PANTHEON.tech s.r.o.
diff --git a/vm_images/create_vm_image.py b/vm_images/create_vm_image.py
index 221477bb..4be9630d 100755
--- a/vm_images/create_vm_image.py
+++ b/vm_images/create_vm_image.py
@@ -1,4 +1,4 @@
-#!/usr/bin/python3
+#!/usr/bin/env python3
 
 import argparse
 import enum
-- 
2.20.1

RE: [PATCH v1 1/2] stats_reporter: properly use iterable in for cycle

[Dong, JunX]

Acked-by: Jun Dong <junx.dong@intel.com>

> -----Original Message-----
> From: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Sent: Wednesday, April 27, 2022 3:42 PM
> To: Dong, JunX <junx.dong@intel.com>; Tu, Lijuan <lijuan.tu@intel.com>;
> ohilyard@iol.unh.edu
> Cc: dts@dpdk.org; Juraj Linkeš <juraj.linkes@pantheon.tech>
> Subject: [PATCH v1 1/2] stats_reporter: properly use iterable in for cycle
> 
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> ---
>  framework/stats_reporter.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/framework/stats_reporter.py b/framework/stats_reporter.py
> index 80ac942c..1c1ab3fd 100644
> --- a/framework/stats_reporter.py
> +++ b/framework/stats_reporter.py
> @@ -70,7 +70,7 @@ class StatsReporter(object):
>                  "dpdk_version =
> {}\n".format(self.result.current_dpdk_version(duts[0]))
>              )
>          else:
> -            for dut in duts():
> +            for dut in duts:
>                  dpdk_version = self.result.current_dpdk_version(dut)
>                  self.stats_file.write(
>                      "{}.dpdk_version = {}\n".format(dut, dpdk_version)
> --
> 2.20.1