[OE-core] [PATCH] lib/oeqa: allow a layer to provide it's own TEST_TARGET class

[Sipke Vriend]

Hi Stefan,

I have started using this patch in master to extend test targets to our
bsp layer, and it works well. Thanks.
Will the extension of a controllers.py file work for multiple bsp layers
in a single build/conf/bblayers.conf?
I did a quick test and think that the first controller.py found is the
only one "extended". If this is true then  the current implementation
could get a false negative for a given TEST_TARGET, if more than one
layer implements controllers.py.

I tried a minor modification which assumes each testtarget class is in
its own file with the name <testtarget>.py (lower case). This extended
across multiple layers, assuming no duplication of test target names. I
put them in the utils directory for the test, but they could go into a
controllers folder as that might make more sense.

diff --git a/meta/lib/oeqa/targetcontrol.py b/meta/lib/oeqa/targetcontrol.py
index 757f9d3..d7cab7e 100644
--- a/meta/lib/oeqa/targetcontrol.py
+++ b/meta/lib/oeqa/targetcontrol.py
@@ -28,10 +28,11 @@ def get_target_controller(d):
          except AttributeError:
              # nope, perhaps a layer defined one
              try:
-                module = __import__("oeqa.utils.controllers",
globals(), locals(), [testtarget])
+                modulename = "oeqa.utils.{0}".format(testtarget.lower())
+                module = __import__(modulename, globals(), locals(),
[testtarget])
                  controller = getattr(module, testtarget)
              except ImportError as e:
-                bb.fatal("Failed to import oeqa.utils.controllers:\n%s"
% traceback.format_exc())
+                bb.fatal("Failed to import
{0}:\n{1}".format(modulename,traceback.format_exc()))
              except AttributeError:
                  bb.fatal("\"%s\" is not a valid value for TEST_TARGET"
% testtarget)
          return controller(d)

Cheers
Sipke

On 16/01/2014 10:48 PM, Stefan Stanacar wrote:
> Allows a layer to define new classes in<layer>/lib/oeqa/utils/controllers.py
> and completely control or extend deployment of a target. (core currently
> has QemuTarget and SimpleRemoteTarget).
> The value of TEST_TARGET must be the name of the new class.
>
> Signed-off-by: Stefan Stanacar<stefanx.stanacar at intel.com>
> ---
>   meta/lib/oeqa/targetcontrol.py  | 23 +++++++++++++++++++----
>   meta/lib/oeqa/utils/__init__.py |  3 +++
>   2 files changed, 22 insertions(+), 4 deletions(-)
>
> diff --git a/meta/lib/oeqa/targetcontrol.py b/meta/lib/oeqa/targetcontrol.py
> index dee38ec..757f9d3 100644
> --- a/meta/lib/oeqa/targetcontrol.py
> +++ b/meta/lib/oeqa/targetcontrol.py
> @@ -8,18 +8,33 @@ import os
>   import shutil
>   import subprocess
>   import bb
> -
> +import traceback
>   from oeqa.utils.sshcontrol import SSHControl
>   from oeqa.utils.qemurunner import QemuRunner
>
>
>   def get_target_controller(d):
> -    if d.getVar("TEST_TARGET", True) == "qemu":
> +    testtarget = d.getVar("TEST_TARGET", True)
> +    # old, simple names
> +    if testtarget == "qemu":
>           return QemuTarget(d)
> -    elif d.getVar("TEST_TARGET", True) == "simpleremote":
> +    elif testtarget == "simpleremote":
>           return SimpleRemoteTarget(d)
>       else:
> -        bb.fatal("Please set a valid TEST_TARGET")
> +        # use the class name
> +        try:
> +            # is it a core class defined here?
> +            controller = getattr(__name__, testtarget)
> +        except AttributeError:
> +            # nope, perhaps a layer defined one
> +            try:
> +                module = __import__("oeqa.utils.controllers", globals(), locals(), [testtarget])
> +                controller = getattr(module, testtarget)
> +            except ImportError as e:
> +                bb.fatal("Failed to import oeqa.utils.controllers:\n%s" % traceback.format_exc())
> +            except AttributeError:
> +                bb.fatal("\"%s\" is not a valid value for TEST_TARGET" % testtarget)
> +        return controller(d)
>
>
>   class BaseTarget(object):
> diff --git a/meta/lib/oeqa/utils/__init__.py b/meta/lib/oeqa/utils/__init__.py
> index e69de29..8eda927 100644
> --- a/meta/lib/oeqa/utils/__init__.py
> +++ b/meta/lib/oeqa/utils/__init__.py
> @@ -0,0 +1,3 @@
> +# Enable other layers to have modules in the same named directory
> +from pkgutil import extend_path
> +__path__ = extend_path(__path__, __name__)
>



This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.