diff --git a/misc.py b/misc.py index ee8b2e01..af00d059 100644 --- a/misc.py +++ b/misc.py @@ -205,6 +205,17 @@ def changeContainsProduct(change, productName): return False +def changeBaseTagContainsScriptRepoRevision(change, baseTag): + script_repo_revision = change.properties.getProperty("script_repo_revision") + baseTag = baseTag + "_" + if isinstance(script_repo_revision, basestring) and \ + baseTag in script_repo_revision: + log.msg("baseTag '%s' IS in script_repo_revision '%s'" % (baseTag, script_repo_revision)) + return True + log.msg("baseTag '%s' IS NOT in script_repo_revision '%s'" % (baseTag, script_repo_revision)) + return False + + def changeContainsProperties(change, props={}): for prop, value in props.iteritems(): if change.properties.getProperty(prop) != value: diff --git a/process/factory.py b/process/factory.py index 4a12bff5..141793d0 100644 --- a/process/factory.py +++ b/process/factory.py @@ -1956,7 +1956,7 @@ def addSubmitBalrogUpdates(self): cmd = [ self.env.get('PYTHON26', 'python'), WithProperties( - '%(toolsdir)s/scripts/updates/balrog-client.py'), + '%(toolsdir)s/scripts/updates/balrog-submitter.py'), '--build-properties', 'buildprops_balrog.json', '--api-root', self.balrog_api_root, '--verbose', diff --git a/process/release.py b/process/release.py index 0fea926a..90d61fa1 100644 --- a/process/release.py +++ b/process/release.py @@ -27,7 +27,8 @@ from buildbotcustom.status.mail import ChangeNotifier from buildbotcustom.misc import get_l10n_repositories, \ generateTestBuilderNames, generateTestBuilder, _nextFastSlave, \ - changeContainsProduct, nomergeBuilders, changeContainsProperties + changeContainsProduct, nomergeBuilders, changeContainsProperties, \ + changeBaseTagContainsScriptRepoRevision from buildbotcustom.common import normalizeName from buildbotcustom.process.factory import StagingRepositorySetupFactory, \ ScriptFactory, SingleSourceFactory, ReleaseBuildFactory, \ @@ -1581,7 +1582,8 @@ def getMessageId(): builderNames=[builderPrefix( '%s_reset_schedulers' % releaseConfig['productName'])], fileIsImportant=lambda c: changeContainsProduct(c, - releaseConfig['productName']) + releaseConfig['productName']) \ + and changeBaseTagContainsScriptRepoRevision(c, releaseConfig['baseTag']), ) schedulers.append(reset_schedulers_scheduler) if releaseConfig.get('enable_repo_setup'): diff --git a/steps/unittest.py b/steps/unittest.py index 660d0250..2b729f19 100644 --- a/steps/unittest.py +++ b/steps/unittest.py @@ -588,7 +588,11 @@ def __init__(self, suite, symbols_path=None, leakThreshold=None, **kwargs): self.name = suite - self.command = ['python', WithProperties("%(toolsdir)s/buildfarm/utils/run_jetpack.py"), '-p', WithProperties("%(platform)s")] + self.command = [ + 'python', 'jetpack/bin/cfx', + WithProperties('--binary=%(exepath)s'), + '--parseable', 'testpkgs' + ] # TODO: When jetpack can handle symbols path and leak testing, add those # until then, we skip that.