oeqa/runtime/context.py: Fix use of getTarget() with testexport
The idea on getTarget is to use kwargs to send custom variables to different targets, instead of this, a new variable was added (just used for custom targets) and this broke testexport. So in order to fix it, just add the custom variable to kwargs. This fixes the use of getTarget() in testexport class that was introduced in 1dc8010afd71fe46fb28bb86fb7c07a5fbd3d7cf. (From OE-Core rev: cf138029a1f18f991fc7a28c81d85884942e9d56) Signed-off-by: Mariano Lopez <mariano.lopez@linux.intel.com> Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
parent
5ff7db946c
commit
cf0b92f893
|
@ -229,6 +229,10 @@ def testimage_main(d):
|
|||
'kvm' : kvm,
|
||||
}
|
||||
|
||||
# TODO: Currently BBPATH is needed for custom loading of targets.
|
||||
# It would be better to find these modules using instrospection.
|
||||
target_kwargs['target_modules_path'] = d.getVar('BBPATH')
|
||||
|
||||
# runtime use network for download projects for build
|
||||
export_proxies(d)
|
||||
|
||||
|
@ -239,7 +243,7 @@ def testimage_main(d):
|
|||
|
||||
# the robot dance
|
||||
target = OERuntimeTestContextExecutor.getTarget(
|
||||
d.getVar("TEST_TARGET"), d.getVar("BBPATH"), None, d.getVar("TEST_TARGET_IP"),
|
||||
d.getVar("TEST_TARGET"), None, d.getVar("TEST_TARGET_IP"),
|
||||
d.getVar("TEST_SERVER_IP"), **target_kwargs)
|
||||
|
||||
# test context
|
||||
|
|
|
@ -15,7 +15,7 @@ class OEQemuTarget(OESSHTarget):
|
|||
def __init__(self, logger, ip, 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):
|
||||
tmpdir='', dir_image='', boottime=60, **kwargs):
|
||||
|
||||
super(OEQemuTarget, self).__init__(logger, ip, server_ip, timeout,
|
||||
user, port)
|
||||
|
|
|
@ -89,7 +89,7 @@ class OERuntimeTestContextExecutor(OETestContextExecutor):
|
|||
help="Qemu boot configuration, only needed when target_type is QEMU.")
|
||||
|
||||
@staticmethod
|
||||
def getTarget(target_type, target_modules_path, logger, target_ip, server_ip, **kwargs):
|
||||
def getTarget(target_type, logger, target_ip, server_ip, **kwargs):
|
||||
target = None
|
||||
|
||||
if target_type == 'simpleremote':
|
||||
|
@ -97,8 +97,17 @@ class OERuntimeTestContextExecutor(OETestContextExecutor):
|
|||
elif target_type == 'qemu':
|
||||
target = OEQemuTarget(logger, target_ip, 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
|
||||
# most of the time was just a wrapper.
|
||||
# XXX: This code tries to import modules from lib/oeqa/controllers
|
||||
# directory and treat them as controllers, it will less error prone
|
||||
# to use introspection to load such modules.
|
||||
# XXX: Don't base your targets on this code it will be refactored
|
||||
# in the near future.
|
||||
# Custom target module loading
|
||||
try:
|
||||
target_modules_path = kwargs.get('target_modules_path', '')
|
||||
controller = OERuntimeTestContextExecutor.getControllerModule(target_type, target_modules_path)
|
||||
target = controller(logger, target_ip, server_ip, **kwargs)
|
||||
except ImportError as e:
|
||||
|
|
Loading…
Reference in New Issue