enhance logging again draft
authorJohan Lorenzo <jlorenzo@mozilla.com>
Thu, 01 Sep 2016 19:23:24 +0200
changeset 7138 be5df5d9b6d54735f52561a3db4ef8c1bd37564d
parent 7137 a32a72e38ac1b28e5bd03899d5837567b8db81e4
child 7139 b10e6802c9e24bcab3345f9a6fc627a90c5d68a7
push id122
push userjlorenzo@mozilla.com
push dateThu, 01 Sep 2016 17:23:37 +0000
enhance logging again MozReview-Commit-ID: HGrnui04Wk8
buildfarm/release/build_status.py
buildfarm/release/release-runner.py
--- a/buildfarm/release/build_status.py
+++ b/buildfarm/release/build_status.py
@@ -12,17 +12,17 @@ PLATFORMS_TO_WAIT_ON = {
 }
 
 
 def are_en_us_builds_completed(index, queue, release):
     product = release['product']
     platforms = PLATFORMS_TO_WAIT_ON[product]
     # XXX Doesn't work with Thunderbird
     branch = release['branch'].split('/')[-1]
-    log.debug('Product "%s" detected. Looking up platforms %s', product, platforms)
+    log.debug('Product "%s" detected. Looking up platforms: %s', product, platforms)
 
     try:
         tasks_to_watch = [
             task_for_revision(
                 index, branch=branch, revision=release['mozillaRevision'], product=product, platform=platform
             )['taskId']
             for platform in platforms
         ]
--- a/buildfarm/release/release-runner.py
+++ b/buildfarm/release/release-runner.py
@@ -335,21 +335,21 @@ def main(options):
         postrelease_mark_as_shipped_enabled = branchConfig['postrelease_mark_as_shipped_enabled']
         final_verify_channels = release_channels
         publish_to_balrog_channels = release_channels
         push_to_releases_enabled = True
 
     rc = 0
     for release in rr.new_releases:
         if not are_en_us_builds_completed(index, queue, release):
-            log.info('Builds for release "%s" are not completed yet. Skipping this one for now', release)
+            log.info('Builds are not completed yet, skipping release for now. Release:', release)
             rr.update_status(release, 'Waiting for builds to be completed')
             continue
 
-        log.info('Every build for release "%s" is completed!', release)
+        log.info('Every build is completed for release:', release)
         graph_id = slugId()
         try:
             rr.update_status(release, 'Generating task graph')
             l10n_changesets = parsePlainL10nChangesets(rr.get_release_l10n(release["name"]))
 
             kwargs = {
                 "public_key": docker_worker_key,
                 "version": release["version"],