Bug 1288434 - fix buildbot props in mark release as shipped. r=rail a=release DONTBUILD
authorMihai Tabara <mtabara@mozilla.com>
Tue, 30 Aug 2016 19:58:07 +0300
changeset 349977 16bf54c7ba56ef29033c9026e652de0a86d7a82a
parent 349976 c819e284a13937fc955c2efb6da7fdf9c5bb8b88
child 349978 72984b90a3534fafe0dcaf56c51a8876f59ec8c6
push id1230
push userjlund@mozilla.com
push dateMon, 31 Oct 2016 18:13:35 +0000
treeherdermozilla-release@5e06e3766db2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersrail, release
bugs1288434
milestone50.0a2
Bug 1288434 - fix buildbot props in mark release as shipped. r=rail a=release DONTBUILD MozReview-Commit-ID: ICEezfZDuO6
testing/mozharness/scripts/release/postrelease_mark_as_shipped.py
--- a/testing/mozharness/scripts/release/postrelease_mark_as_shipped.py
+++ b/testing/mozharness/scripts/release/postrelease_mark_as_shipped.py
@@ -61,32 +61,32 @@ class MarkReleaseAsShipped(BaseScript, V
         super(MarkReleaseAsShipped, self)._pre_config_lock(rw_config)
         # override properties from buildbot properties here as defined by
         # taskcluster properties
         self.read_buildbot_config()
         if not self.buildbot_config:
             self.warning("Skipping buildbot properties overrides")
             return
         props = self.buildbot_config['properties']
-        mandatory_props = ['product', 'version', 'buildnumber']
+        mandatory_props = ['product', 'version', 'build_number']
         missing_props = []
         for prop in mandatory_props:
-            if props.get(prop):
+            if prop in props:
                 self.info("Overriding %s with %s" % (prop, props[prop]))
                 self.config[prop] = props.get(prop)
             else:
                 self.warning("%s could not be found within buildprops" % prop)
                 missing_props.append(prop)
 
         if missing_props:
             raise Exception("%s not found in configs" % missing_props)
 
         self.config['name'] = build_release_name(self.config['product'],
                                                  self.config['version'],
-                                                 self.config['buildnumber'])
+                                                 self.config['build_number'])
 
     def mark_as_shipped(self):
         """Method to make a simple call to Ship-it API to change a release
         status to 'shipped'
         """
         credentials_file = os.path.join(os.getcwd(),
                                         self.config["credentials_file"])
         credentials = {}