Summary: | results.webkit.org: Start reporting results | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jonathan Bedard <jbedard> | ||||||||||
Component: | Tools / Tests | Assignee: | Jonathan Bedard <jbedard> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | aakash_jain, dean_johnson, dewei_zhu, ews-watchlist, glenn, jlewis3, lingho, webkit-bug-importer | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
See Also: |
https://bugs.webkit.org/show_bug.cgi?id=202928 https://bugs.webkit.org/show_bug.cgi?id=203829 |
||||||||||||
Attachments: |
|
Description
Jonathan Bedard
2019-10-07 09:13:08 PDT
Created attachment 380335 [details]
Patch
Comment on attachment 380335 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=380335&action=review > Tools/BuildSlaveSupport/build.webkit.org-config/loadConfig.py:52 > + results_server_api_key = passwords.get('RESULTS_SERVER_API_KEY') We usually use lower case for config json keys? Committed r250784: <https://trac.webkit.org/changeset/250784> Reverted r250784 for reason: Broke running layout tests. Committed r250851: <https://trac.webkit.org/changeset/250851> (In reply to Matt Lewis from comment #5) > Reverted r250784 for reason: > > Broke running layout tests. > > Committed r250851: <https://trac.webkit.org/changeset/250851> This was at my request. Don't want to be rushed when root-causing this failure. Created attachment 380483 [details]
Patch
Created attachment 380536 [details]
Patch
Created attachment 380561 [details]
Patch
Comment on attachment 380561 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=380561&action=review > Tools/Scripts/webkitpy/results/upload_unittest.py:129 > + with mock.patch('requests.post', new=lambda url, headers={}, data={}: self.MockResponse()): Is this syntax valid? `data={}: self.MockResponse()`? Comment on attachment 380561 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=380561&action=review >> Tools/Scripts/webkitpy/results/upload_unittest.py:129 >> + with mock.patch('requests.post', new=lambda url, headers={}, data={}: self.MockResponse()): > > Is this syntax valid? `data={}: self.MockResponse()`? Never mind. Missed `lambda` Committed r250966: <https://trac.webkit.org/changeset/250966> |