Bug 1288434 - fix buildbot props in mark release as shipped. r=rail draft
authorMihai Tabara <mtabara@mozilla.com>
Tue, 30 Aug 2016 19:58:07 +0300
changeset 407602 0e1434f625f79e9c314f8b272911ce6a5e3daee0
parent 407578 1a6361b000fcb97f941e4091001e88be0e46927f
child 529902 091a135295faae07dc3c9ff55d82b6a3b6f1b94d
push id27994
push userbmo:mtabara@mozilla.com
push dateTue, 30 Aug 2016 16:59:14 +0000
reviewersrail
bugs1288434
milestone51.0a1
Bug 1288434 - fix buildbot props in mark release as shipped. r=rail 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 = {}