Bug 1163465 - Remove HACK_first_released_version r=jlund draft
authorRail Aliiev <rail@mozilla.com>
Mon, 13 Feb 2017 15:25:37 -0500
changeset 7506 56a7b5fb82c3c1c359b99c6627e0ff313910010c
parent 7505 1b31afeb1600361c40e6bf5709a6f67ed2590f0a
push id185
push userbmo:rail@mozilla.com
push dateMon, 13 Feb 2017 20:25:48 +0000
reviewersjlund
bugs1163465
Bug 1163465 - Remove HACK_first_released_version r=jlund MozReview-Commit-ID: KrXwXnDeIJu
buildbot-helpers/release_sanity.py
--- 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(