Bug 6354

Summary: PublishArtifacts needs md5sums as an artifact.
Product: [Infrastructure] AutoBuilder Reporter: Beth Flanagan <elizabeth.flanagan>
Component: autobuilderAssignee: Beth Flanagan <elizabeth.flanagan>
Status: RESOLVED FIXED QA Contact:
Severity: enhancement    
Priority: Medium CC: bluelightning, elizabeth.flanagan, infras.ab.watcher, Infras.watcher, jessica.zhang
Version: 1.7   
Target Milestone: 1.7   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)

Description Beth Flanagan 2014-05-20 18:26:32 UTC 
Currently, we have md5sums generated by the MakeMD5Sums step and published with image artifacts via PublishArtifacts. So, an assumption is being made that that step is being run, even though it is not required.

We need to pull md5sum publishing out of image publishing, create a new artifact publish block and actually publish md5sums in their own artifact block. I wouldn't be opposed to stripping that functionality into a new buildstep entirely, to be perfectly honest.

Needed:

- Strip md5sum publishing out of image artifacts in PublishArtifacts
- Do magic stuff (fix it)
- Fix any buildset-config that calls MakeMD5Sums
Comment 1 Lynn Cyrin 2014-05-21 22:08:12 UTC 
Relevant files:

/lib/python2.7/site-packages/autobuilder/buildsteps/MakeImageMD5s.py
/lib/python2.7/site-packages/autobuilder/buildsteps/PublishArtifacts.py

Recent commits:

http://git.yoctoproject.org/cgit/cgit.cgi/yocto-autobuilder/commit/?id=94aa8d3de3af79dcf8055370139f18047d25c3ff
Comment 2 Beth Flanagan 2014-05-21 22:31:59 UTC 
Yes, that commit was to fix the "Whoops. No tmp/deploy/images dir. I know, let's md5sum the entire build dir!" (BAD!) It shouldn't really effect this. Essentially if you look at how PublishArtifacts is called in buildset-config.master/* we should have like an 'md5sum' addition to it.