Changeset 242185 in webkit

Timestamp:

Feb 27, 2019 5:47:03 PM (5 years ago)

Author:

aakash_jain@apple.com

Message:

[ews-build] Buildbot should include builder_display_name in the build events
​https://bugs.webkit.org/show_bug.cgi?id=195045

Reviewed by Dewei Zhu.

• BuildSlaveSupport/ews-build/events.py:

(Events.buildStarted): Included builder_display_name in event data. Also renamed
buildername to builder_name to be consistent in naming style.
(Events.buildFinished): Ditto.

Location:

trunk/Tools

Files:

• BuildSlaveSupport/ews-build/events.py (modified) (4 diffs)
• ChangeLog (modified) (1 diff)

trunk/Tools/BuildSlaveSupport/ews-build/events.py

@@ -110 +110 @@
             build['properties'] = yield self.master.db.builds.getBuildProperties(build.get('buildid'))
 
+        builder = yield self.master.db.builders.getBuilder(build.get('builderid'))
+        builder_display_name = builder.get('description')
+
         data = {
             "type": self.type_prefix + "build",
@@ -121 +124 @@
             "complete_at": build.get('complete_at'),
             "state_string": build.get('state_string'),
-            "buildername": self.getBuilderName(build),
+            "builder_name": self.getBuilderName(build),
+            "builder_display_name": builder_display_name,
         }
 
@@ -132 +136 @@
         if not build.get('steps'):
             build['steps'] = yield self.master.db.steps.getSteps(build.get('buildid'))
+
+        builder = yield self.master.db.builders.getBuilder(build.get('builderid'))
+        builder_display_name = builder.get('description')
 
         data = {
@@ -144 +151 @@
             "complete_at": build.get('complete_at'),
             "state_string": build.get('state_string'),
-            "buildername": self.getBuilderName(build),
+            "builder_name": self.getBuilderName(build),
+            "builder_display_name": builder_display_name,
             "steps": build.get('steps'),
         }

trunk/Tools/ChangeLog

@@ +1 @@
+2019-02-27  Aakash Jain  <aakash_jain@apple.com>
+
+        [ews-build] Buildbot should include builder_display_name in the build events
+        https://bugs.webkit.org/show_bug.cgi?id=195045
+
+        Reviewed by Dewei Zhu.
+
+        * BuildSlaveSupport/ews-build/events.py:
+        (Events.buildStarted): Included builder_display_name in event data. Also renamed
+        buildername to builder_name to be consistent in naming style.
+        (Events.buildFinished): Ditto.
+
 2019-02-27  Chris Dumez  <cdumez@apple.com>