Merge lp:~qzhang/lava-dispatcher/fix-886102 into lp:lava-dispatcher

Proposed by Spring Zhang
Status: Merged
Merged at revision: 154
Proposed branch: lp:~qzhang/lava-dispatcher/fix-886102
Merge into: lp:lava-dispatcher
Diff against target: 17 lines (+7/-0)
1 file modified
lava_dispatcher/actions/deploy.py (+7/-0)
To merge this branch: bzr merge lp:~qzhang/lava-dispatcher/fix-886102
Reviewer Review Type Date Requested Status
Yongqin Liu Approve
Review via email: mp+81324@code.launchpad.net

Description of the change

include version in metadata

To post a comment you must log in.
Revision history for this message
Spring Zhang (qzhang) wrote :

Not tested yet, will report later.

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Looks basically fine, but I'm quite jetlagged so don't take that too
seriously. I think the attribute name is a bit abbreviated --
'target.linaro-media-create-version' would be clearer I think!

Revision history for this message
Yongqin Liu (liuyq0307) wrote :

it seems ok,
thanks

review: Approve
155. By Spring Zhang

modify metadata name

Revision history for this message
Spring Zhang (qzhang) wrote :

modified by comments, tested without error, but I can't find the bundle view in my local deployment lava-dashboard, so I can't verify if the metadata is updated.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lava_dispatcher/actions/deploy.py'
--- lava_dispatcher/actions/deploy.py 2011-10-20 15:20:56 +0000
+++ lava_dispatcher/actions/deploy.py 2011-11-08 05:35:26 +0000
@@ -155,6 +155,13 @@
155 logging.info("Downloading the %s file" % rootfs_url)155 logging.info("Downloading the %s file" % rootfs_url)
156 rootfs_path = download(rootfs_url, tarball_dir)156 rootfs_path = download(rootfs_url, tarball_dir)
157157
158 logging.info("linaro-media-create version information")
159 cmd = "sudo linaro-media-create -v"
160 rc, output = getstatusoutput(cmd)
161 metadata = self.context.test_data.get_metadata()
162 metadata['target.linaro-media-create-version'] = output
163 self.context.test_data.add_metadata(metadata)
164
158 image_file = os.path.join(tarball_dir, "lava.img")165 image_file = os.path.join(tarball_dir, "lava.img")
159 #XXX Hack for removing startupfiles from snowball hwpacks166 #XXX Hack for removing startupfiles from snowball hwpacks
160 if client.device_type == "snowball_sd":167 if client.device_type == "snowball_sd":

Subscribers

People subscribed via source and target branches