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(a)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()
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
Show replies by date