On Mon, Mar 23, 2020 at 03:22:23PM +0100, Jan Tluka wrote:
Fri, Mar 20, 2020 at 03:13:44PM CET, olichtne(a)redhat.com wrote:
>From: Ondrej Lichtner <olichtne(a)redhat.com>
>
>Signed-off-by: Ondrej Lichtner <olichtne(a)redhat.com>
>---
> lnst/Recipes/ENRT/BaseEnrtRecipe.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/lnst/Recipes/ENRT/BaseEnrtRecipe.py
b/lnst/Recipes/ENRT/BaseEnrtRecipe.py
>index 5862311..fb3aae4 100644
>--- a/lnst/Recipes/ENRT/BaseEnrtRecipe.py
>+++ b/lnst/Recipes/ENRT/BaseEnrtRecipe.py
>@@ -29,7 +29,7 @@ class BaseEnrtRecipe(BaseSubConfigMixin, PingTestAndEvaluate,
PerfRecipe):
> ping_parallel = BoolParam(default=False)
> ping_bidirect = BoolParam(default=False)
> ping_count = IntParam(default=100)
>- ping_interval = StrParam(default=0.2)
>+ ping_interval = FloatParam(default=0.2)
> ping_psize = IntParam(default=None)
>
> #common perf test params
I just tried to run a recipe after applying this patch and got
File "/home/igyn/tmp/lnst/lnst/Recipes/ENRT/BaseEnrtRecipe.py", line 33, in
BaseEnrtRecipe
ping_interval = FloatParam(default=0.2)
NameError: name 'FloatParam' is not defined
You did not include FloatParam in imports.
-Jan
:/ ... I thought I fixed that... must have lost it when rebasing my
working commits. Sent a patch that adds this.
-Ondrej