Bug 1230657 - Make Mozharness check for testPackagesUrl and packageUrl properties before trying to find TC artifacts. r=jlund

In read-buildbot-configs we added support to find artifacts through taskId's (set by Buildbot bridge).
This is useful when trying to schedule Buildbot build and associated test jobs via TaskCluster.
However, this is not good enough if there is a completed Buildbot build to which we want to add test jobs
(since it has no taskId set by the Buildbot bridge). In this case, we can determine the installer and test
url in Mozci and can add this information to the task as two properties.
In this patch, we add Mozharness the ability to consider the two properties before trying to use taskId's
to find the artifacts (which doesn't work).

This ultimately supports the recently added push-extender feature to Treeherder.

--HG--
extra : commitid : BESuP0CLna1
This commit is contained in:
Armen Zambrano Gasparnian 2015-12-07 09:06:58 -05:00
parent 0eaa01e45c
commit f746c2d5c8

View File

@ -309,8 +309,17 @@ class TestingMixin(VirtualenvMixin, BuildbotMixin, ResourceMonitoringMixin,
if c.get("test_packages_url"):
self.test_packages_url = c['test_packages_url']
# This supports original Buildbot to Buildbot mode
if self.buildbot_config['sourcestamp']['changes']:
self.find_artifacts_from_buildbot_changes()
# This supports TaskCluster/BBB task to Buildbot job
elif 'testPackagesUrl' in self.buildbot_config['properties'] and \
'packageUrl' in self.buildbot_config['properties']:
self.installer_url = self.buildbot_config['properties']['packageUrl']
self.test_packages_url = self.buildbot_config['properties']['testPackagesUrl']
# This supports TaskCluster/BBB task to TaskCluster/BBB task
elif 'taskId' in self.buildbot_config['properties']:
self.find_artifacts_from_taskcluster()