[PATCH] Tests.Iperf: add missing initialization of oneoff iperf parameter

[Jan Tluka]

If parameter 'oneoff' is not specified, the default value False is used and
consequently None is passed as value of oneoff variable because the
initialization of the variable is skipped.

Signed-off-by: Jan Tluka <jtluka(a)redhat.com&gt;
---
 lnst/Tests/Iperf.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lnst/Tests/Iperf.py b/lnst/Tests/Iperf.py
index db8f38d1..bedeb518 100644
--- a/lnst/Tests/Iperf.py
+++ b/lnst/Tests/Iperf.py
@@ -60,6 +60,7 @@ class IperfServer(IperfBase):
     def _compose_cmd(self):
         bind = ""
         port = ""
+        oneoff = ""
 
         if "bind" in self.params:
             bind = "-B {}".format(self.params.bind)
-- 
2.26.2

[Ondrej Lichtner]

On Thu, Jan 21, 2021 at 09:45:59AM +0100, Jan Tluka wrote:
>  If parameter 'oneoff' is not specified, the default value
> False is used and
>  consequently None is passed as value of oneoff variable because the
>  initialization of the variable is skipped.
>  
>  Signed-off-by: Jan Tluka <jtluka(a)redhat.com&gt;
>  ---
>   lnst/Tests/Iperf.py | 1 +
>   1 file changed, 1 insertion(+)
>  
>  diff --git a/lnst/Tests/Iperf.py b/lnst/Tests/Iperf.py
>  index db8f38d1..bedeb518 100644
>  --- a/lnst/Tests/Iperf.py
>  +++ b/lnst/Tests/Iperf.py
>  @@ -60,6 +60,7 @@ class IperfServer(IperfBase):
>       def _compose_cmd(self):
>           bind = ""
>           port = ""
>  +        oneoff = ""
>   
>           if "bind" in self.params:
>               bind = "-B {}".format(self.params.bind)
>  -- 
>  2.26.2
>  _______________________________________________
>  LNST-developers mailing list -- lnst-developers(a)lists.fedorahosted.org
>  To unsubscribe send an email to lnst-developers-leave(a)lists.fedorahosted.org
>  Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
>  List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
>  List Archives:
> https://lists.fedorahosted.org/archives/list/[e-mail]...
pushed, thanks.

-Ondrej