<div dir="ltr">On Tue, Aug 25, 2015 at 12:25 AM Po-Hsu Lin <<a href="mailto:po-hsu.lin@canonical.com">po-hsu.lin@canonical.com</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
currently we have two sets of fwts jobs:<br>
<br>
firmware/fwts_desktop_diagnosis<br>
(attachment) firmware/fwts_desktop_diagnosis_results.log<br>
(attachment) firmware/fwts_desktop_diagnosis_results_hwe.log<br></blockquote><div><br></div><div>I'm not familar with the history here or why there was a need to duplicate miscellanea/fwts_test by just renaming it fwts_desktop_diagnosis.  I'm also not familiar with why it was necessary to split the tests out and create two separate log files (which changed the code internally) when a parser to parse the FWTS log nad just grab out the bits you want for the separate HWE log would be sufficient... <br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
And<br>
miscellanea/fwts_test<br>
(attachment) miscellanea/fwts_results.log<br>
(missing the _hwe attachment job, issue could be found here:<br>
<a href="http://pad.lv/1483598" rel="noreferrer" target="_blank">http://pad.lv/1483598</a>)<br>
<br>
For the fwts job itself, these two jobs are running the same command,<br>
just with different output name<br>
 fwts_test -l $PLAINBOX_SESSION_SHARE/fwts_desktop_diagnosis_results.log<br>
 fwts_test -l $PLAINBOX_SESSION_SHARE/fwts_results.log<br>
<br>
I am not sure why we have two set of these, maybe we can merge these together<br>
get rid of the desktop one to make it more easier to maintain.<br></blockquote><div><br><br></div><div>That would be nice, thought as stated above, I'd also revisit the changes that create two logs and instead have just one log and a parser job to create the second HWE log if it's really that necessary to pull out the results for those tests.<br><br></div><div>Just my opinion, though, as I said, I'm not familiar with why those duplicated jobs and the script changes were necessary to begin with.<br><br></div><div>Cheers<br></div><div>Jeff<br></div><div><br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers<br>
Sam<br>
<br>
--<br>
Checkbox-devel mailing list<br>
<a href="mailto:Checkbox-devel@lists.ubuntu.com" target="_blank">Checkbox-devel@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/checkbox-devel" rel="noreferrer" target="_blank">https://lists.ubuntu.com/mailman/listinfo/checkbox-devel</a><br>
</blockquote></div></div><div dir="ltr">-- <br></div><div dir="ltr"><div>"Entropy isn't what it used to be."</div><div><br></div><div>Jeff Lane - Server Certification Lead, OCP Certification Tools Engineering Lead,</div><div>  Â  Â  Â  Â  Â  Â  Â  Â  Warrior Poet, Biker, Lover of Pie</div><div>Phone: 919-442-8649</div><div>Ubuntu Ham: W4KDH Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â Freenode IRC: bladernr or bladernr_</div><div>gpg: 1024D/3A14B2DD 8C88 B076 0DD7 B404 1417 Â C466 4ABD 3635 3A14 B2DD</div></div>