[PATCH] Recipes.ENRT.TeamVsBondRecipe: extend the recipe with PerfReversibleFlowMixin

[Jan Tluka]

Signed-off-by: Jan Tluka <jtluka(a)redhat.com&gt;
---
 lnst/Recipes/ENRT/TeamVsBondRecipe.py | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/lnst/Recipes/ENRT/TeamVsBondRecipe.py
b/lnst/Recipes/ENRT/TeamVsBondRecipe.py
index 515d2ea9..a617473c 100644
--- a/lnst/Recipes/ENRT/TeamVsBondRecipe.py
+++ b/lnst/Recipes/ENRT/TeamVsBondRecipe.py
@@ -6,12 +6,14 @@ from lnst.Recipes.ENRT.ConfigMixins.OffloadSubConfigMixin import (
     OffloadSubConfigMixin)
 from lnst.Recipes.ENRT.ConfigMixins.CommonHWSubConfigMixin import (
     CommonHWSubConfigMixin)
+from lnst.Recipes.ENRT.ConfigMixins.PerfReversibleFlowMixin import (
+    PerfReversibleFlowMixin)
 from lnst.RecipeCommon.Ping.PingEndpoints import PingEndpoints
 from lnst.Devices import TeamDevice
 from lnst.Devices import BondDevice
 
-class TeamVsBondRecipe(CommonHWSubConfigMixin, OffloadSubConfigMixin,
-    BaseEnrtRecipe):
+class TeamVsBondRecipe(PerfReversibleFlowMixin, CommonHWSubConfigMixin,
+    OffloadSubConfigMixin, BaseEnrtRecipe):
     host1 = HostReq()
     host1.eth0 = DeviceReq(label="tnet",
driver=RecipeParam("driver"))
     host1.eth1 = DeviceReq(label="tnet",
driver=RecipeParam("driver"))
@@ -108,8 +110,7 @@ class TeamVsBondRecipe(CommonHWSubConfigMixin, OffloadSubConfigMixin,
         ]
 
     def generate_perf_endpoints(self, config):
-        return [(self.matched.host1.team0, self.matched.host2.bond0),
-            (self.matched.host2.bond0, self.matched.host1.team0)]
+        return [(self.matched.host1.team0, self.matched.host2.bond0)]
 
     @property
     def offload_nics(self):
-- 
2.21.3

[Jan Tluka]

Scratch this one. I need to remove perf_reverse parameter defined by the
recipe so that it does not conflict with the parameter defined by the
PerfReversibleFlowMixin.

-Jan

Wed, Aug 12, 2020 at 03:44:37PM CEST, jtluka(a)redhat.com wrote:
> Signed-off-by: Jan Tluka <jtluka(a)redhat.com&gt;
> ---
>  lnst/Recipes/ENRT/TeamVsBondRecipe.py | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/lnst/Recipes/ENRT/TeamVsBondRecipe.py
> b/lnst/Recipes/ENRT/TeamVsBondRecipe.py
> index 515d2ea9..a617473c 100644
> --- a/lnst/Recipes/ENRT/TeamVsBondRecipe.py
> +++ b/lnst/Recipes/ENRT/TeamVsBondRecipe.py
> @@ -6,12 +6,14 @@ from lnst.Recipes.ENRT.ConfigMixins.OffloadSubConfigMixin import (
>      OffloadSubConfigMixin)
>  from lnst.Recipes.ENRT.ConfigMixins.CommonHWSubConfigMixin import (
>      CommonHWSubConfigMixin)
> +from lnst.Recipes.ENRT.ConfigMixins.PerfReversibleFlowMixin import (
> +    PerfReversibleFlowMixin)
>  from lnst.RecipeCommon.Ping.PingEndpoints import PingEndpoints
>  from lnst.Devices import TeamDevice
>  from lnst.Devices import BondDevice
>  
> -class TeamVsBondRecipe(CommonHWSubConfigMixin, OffloadSubConfigMixin,
> -    BaseEnrtRecipe):
> +class TeamVsBondRecipe(PerfReversibleFlowMixin, CommonHWSubConfigMixin,
> +    OffloadSubConfigMixin, BaseEnrtRecipe):
>      host1 = HostReq()
>      host1.eth0 = DeviceReq(label="tnet",
> driver=RecipeParam("driver"))
>      host1.eth1 = DeviceReq(label="tnet",
> driver=RecipeParam("driver"))
> @@ -108,8 +110,7 @@ class TeamVsBondRecipe(CommonHWSubConfigMixin, OffloadSubConfigMixin,
>          ]
>  
>      def generate_perf_endpoints(self, config):
> -        return [(self.matched.host1.team0, self.matched.host2.bond0),
> -            (self.matched.host2.bond0, self.matched.host1.team0)]
> +        return [(self.matched.host1.team0, self.matched.host2.bond0)]
>  
>      @property
>      def offload_nics(self):
> -- 
> 2.21.3
> _______________________________________________
> 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]...