<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body bgcolor="#f7f7f7" text="#2c2c2c" link="#4a90d9" vlink="#4a90d9">
<div>On Thu, 2016-07-14 at 10:38 +0100, Joshua G Lock wrote:</div>
<blockquote type="cite">
<pre>On Wed, 2016-07-06 at 13:16 -0700, Bill Randle wrote:
<blockquote type="cite">
Publish the wic images created by the nightly-wic build so they are
available for QA testing.
[YOCTO #9397]
Signed-off-by: Bill Randle <<a href="mailto:william.c.randle@intel.com">william.c.randle@intel.com</a>>
---
buildset-config.controller/nightly-wic.conf | 8 ++---
.../autobuilder/buildsteps/PublishArtifacts.py | 35
++++++++++++++++------
2 files changed, 28 insertions(+), 15 deletions(-)
diff --git a/buildset-config.controller/nightly-wic.conf b/buildset-
config.controller/nightly-wic.conf
index 7d2ae88..27584cb 100644
--- a/buildset-config.controller/nightly-wic.conf
+++ b/buildset-config.controller/nightly-wic.conf
@@ -19,10 +19,6 @@ steps: [{'SetDest':{}},
{'BuildImages':{'images':'core-image-sato'}},
{'CreateWicImages':{'wic_img_type':'directdisk',
'target_img':'core-image-sato'}},
{'CreateWicImages':{'wic_img_type':'directdisk-gpt',
'target_img':'core-image-sato'}},
- {'CreateWicImages':{'wic_img_type':'mkefidisk',
'target_img':'core-image-sato'}}]
-
-
-
-
-
+ {'CreateWicImages':{'wic_img_type':'mkefidisk',
'target_img':'core-image-sato'}},
+ {'PublishArtifacts': {'artifacts': ['qemux86-64',
'genericx86-64']}}]
diff --git a/lib/python2.7/site-
packages/autobuilder/buildsteps/PublishArtifacts.py
b/lib/python2.7/site-
packages/autobuilder/buildsteps/PublishArtifacts.py
index 952a458..f9df16c 100644
--- a/lib/python2.7/site-
packages/autobuilder/buildsteps/PublishArtifacts.py
+++ b/lib/python2.7/site-
packages/autobuilder/buildsteps/PublishArtifacts.py
@@ -174,7 +174,14 @@ class PublishArtifacts(ShellCommand):
artifact_name, deploy_image_dir =
self.getDeployNames(artifact, buildername)
command += self.generateMD5cmd(artifact,
deploy_image_dir)
command=command+"mkdir -p " + DEST + "/" +
QEMU_PUBLISH_DIR + "/" + artifact_name + ";"
- command=command+"cp -R --no-dereference --
preserve=links " + \
+ if "-wic" in buildername:
+ deploy_image_dir += "/*/*/build";
+ command=command+"cp --no-dereference --
preserve=links " + \
+ deploy_image_dir + "/*\.direct "
+ \
+ deploy_image_dir +
"/*\.direct.md5sum " + \
+ DEST + "/" + QEMU_PUBLISH_DIR +
"/" + artifact_name + ";"
+ else:
+ command=command+"cp -R --no-dereference --
preserve=links " + \
deploy_image_dir + \
"/*" + artifact + "* " + DEST +
"/" + QEMU_PUBLISH_DIR + "/" + artifact_name + ";"
elif "mpc8315e" in artifact:
@@ -213,18 +220,26 @@ class PublishArtifacts(ShellCommand):
"genericx86-64" in artifact:
command = command+"echo 'Skipping copy of
genericx86-64.'; "
else:
- command=command+"mkdir -p " + DEST + "/"+
MACHINE_PUBLISH_DIR +"/" + artifact_name + ";"
- if "beagle" in artifact:
- command=command+"cp -R --no-dereference
--preserve=links " + \
+ command += self.generateMD5cmd(artifact,
deploy_image_dir)
</blockquote>
If I'm reading this patch correctly this change moves the addition of
generateMD5cmd() to the command string into the else block of the
"layerversion_yoctobsp < 2" check, whereas before it was appended
regardless after the if/else statement. Was that change intentional?
Regards,
Joshua
</pre>
</blockquote>
<div><br>
</div>
<div>Yes, it was intentional. If the layerversion is < 2 and you're expecting genericx86_64, that's not a supported configuration, so there is no image to run a md5sum on. We also want to run the md5sum calculation before we copy artifacts, rather than after,
so that the md5sum files are included in the copy.</div>
<div><br>
</div>
<div> -Bill</div>
<div><br>
</div>
<blockquote type="cite">
<pre><blockquote type="cite">
+ if "-wic" in buildername:
+ deploy_image_dir += "/*/*/build";
+ command=command+"mkdir -p " + DEST + "/"
+ MACHINE_PUBLISH_DIR + "/" + artifact_name + ";"
+ command=command+"cp --no-dereference --
preserve=links " + \
+ deploy_image_dir + "/*\.direct "
+ \
+ deploy_image_dir +
"/*\.direct.md5sum " + \
+ DEST + "/" + MACHINE_PUBLISH_DIR
+ "/" + artifact_name + ";"
+ else:
+ command=command+"mkdir -p " + DEST +
"/"+ MACHINE_PUBLISH_DIR +"/" + artifact_name + ";"
+ if "beagle" in artifact:
+ command=command+"cp -R --no-
dereference --preserve=links " + \
deploy_image_dir + \
"/*Image* " + DEST +
"/" + MACHINE_PUBLISH_DIR +"/" + artifact_name + ";"
- command=command+"cp -R --no-dereference
--preserve=links " + \
+ command=command+"cp -R --no-
dereference --preserve=links " + \
deploy_image_dir + \
"/u-boot* " + DEST +
"/" + MACHINE_PUBLISH_DIR +"/" + artifact_name + ";"
- command=command+"cp -R --no-dereference --
preserve=links " + \
+ command=command+"cp -R --no-dereference
--preserve=links " + \
deploy_image_dir + \
"/*"+artifact+"* " + DEST +
"/" + MACHINE_PUBLISH_DIR +"/" + artifact_name + ";"
- command += self.generateMD5cmd(artifact,
deploy_image_dir)
self.command = command
else:
self.command = "echo 'Skipping Step.'"
@@ -241,8 +256,10 @@ class PublishArtifacts(ShellCommand):
artifact_name = artifact
if "-lsb" in buildername:
artifact_name = artifact_name + "-lsb"
- if artifact_name is "md5sums":
- deploy_dir_image = os.path.join(self.tmpdir,
"deploy/images/")
+ if "-wic" in buildername:
+ deploy_dir_image =
os.path.join(os.path.join(self.tmpdir, "deploy/wic_images/"),
artifact)
+ elif artifact_name is "md5sums":
+ deploy_dir_image = os.path.join(self.tmpdir,
"deploy/images/")
else:
if self.layerversion_core is not None and
int(self.layerversion_core) > 2:
deploy_dir_image =
os.path.join(os.path.join(self.tmpdir, "deploy/images/"), artifact)
</blockquote></pre>
</blockquote>
</body>
</html>