Bug 4838

Summary: Use the new testimage class for the sanity tests
Product: [Infrastructure] AutoBuilder Reporter: Stefan Stanacar <sstncr>
Component: autobuilderAssignee: Stefan Stanacar <sstncr>
Status: VERIFIED FIXED QA Contact:
Severity: enhancement    
Priority: Medium+ CC: alexandrux.palalau, elizabeth.flanagan, infras.ab.watcher, Infras.watcher, jessica.zhang
Version: 1.5   
Target Milestone: 1.5 M3   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: Done (doc changes complete)

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