[PATCH 1/2] Store interfaces driver info in SlavePool mapping and show it in summary
Jan Tluka
jtluka at redhat.com
Mon Jun 29 08:27:54 UTC 2015
Thu, Jun 25, 2015 at 03:27:26PM CEST, jprochaz at redhat.com wrote:
>For issue #111. Since RPC connection is created after mapping, we can't get
>interface driver info sooner without creating extra connection to the Slave.
>This patch stores driver info after device configuration, after method
>interface_update() was called, so driver info should be present in Slave.
>
>Signed-off-by: Jiri Prochazka <jprochaz at redhat.com>
>---
> lnst/Controller/NetTestController.py | 5 +++++
> lnst/Controller/NetTestResultSerializer.py | 14 +++++++++++---
> 2 files changed, 16 insertions(+), 3 deletions(-)
>
>diff --git a/lnst/Controller/NetTestController.py b/lnst/Controller/NetTestController.py
>index 5c2695d..0791897 100644
>--- a/lnst/Controller/NetTestController.py
>+++ b/lnst/Controller/NetTestController.py
>@@ -204,6 +204,11 @@ class NetTestController:
>
> for iface in ifaces:
> iface.configure()
>+ if m._libvirt_domain is None:
>+ driver = iface._driver
>+ if_id = iface._id
>+ mapped_machine = self._slave_pool._map['machines'][m_id]
>+ mapped_machine['interfaces'][if_id]['driver'] = driver
> for iface in ifaces:
> iface.up()
>
Jirko, why you limit this to non-virt environment. Is this not working
with virtio devices?
>diff --git a/lnst/Controller/NetTestResultSerializer.py b/lnst/Controller/NetTestResultSerializer.py
>index 9f8955a..49f632c 100644
>--- a/lnst/Controller/NetTestResultSerializer.py
>+++ b/lnst/Controller/NetTestResultSerializer.py
>@@ -100,9 +100,17 @@ class NetTestResultSerializer:
> (m_id, m["target"]), ""))
> for if_id, pool_if in m["interfaces"].iteritems():
> pool_id = pool_if["target"]
>- output_pairs.append((6*" " + "interface \"%s\" "\
>- "matched to \"%s\"" %\
>- (if_id, pool_id), ""))
>+ if "driver" in pool_if:
>+ driver = pool_if["driver"]
>+ output_pairs.append((6*" " + "interface \"%s\" "
>+ "matched to \"%s\" "
>+ "(driver: \"%s\")" %
>+ (if_id, pool_id,
>+ driver), ""))
>+ else:
>+ output_pairs.append((6*" " + "interface \"%s\" "
>+ "matched to \"%s\" " %
>+ (if_id, pool_id), ""))
>
> if recipe["result"] == "FAIL" and \
> "err_msg" in recipe and recipe["err_msg"] != "":
>--
>2.4.3
>
>_______________________________________________
>LNST-developers mailing list
>LNST-developers at lists.fedorahosted.org
>https://lists.fedorahosted.org/mailman/listinfo/lnst-developers
More information about the LNST-developers
mailing list