Title: [242185] trunk/Tools
- Revision
- 242185
- Author
- aakash_j...@apple.com
- Date
- 2019-02-27 17:47:03 -0800 (Wed, 27 Feb 2019)
Log 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.
Modified Paths
Diff
Modified: trunk/Tools/BuildSlaveSupport/ews-build/events.py (242184 => 242185)
--- trunk/Tools/BuildSlaveSupport/ews-build/events.py 2019-02-28 00:58:13 UTC (rev 242184)
+++ trunk/Tools/BuildSlaveSupport/ews-build/events.py 2019-02-28 01:47:03 UTC (rev 242185)
@@ -109,6 +109,9 @@
if not build.get('properties'):
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",
"status": "started",
@@ -120,7 +123,8 @@
"started_at": build.get('started_at'),
"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,
}
self.sendData(data)
@@ -132,6 +136,9 @@
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 = {
"type": self.type_prefix + "build",
"status": "finished",
@@ -143,7 +150,8 @@
"started_at": build.get('started_at'),
"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'),
}
Modified: trunk/Tools/ChangeLog (242184 => 242185)
--- trunk/Tools/ChangeLog 2019-02-28 00:58:13 UTC (rev 242184)
+++ trunk/Tools/ChangeLog 2019-02-28 01:47:03 UTC (rev 242185)
@@ -1,3 +1,15 @@
+2019-02-27 Aakash Jain <aakash_j...@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 <cdu...@apple.com>
Flaky API Test: TestWebKitAPI.ProcessSwap.SessionStorage
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes