Wed, Aug 19, 2015 at 11:16:20AM CEST, olichtne(a)redhat.com wrote:
On Mon, Aug 17, 2015 at 05:25:23PM +0200, Jan Tluka wrote:
> Method map_if_by_params provides ability to map interface based on more
> variable query. For example to map interface based on hardware address
> and driver (e.g. loopback).
>
> Signed-off-by: Jan Tluka <jtluka(a)redhat.com>
> ---
> lnst/Slave/NetTestSlave.py | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/lnst/Slave/NetTestSlave.py b/lnst/Slave/NetTestSlave.py
> index 23fd009..b66dfd1 100644
> --- a/lnst/Slave/NetTestSlave.py
> +++ b/lnst/Slave/NetTestSlave.py
> @@ -120,6 +120,15 @@ class SlaveMethods:
>
> return devices
>
> + def map_if_by_params(self, if_id, params):
> + devices = self.get_devices_by_params(params)
> +
> + if len(devices) == 1:
> + dev = self._if_manager.get_device_by_params(params)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
InterfaceManager doesn't have a get_device_by_params method and this
patchset doesn't add it. Did you forget to include it in your patches?
Thanks for catching this. I've forgotten to include the patch in the
series. Will repost.
-Jan