Bug 6354 - PublishArtifacts needs md5sums as an artifact.
Summary: PublishArtifacts needs md5sums as an artifact.
Status: RESOLVED FIXED
Alias: None
Product: AutoBuilder
Classification: Infrastructure
Component: autobuilder (show other bugs)
Version: 1.7
Hardware: x86 Multiple
: Medium enhancement
Target Milestone: 1.7
Assignee: Beth Flanagan
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2014-05-20 18:26 UTC by Beth Flanagan
Modified: 2014-09-09 19:25 UTC (History)
5 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.