[PATCH-next] lnst.Recipes.ENRT: typos in DoubleBondRecipe, VirtualBridgeVlanInGuestMirroredRecipe

From: Christos Sfakianakis <csfakian(a)redhat.com&gt;

Replace the first argument in self._pin_dev_interrupts() with correct
device.

Signed-off-by: Christos Sfakianakis <csfakian(a)redhat.com&gt;
---
 lnst/Recipes/ENRT/DoubleBondRecipe.py                       | 2 +-
 lnst/Recipes/ENRT/VirtualBridgeVlanInGuestMirroredRecipe.py | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lnst/Recipes/ENRT/DoubleBondRecipe.py
b/lnst/Recipes/ENRT/DoubleBondRecipe.py
index bc604d6..3b437af 100644
--- a/lnst/Recipes/ENRT/DoubleBondRecipe.py
+++ b/lnst/Recipes/ENRT/DoubleBondRecipe.py
@@ -67,7 +67,7 @@ class DoubleBondRecipe(BaseEnrtRecipe):
             for host in [host1, host2]:
                 host.run("service irqbalance stop")
                 for dev in [host.eth0, host.eth1]:
-                    self._pin_dev_interrupts(host.eth0, self.params.dev_intr_cpu)
+                    self._pin_dev_interrupts(dev, self.params.dev_intr_cpu)
 
         if self.params.perf_parallel_streams > 1:
             for host in [host1, host2]:
diff --git a/lnst/Recipes/ENRT/VirtualBridgeVlanInGuestMirroredRecipe.py
b/lnst/Recipes/ENRT/VirtualBridgeVlanInGuestMirroredRecipe.py
index f33f28b..9a2ae93 100644
--- a/lnst/Recipes/ENRT/VirtualBridgeVlanInGuestMirroredRecipe.py
+++ b/lnst/Recipes/ENRT/VirtualBridgeVlanInGuestMirroredRecipe.py
@@ -102,7 +102,7 @@ class VirtualBridgeVlanInGuestMirroredRecipe(BaseEnrtRecipe):
         if "dev_intr_cpu" in self.params:
             for host in [host1, host2]:
                 host.run("service irqbalance stop")
-                self._pin_dev_interrupts(m.eth0, self.params.dev_intr_cpu)
+                self._pin_dev_interrupts(host.eth0, self.params.dev_intr_cpu)
 
         if self.params.perf_parallel_streams > 1:
             for host in [host1, host2]:
-- 
2.17.1

[Ondrej Lichtner]

On Tue, May 21, 2019 at 01:24:20PM +0200, csfakian(a)redhat.com wrote:
>  From: Christos Sfakianakis <csfakian(a)redhat.com&gt;
>  
>  Replace the first argument in self._pin_dev_interrupts() with correct
>  device.
>  
>  Signed-off-by: Christos Sfakianakis <csfakian(a)redhat.com&gt;
>  ---
>   lnst/Recipes/ENRT/DoubleBondRecipe.py                       | 2 +-
>   lnst/Recipes/ENRT/VirtualBridgeVlanInGuestMirroredRecipe.py | 2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
>  
>  diff --git a/lnst/Recipes/ENRT/DoubleBondRecipe.py
> b/lnst/Recipes/ENRT/DoubleBondRecipe.py
>  index bc604d6..3b437af 100644
>  --- a/lnst/Recipes/ENRT/DoubleBondRecipe.py
>  +++ b/lnst/Recipes/ENRT/DoubleBondRecipe.py
>  @@ -67,7 +67,7 @@ class DoubleBondRecipe(BaseEnrtRecipe):
>               for host in [host1, host2]:
>                   host.run("service irqbalance stop")
>                   for dev in [host.eth0, host.eth1]:
>  -                    self._pin_dev_interrupts(host.eth0, self.params.dev_intr_cpu)
>  +                    self._pin_dev_interrupts(dev, self.params.dev_intr_cpu)
>   
>           if self.params.perf_parallel_streams > 1:
>               for host in [host1, host2]:
>  diff --git a/lnst/Recipes/ENRT/VirtualBridgeVlanInGuestMirroredRecipe.py
> b/lnst/Recipes/ENRT/VirtualBridgeVlanInGuestMirroredRecipe.py
>  index f33f28b..9a2ae93 100644
>  --- a/lnst/Recipes/ENRT/VirtualBridgeVlanInGuestMirroredRecipe.py
>  +++ b/lnst/Recipes/ENRT/VirtualBridgeVlanInGuestMirroredRecipe.py
>  @@ -102,7 +102,7 @@ class VirtualBridgeVlanInGuestMirroredRecipe(BaseEnrtRecipe):
>           if "dev_intr_cpu" in self.params:
>               for host in [host1, host2]:
>                   host.run("service irqbalance stop")
>  -                self._pin_dev_interrupts(m.eth0, self.params.dev_intr_cpu)
>  +                self._pin_dev_interrupts(host.eth0, self.params.dev_intr_cpu)
>   
>           if self.params.perf_parallel_streams > 1:
>               for host in [host1, host2]:
>  -- 
>  2.17.1
>  _______________________________________________
>  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://getfedora.org/code-of-conduct.html
>  List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
>  List Archives:
> https://lists.fedorahosted.org/archives/list/[e-mail]...
pushed, thanks.

-Ondrej