Summary: | results.webkit.org: Report more descriptive configurations for non-Apple ports | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jonathan Bedard <jbedard> | ||||||
Component: | Tools / Tests | Assignee: | Jonathan Bedard <jbedard> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | aakash_jain, ap, clopez, commit-queue, don.olmstead, ews-watchlist, glenn, lingho, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Jonathan Bedard
2019-10-28 10:02:08 PDT
Created attachment 382078 [details]
Patch
Comment on attachment 382078 [details]
Patch
Patch looks fine. But there are some webkitpy test errors from the EWS.
Created attachment 382080 [details]
Patch
(In reply to Carlos Alberto Lopez Perez from comment #3) > Comment on attachment 382078 [details] > Patch > > Patch looks fine. But there are some webkitpy test errors from the EWS. Yep, that's what I get for not running the tests locally before uploading... Comment on attachment 382080 [details]
Patch
r=me. thanks!
Comment on attachment 382080 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=382080&action=review > Tools/Scripts/webkitpy/port/gtk.py:256 > + configuration['platform'] = 'GTK' is the capitalization intention? (In reply to Aakash Jain from comment #7) > Comment on attachment 382080 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=382080&action=review > > > Tools/Scripts/webkitpy/port/gtk.py:256 > > + configuration['platform'] = 'GTK' > > is the capitalization intention? It is, it's trying to match the behavior of the old database and buildbot: https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#builder=GTK%20Linux%2064-bit%20Debug%20(Tests) The commit-queue encountered the following flaky tests while processing attachment 382080 [details]: imported/w3c/web-platform-tests/css/css-position/position-absolute-container-dynamic-002.html bug 203473 (author: simon.fraser@apple.com) imported/w3c/web-platform-tests/css/css-position/position-absolute-crash-chrome-005.html bug 203474 (author: simon.fraser@apple.com) imported/w3c/web-platform-tests/websockets/bufferedAmount-unchanged-by-sync-xhr.any.worker.html bug 202003 (author: youennf@gmail.com) The commit-queue is continuing to process your patch. Comment on attachment 382080 [details] Patch Clearing flags on attachment: 382080 Committed r251661: <https://trac.webkit.org/changeset/251661> All reviewed patches have been landed. Closing bug. > Example: https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fcss%2Fcss-position%2Fposition-absolute-container-dynamic-002.html We should further standardize the naming. I see "Xvfb wk2 Release" on this link now, but that doesn't match any of the queue names on https://build.webkit.org/dashboard/ (In reply to Alexey Proskuryakov from comment #12) > > Example: https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fcss%2Fcss-position%2Fposition-absolute-container-dynamic-002.html > > We should further standardize the naming. I see "Xvfb wk2 Release" on this > link now, but that doesn't match any of the queue names on > https://build.webkit.org/dashboard/ That would be GTK, but yeah, I agree, the name we're displaying isn't quite right. (In reply to Alexey Proskuryakov from comment #12) > > Example: https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fcss%2Fcss-position%2Fposition-absolute-container-dynamic-002.html > > We should further standardize the naming. I see "Xvfb wk2 Release" on this > link now, but that doesn't match any of the queue names on > https://build.webkit.org/dashboard/ For layout tests for GTK we basically have two types of bots: - Wayland - Xvfb (but the name Xvfb is usually omitted) So I suggest to remove "Xvfb" from the name here. |