Closed Bug 1576971 Opened 1 year ago Closed 1 year ago

browsertime toolchain task doesn't fail hard enough


(Testing :: Performance, defect)

Version 3
Not set


(firefox70 fixed)

Tracking Status
firefox70 --- fixed


(Reporter: nalexander, Assigned: nalexander)





(2 files)

Bug 1564256 added a toolchain-browsertime task. It can fail during download, like this task:

[task 2019-08-26T21:53:45.754Z] # We don't install ImageMagick, so this will fail.  Continue.
[task 2019-08-26T21:53:45.754Z] ./mach browsertime --setup || true
[task 2019-08-26T21:53:45.754Z] + ./mach browsertime --setup
[task 2019-08-26T21:53:46.385Z] 
[task 2019-08-26T21:53:46.386Z] Creating default state directory: /builds/worker/.mozbuild
[task 2019-08-26T21:53:46.386Z]  0:00.54 Downloading to temporary location /builds/worker/.mozbuild/cache/browsertime/d64f870ceed29d8a-ffmpeg-4.0.3-64bit-static.tar.xz
[task 2019-08-26T21:53:46.720Z]  0:00.87 Downloading... 0.0 %
[task 2019-08-26T21:53:46.721Z]  0:00.87 Downloading... 100.0 %
[task 2019-08-26T21:53:46.724Z] Error running mach:
[task 2019-08-26T21:53:46.724Z] 
[task 2019-08-26T21:53:46.724Z]     ['browsertime', '--setup']
[task 2019-08-26T21:53:46.724Z] 
[task 2019-08-26T21:53:46.724Z] The error occurred in code that was called by the mach command. This is either
[task 2019-08-26T21:53:46.724Z] a bug in the called code itself or in the way that mach is calling it.
[task 2019-08-26T21:53:46.724Z] You can invoke |./mach busted| to check if this issue is already on file. If it
[task 2019-08-26T21:53:46.724Z] isn't, please use |./mach busted file| to report it. If |./mach busted| is
[task 2019-08-26T21:53:46.724Z] misbehaving, you can also inspect the dependencies of bug 1543241.
[task 2019-08-26T21:53:46.724Z] 
[task 2019-08-26T21:53:46.724Z] If filing a bug, please include the full output of mach, including this error
[task 2019-08-26T21:53:46.724Z] message.
[task 2019-08-26T21:53:46.724Z] 
[task 2019-08-26T21:53:46.724Z] The details of the failure are as follows:
[task 2019-08-26T21:53:46.724Z] 
[task 2019-08-26T21:53:46.724Z] HTTPError: 406 Client Error: Not Acceptable for url:
[task 2019-08-26T21:53:46.724Z] 
[task 2019-08-26T21:53:46.724Z]   File "/builds/worker/workspace/build/src/tools/browsertime/", line 399, in browsertime
[task 2019-08-26T21:53:46.724Z]     return self.setup(should_clobber=clobber)
[task 2019-08-26T21:53:46.724Z]   File "/builds/worker/workspace/build/src/tools/browsertime/", line 197, in setup
[task 2019-08-26T21:53:46.724Z]     archive = artifact_cache.fetch(fetch['url'])
[task 2019-08-26T21:53:46.724Z]   File "/builds/worker/workspace/build/src/python/mozbuild/mozbuild/", line 208, in fetch
[task 2019-08-26T21:53:46.725Z]     dl.wait()
[task 2019-08-26T21:53:46.725Z]   File "/builds/worker/workspace/build/src/third_party/python/dlmanager/dlmanager/", line 101, in wait
[task 2019-08-26T21:53:46.725Z]     self.raise_if_error()
[task 2019-08-26T21:53:46.725Z]   File "/builds/worker/workspace/build/src/third_party/python/dlmanager/dlmanager/", line 116, in raise_if_error
[task 2019-08-26T21:53:46.725Z]     six.reraise(*self.__error)
[task 2019-08-26T21:53:46.725Z]   File "/builds/worker/workspace/build/src/third_party/python/dlmanager/dlmanager/", line 175, in _download
[task 2019-08-26T21:53:46.725Z]     response.raise_for_status()
[task 2019-08-26T21:53:46.725Z]   File "/builds/worker/workspace/build/src/third_party/python/requests/requests/", line 840, in raise_for_status
[task 2019-08-26T21:53:46.725Z]     raise HTTPError(http_error_msg, response=self)
[task 2019-08-26T21:53:46.743Z] + true

That's not good. This ticket tracks not running --check rather than suppressing all errors.

In automation, we install ffmpeg as part of mach browsertime --setup in the browsertime toolchain task. Those tasks run on Linux
64 from within AWS, and most of the hosts we hit (intermittently) deny
AWS traffic. Let's just use in automation (and locally),
for all platforms, which will agree with upcoming fetch tasks.

Depends on D43655

Pushed by
Fail browsertime toolchain harder. r=barret
Post: Use for fetching browsertime binary dependencies. r=barret
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.