]> code.ossystems Code Review - openembedded-core.git/commitdiff
oeqa/qemu & runtime: qemu do not need ip input from external
authorYeoh Ee Peng <ee.peng.yeoh@intel.com>
Thu, 22 Nov 2018 09:10:43 +0000 (17:10 +0800)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 22 Nov 2018 12:19:26 +0000 (12:19 +0000)
Qemu do not use the ip input from external. It will
retrieve ip from QemuRunner instance and assign
ip value.

Signed-off-by: Yeoh Ee Peng <ee.peng.yeoh@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/lib/oeqa/core/target/qemu.py
meta/lib/oeqa/runtime/context.py

index bf3b633f04759322bfa9f244ab2372cdcd4e8d61..538bf12b6aa3467084ebdc8366611802ed9c4024 100644 (file)
@@ -12,15 +12,14 @@ from oeqa.utils.qemurunner import QemuRunner
 supported_fstypes = ['ext3', 'ext4', 'cpio.gz', 'wic']
 
 class OEQemuTarget(OESSHTarget):
-    def __init__(self, logger, ip, server_ip, timeout=300, user='root',
+    def __init__(self, logger, server_ip, timeout=300, user='root',
             port=None, machine='', rootfs='', kernel='', kvm=False,
             dump_dir='', dump_host_cmds='', display='', bootlog='',
             tmpdir='', dir_image='', boottime=60, **kwargs):
 
-        super(OEQemuTarget, self).__init__(logger, ip, server_ip, timeout,
+        super(OEQemuTarget, self).__init__(logger, None, server_ip, timeout,
                 user, port)
 
-        self.ip = ip
         self.server_ip = server_ip
         self.machine = machine
         self.rootfs = rootfs
index a7f382399668c35e1ab83a192f66fb21b0587819..943e29b0d99e93f969d1ef433e04785900f6e792 100644 (file)
@@ -101,7 +101,7 @@ class OERuntimeTestContextExecutor(OETestContextExecutor):
         if target_type == 'simpleremote':
             target = OESSHTarget(logger, target_ip, server_ip, **kwargs)
         elif target_type == 'qemu':
-            target = OEQemuTarget(logger, target_ip, server_ip, **kwargs)
+            target = OEQemuTarget(logger, server_ip, **kwargs)
         else:
             # XXX: This code uses the old naming convention for controllers and
             # targets, the idea it is to leave just targets as the controller