Alexey pointed out to me that we're doing a pretty terrible job of distinguishing between our various different Linux ports in the results database. This carries over to WinCario too. Example: https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fcss%2Fcss-position%2Fposition-absolute-container-dynamic-002.html I'm going to suggest that we make the 'platform' for these match the 'port' from webkitpy and then, in the case of GTK, add a version_name based on the display driver.
<rdar://problem/56658244>
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.