It was suggested that option 'target_bridge' should be renamed to 'libvirt_bridge'[1]. In following two lines the change was not made resulting in ignoring user supplied bridge device.
[1] https://lists.fedorahosted.org/pipermail/lnst-developers/2012-August/000271....
Signed-off-by: Jan Tluka jtluka@redhat.com --- NetTest/NetTestController.py | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/NetTest/NetTestController.py b/NetTest/NetTestController.py index b028e35..2a6bbd6 100644 --- a/NetTest/NetTestController.py +++ b/NetTest/NetTestController.py @@ -122,8 +122,8 @@ class NetTestController: if not len(query_result): break
- if "target_bridge" in dev: - brctl = BridgeCtl(dev["target_bridge"]) + if "libvirt_bridge" in dev: + brctl = BridgeCtl(dev["libvirt_bridge"]) else: if "default_bridge" in dev_net: brctl = dev_net["default_bridge"]
On Fri, Sep 07, 2012 at 05:27:10PM +0200, Jan Tluka wrote:
It was suggested that option 'target_bridge' should be renamed to 'libvirt_bridge'[1]. In following two lines the change was not made resulting in ignoring user supplied bridge device.
This one is mine :(, i forgot to change it there, sorry.
Radek
[1] https://lists.fedorahosted.org/pipermail/lnst-developers/2012-August/000271....
Signed-off-by: Jan Tluka jtluka@redhat.com
NetTest/NetTestController.py | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/NetTest/NetTestController.py b/NetTest/NetTestController.py index b028e35..2a6bbd6 100644 --- a/NetTest/NetTestController.py +++ b/NetTest/NetTestController.py @@ -122,8 +122,8 @@ class NetTestController: if not len(query_result): break
if "target_bridge" in dev:
brctl = BridgeCtl(dev["target_bridge"])
if "libvirt_bridge" in dev:
brctl = BridgeCtl(dev["libvirt_bridge"]) else: if "default_bridge" in dev_net: brctl = dev_net["default_bridge"]
-- 1.7.7.6
LNST-developers mailing list LNST-developers@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/lnst-developers
lnst-developers@lists.fedorahosted.org