InterfaceManager: fix iteration over tmp_mapping

This patch fixes "Too many values to unpack" exceptions that sometimes
occur and leave lnst-slave in an unresponsive state.

Signed-off-by: Ondrej Lichtner <olichtne@redhat.com>
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
InterfaceManager: is_name_used searches tmp_mapping

This should solve race conditions that occur when creating interfaces of
the same kind when the name of the first one is not yet recognized by
LNST as used.

Signed-off-by: Ondrej Lichtner <olichtne@redhat.com>
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
SlavePool: update debug message

The allow_virt attribute now has a different meaning so the debug
message should be updated.

Signed-off-by: Ondrej Lichtner <olichtne@redhat.com>
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
lnst-ctl: first log creation should be sooner

This patch moves the log creation of the first recipe match before the
creation of the NetTestController object. Without this some logs were
misplaced.

Signed-off-by: Ondrej Lichtner <olichtne@redhat.com>
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
NetConfigDevice: set bond down before configuration

When NM is running on a machine and lnst is using normal configuration
the bond is created, NM sets it up which makes the configuration of the
bonding mode fail. This patch makes sure that before configuring any
options the device is down. It's a temporary workaround until the issue
is fixed in NM.

Signed-off-by: Ondrej Lichtner <olichtne@redhat.com>
Signed-off-by: Jiri Pirko <jiri@resnulli.us>