218904 2020-11-13 08:03:58 -0800 [build.webkit.org] Use python3 to run transfer-to-s3 script 2021-02-08 07:37:49 -0800 1 1 1 Unclassified WebKit Tools / Tests Other Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 175056 1 aakash_jain aakash_jain aakash_jain jbedard ryanhaddad webkit-bug-importer oldest_to_newest 1707238 0 aakash_jain 2020-11-13 08:03:58 -0800 We are planning to upgrade build.webkit.org to latest Buildbot, which supports only python3. We should use python3 to run transfer-to-s3 script for newer Buildbot (this script is run on the server). 1707240 1 414043 aakash_jain 2020-11-13 08:07:01 -0800 Created attachment 414043 Patch 1707242 2 aakash_jain 2020-11-13 08:09:12 -0800 Sample run: https://build.webkit-dev.org/#/builders/52/builds/14/steps/11/logs/stdio 1707257 3 jbedard 2020-11-13 08:39:06 -0800 Not sure the claim in the bug makes sense. buildbot being Python3 doesn't prevent it from running Python 2 processes, right? We are still running on Mojave, we can't make this change until we stop supporting it. 1707260 4 jbedard 2020-11-13 08:44:11 -0800 (In reply to Jonathan Bedard from comment #3) > Not sure the claim in the bug makes sense. > > buildbot being Python3 doesn't prevent it from running Python 2 processes, > right? We are still running on Mojave, we can't make this change until we > stop supporting it. Turns out, this code is being run on the master, not the workers. 1707273 5 ews-feeder 2020-11-13 09:21:41 -0800 Committed r269781: <https://trac.webkit.org/changeset/269781> All reviewed patches have been landed. Closing bug and clearing flags on attachment 414043. 1707274 6 webkit-bug-importer 2020-11-13 09:22:18 -0800 <rdar://problem/71373851> 414043 2020-11-13 08:07:01 -0800 2020-11-13 09:21:43 -0800 Patch bug-218904-20201113110700.patch text/plain 1128 aakash_jain SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl dmlzaW9uIDI2OTc3OCkKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz ICsxLDEzIEBACisyMDIwLTExLTEzICBBYWthc2ggSmFpbiAgPGFha2FzaF9qYWluQGFwcGxlLmNv bT4KKworICAgICAgICBbYnVpbGQud2Via2l0Lm9yZ10gVXNlIHB5dGhvbjMgdG8gcnVuIHRyYW5z ZmVyLXRvLXMzIHNjcmlwdAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1 Zy5jZ2k/aWQ9MjE4OTA0CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK KyAgICAgICAgKiBDSVN1cHBvcnQvYnVpbGQtd2Via2l0LW9yZy9zdGVwcy5weToKKyAgICAgICAg KFRyYW5zZmVyVG9TMy5zdGFydCk6CisKIDIwMjAtMTEtMTMgIEtpbW1vIEtpbm51bmVuICA8a2tp bm51bmVuQGFwcGxlLmNvbT4KIAogICAgICAgICBBZGQgS2ltbW8gS2lubnVuZW4gYXMgY29tbWl0 dGVyCkluZGV4OiBUb29scy9DSVN1cHBvcnQvYnVpbGQtd2Via2l0LW9yZy9zdGVwcy5weQo9PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09Ci0tLSBUb29scy9DSVN1cHBvcnQvYnVpbGQtd2Via2l0LW9yZy9zdGVwcy5weQkocmV2 aXNpb24gMjY5Nzc4KQorKysgVG9vbHMvQ0lTdXBwb3J0L2J1aWxkLXdlYmtpdC1vcmcvc3RlcHMu cHkJKHdvcmtpbmcgY29weSkKQEAgLTExMzIsNiArMTEzMiw4IEBAIGNsYXNzIFRyYW5zZmVyVG9T MyhtYXN0ZXIuTWFzdGVyU2hlbGxDb20KICAgICAgICAgbWFzdGVyLk1hc3RlclNoZWxsQ29tbWFu ZC5fX2luaXRfXyhzZWxmLCAqKmt3YXJncykKIAogICAgIGRlZiBzdGFydChzZWxmKToKKyAgICAg ICAgaWYgVVNFX0JVSUxEQk9UX1ZFUlNJT04yOgorICAgICAgICAgICAgc2VsZi5jb21tYW5kWzBd ID0gJ3B5dGhvbjMnCiAgICAgICAgIHJldHVybiBtYXN0ZXIuTWFzdGVyU2hlbGxDb21tYW5kLnN0 YXJ0KHNlbGYpCiAKICAgICBkZWYgZmluaXNoZWQoc2VsZiwgcmVzdWx0KToK