Bug 4838 - Use the new testimage class for the sanity tests
Summary: Use the new testimage class for the sanity tests
Status: VERIFIED FIXED
Alias: None
Product: AutoBuilder
Classification: Infrastructure
Component: autobuilder (show other bugs)
Version: 1.5
Hardware: x86 Multiple
: Medium+ enhancement
Target Milestone: 1.5 M3
Assignee: Stefan Stanacar
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2013-07-10 11:39 UTC by Stefan Stanacar
Modified: 2013-09-30 14:24 UTC (History)
5 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: Done (doc changes complete)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Stanacar 2013-07-10 11:39:22 UTC
Now that the new qemu image test class has been merged  we should use to run the sanity tests on the AB.
This means enabling the option in the RunSanityTest step and change some buildsets to use the new option.
Comment 1 Stefan Stanacar 2013-07-10 11:44:37 UTC
I already have something cooking up:
http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=stefans/yab-newsanity1&id=6e626f75873bbd591bd5c278a07cf407080c75c5
http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=stefans/yab-newsanity1&id=7d5ddc3ec2e6e704910423276e51def350710fed
http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=stefans/yab-newsanity1&id=bf5d5a55ec7d48c6986e068b4a0cab9df7246277

However I get a nice buildbot traceback with the first patch and right now I'm stuck, because it really shouldn't happen.
Seems the getProperty("poky_branch") is at fault, but I have no idea why.
Comment 2 Stefan Stanacar 2013-07-10 12:43:56 UTC
I think I've sorted this out: __init__ is too early to get properties.
Properties aren't available until the step starts as __init__ runs during config parsing, so I moved the command to "start" (if I had just looked at other buildsteps earlier):
(instead of the first link)
http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=stefans/yab-newanity2&id=2a980e37a7097d23d54938ecdb473edc2a6c37a6

I'll send the patches on the ML for review shortly.
Comment 4 Alexandru Palalau 2013-09-30 14:20:32 UTC
Verified on master branch.
Commit ID: 4d8ab87d25ea6e59dc88375813ef88ee0ac813d0
Docs: http://www.yoctoproject.org/docs/latest/dev-manual/dev-manual.html#performing-automated-runtime-testing
Comment 5 Alexandru Palalau 2013-09-30 14:24:03 UTC
yocto-autobuilder commit ID:275d78c2ef4c5bf41940c155d0f812788811d134