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

[Jan Tluka]

Signed-off-by: Jan Tluka 
---
 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):

[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, [e-mail] wrote:
> Signed-off-by: Jan Tluka
>
> 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 -- [e-mail]
> To unsubscribe send an email to [e-mail]
> 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]...