third_party/WebKit/Tools/Scripts/webkitpy/thirdparty/wpt/wpt/manifest is slow (~50 seconds?) |
||||
Issue descriptionWhen running Layout tests the following command seems to be run; /b/s/w/ir/third_party/WebKit/Tools/Scripts/webkitpy/thirdparty/wpt/wpt/manifest --work --tests-root /b/s/w/ir/third_party/WebKit/LayoutTests/external/wpt This command seems to take roughly ~50 seconds to complete as the log below shows; ----------------- 10:32:46.997 1872 Running command: python /b/s/w/ir/third_party/WebKit/Tools/Scripts/webkitpy/thirdparty/wpt/wpt/manifest --work --tests-root /b/s/w/ir/third_party/WebKit/LayoutTests/external/wpt 10:33:22.070 1872 chunk slice [54054:57915] of 69489 is 3861 tests -----------------
,
Apr 24 2017
jeffcarp@ do you know anything about this? dpranke@ with qyearsley out until 8th of May, is there a good person to look into this? This might be unfairly blaming wpt and there is something which isn't logging that is taking up the time.
,
Apr 24 2017
Yes, I wrote the CL to add this functionality: https://crrev.com/c/447959/ We realized a downside is that manifest generation time would go up if WPT_BASE_MANIFEST.json got out of date. However, imports (which update that file) have been happening regularly, so it'd be good to verify on a current local checkout to see if this is the cause. I'm currently troopering today and tomorrow but I can check this soon when I get time.
,
Apr 26 2017
How do you decide when to do the generation?
,
Apr 26 2017
It runs as part of WPT import at 6am PT. If an import is successful, the base manifest gets updated. The last few imports have been successful: https://build.chromium.org/p/chromium.infra.cron/builders/wpt-importer Today I'll try reproducing this slowness on my workstation.
,
Apr 26 2017
On ToT on my workstation the manifest generation step takes between 2-10 seconds. On a machine with less resources I could see it taking longer, but 50 seconds is a big jump. We could add some timing instrumentation code to manifest generation to help diagnose this problem on swarming. Does that sound like a good next step?
,
Apr 27 2017
Let's start by putting in a log call after it finishes to see if it is actually the culprit...
,
Apr 28 2017
,
May 3 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a46809a3fbc319f5679b0c55102c870602119ac7 commit a46809a3fbc319f5679b0c55102c870602119ac7 Author: jeffcarp <jeffcarp@chromium.org> Date: Wed May 03 03:39:20 2017 Add WPTManifest log line to diagnose slow manifest generation BUG= 714503 Review-Url: https://codereview.chromium.org/2848353002 Cr-Commit-Position: refs/heads/master@{#468884} [modify] https://crrev.com/a46809a3fbc319f5679b0c55102c870602119ac7/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/controllers/manager.py [modify] https://crrev.com/a46809a3fbc319f5679b0c55102c870602119ac7/third_party/WebKit/Tools/Scripts/webkitpy/w3c/wpt_manifest.py
,
May 25 2017
Looking at a sample of logs of shards from recent webkit_layout_test jobs on https://luci-milo.appspot.com/buildbot/chromium.fyi/WebKit%20Linux%20-%20RandomOrder/, it seems like the manifest generation is taking about 5-10 seconds recently. Maybe it's possible that WPT_BASE_MANIFEST.json was just out of date on April 24, and as long as it's kept up to date we can expect a time of around 5 seconds for updating the manifest?
,
Jun 19 2017
Closing since this seems to no longer be an issue. |
||||
►
Sign in to add a comment |
||||
Comment 1 by tansell@chromium.org
, Apr 24 2017