[PATCH] Devices.BondDevice: fix miimon setter

[Jan Tluka]

Signed-off-by: Jan Tluka <jtluka(a)redhat.com&gt;
Reported-by: Ondrej Lichtner <olichtne(a)redhat.com&gt;
---
 lnst/Devices/BondDevice.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lnst/Devices/BondDevice.py b/lnst/Devices/BondDevice.py
index ce014272..0965fe92 100644
--- a/lnst/Devices/BondDevice.py
+++ b/lnst/Devices/BondDevice.py
@@ -232,7 +232,7 @@ class BondDevice(MasterDevice):
 
     @miimon.setter
     def miimon(self, val):
-        self._set_linkinfo_data_attr("IFLA_BOND_LACP_RATE", int(val))
+        self._set_linkinfo_data_attr("IFLA_BOND_MIIMON", int(val))
         self._nl_link_sync("set")
 
     @property
-- 
2.21.1

[Aniss Loughlam]

it worked for me:
    Configured host1.t_bond0.slaves = ['host1.enp4s0f0',
'host1.enp4s0f1']
    Configured host1.t_bond0.mode = 1
    Configured host1.t_bond0.miimon = 100

On Thu, Apr 16, 2020 at 9:59 AM Jan Tluka <jtluka(a)redhat.com&gt; wrote:

>  Signed-off-by: Jan Tluka <jtluka(a)redhat.com&gt;
>  Reported-by: Ondrej Lichtner <olichtne(a)redhat.com&gt;
>  ---
>   lnst/Devices/BondDevice.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
>  diff --git a/lnst/Devices/BondDevice.py b/lnst/Devices/BondDevice.py
>  index ce014272..0965fe92 100644
>  --- a/lnst/Devices/BondDevice.py
>  +++ b/lnst/Devices/BondDevice.py
>  @@ -232,7 +232,7 @@ class BondDevice(MasterDevice):
>
>       @miimon.setter
>       def miimon(self, val):
>  -        self._set_linkinfo_data_attr("IFLA_BOND_LACP_RATE", int(val))
>  +        self._set_linkinfo_data_attr("IFLA_BOND_MIIMON", int(val))
>           self._nl_link_sync("set")
>
>       @property
>  --
>  2.21.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://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]...
>

[Jan Tluka]

Thu, Apr 16, 2020 at 03:57:42PM CEST, jtluka(a)redhat.com wrote:
> Signed-off-by: Jan Tluka <jtluka(a)redhat.com&gt;
> Reported-by: Ondrej Lichtner <olichtne(a)redhat.com&gt;
> ---
>  lnst/Devices/BondDevice.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lnst/Devices/BondDevice.py b/lnst/Devices/BondDevice.py
> index ce014272..0965fe92 100644
> --- a/lnst/Devices/BondDevice.py
> +++ b/lnst/Devices/BondDevice.py
> @@ -232,7 +232,7 @@ class BondDevice(MasterDevice):
>  
>      @miimon.setter
>      def miimon(self, val):
> -        self._set_linkinfo_data_attr("IFLA_BOND_LACP_RATE", int(val))
> +        self._set_linkinfo_data_attr("IFLA_BOND_MIIMON", int(val))
>          self._nl_link_sync("set")
>  
>      @property
> --  
Pushed.