Bug 1163465 - Remove HACK_first_released_version r=jlund
MozReview-Commit-ID: KrXwXnDeIJu
--- a/buildbot-helpers/release_sanity.py
+++ b/buildbot-helpers/release_sanity.py
@@ -193,28 +193,22 @@ def verify_options(cmd_options, config):
if not cmd_options.masters_json_file:
log.error("masters json file is required when not skipping reconfig")
success = False
error_tally.add('masters_json_file')
return success
def verify_partial(platforms, product, version, build_number,
- HACK_first_released_versions=None, protocol='http',
- server='ftp.mozilla.org'):
+ protocol='http', server='archive.mozilla.org'):
- from distutils.version import LooseVersion
partial = Partial(product, version, build_number, protocol, server)
log.info("Checking for existence of %s complete mar file..." % partial)
complete_mar_name = partial.complete_mar_name()
for platform in platforms:
- if HACK_first_released_versions and platform in HACK_first_released_versions:
- if LooseVersion(version) < LooseVersion(HACK_first_released_versions[platform]):
- # No partial for this!
- continue
log.info("Platform: %s" % platform)
complete_mar_url = partial.complete_mar_url(platform=platform)
if partial.exists(platform=platform):
log.info("complete mar: %s exists, url: %s" % (complete_mar_name,
complete_mar_url))
else:
log.error("Requested file, %s, does not exist on %s"
" Check again, or use -b to bypass" % (complete_mar_name,
@@ -467,17 +461,16 @@ if __name__ == '__main__':
if partials:
for partial in partials:
build_number = partials[partial]['buildNumber']
# when bug 839926 lands, buildNumber must be None for releases
# but it might have a value for betas (beta *might* use
# unreleased builds see bug 1091694 c2)
if not verify_partial(platforms, product, partial,
build_number,
- releaseConfig.get("HACK_first_released_version"),
server=releaseConfig['ftpServer']):
test_success = False
log.error("Error verifying partials")
if test_success:
if not options.dryrun:
if not options.skip_reconfig:
fabric_helper = FabricHelper(