Support multiple repositories in submitting results to --results-server-host
Created attachment 214415 [details] Fixes the bug
Comment on attachment 214415 [details] Fixes the bug View in context: https://bugs.webkit.org/attachment.cgi?id=214415&action=review > Tools/Scripts/webkitpy/layout_tests/controllers/manager.py:402 > + ('master', 'build.webkit.org' if master_name == 'webkit.org' else master_name), # FIXME: Pass in build.webkit.org. webkit.org is hardcoded in TestResultsServer, won't this break the flakiness dashboard?
(In reply to comment #2) > (From update of attachment 214415 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=214415&action=review > > > Tools/Scripts/webkitpy/layout_tests/controllers/manager.py:402 > > + ('master', 'build.webkit.org' if master_name == 'webkit.org' else master_name), # FIXME: Pass in build.webkit.org. > > webkit.org is hardcoded in TestResultsServer, won't this break the flakiness dashboard? This code is not used by the existing TestResultsServer. It's for the new flakiness dashboard I'm creating.
Committed r157553: <http://trac.webkit.org/changeset/157553>
Comment on attachment 214415 [details] Fixes the bug View in context: https://bugs.webkit.org/attachment.cgi?id=214415&action=review >>> Tools/Scripts/webkitpy/layout_tests/controllers/manager.py:402 >>> + ('master', 'build.webkit.org' if master_name == 'webkit.org' else master_name), # FIXME: Pass in build.webkit.org. >> >> webkit.org is hardcoded in TestResultsServer, won't this break the flakiness dashboard? > > This code is not used by the existing TestResultsServer. It's for the new flakiness dashboard I'm creating. It's super confusing that code in layout_tests/controllers/manager.py is not universal.
(In reply to comment #5) > (From update of attachment 214415 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=214415&action=review > > >>> Tools/Scripts/webkitpy/layout_tests/controllers/manager.py:402 > >>> + ('master', 'build.webkit.org' if master_name == 'webkit.org' else master_name), # FIXME: Pass in build.webkit.org. > >> > >> webkit.org is hardcoded in TestResultsServer, won't this break the flakiness dashboard? > > > > This code is not used by the existing TestResultsServer. It's for the new flakiness dashboard I'm creating. > > It's super confusing that code in layout_tests/controllers/manager.py is not universal. Well, the idea is to get rid of the old code because it's a bloated mess.