Closed Bug 1586396 Opened 2 months ago Closed Last month

WebGPU thread mock

Categories

(Core :: Graphics: WebGPU, task, P3)

task

Tracking

()

RESOLVED FIXED
mozilla71
Tracking Status
firefox71 --- fixed

People

(Reporter: kvark, Assigned: kvark)

References

Details

Attachments

(2 files)

This task is about adding a pref as well as some initialization code for WebGPU client/server.

Assignee: nobody → dmalyshau
Blocks: 1575008
Priority: -- → P3

follow-up to D48229

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&searchStr=wpt&fromchange=2233060e1f08ae399d0a2a67c5457d63467a3ae7&tochange=213b74bb0fa9d04dc84c36e834170dfde67f3d1f&selectedJob=271451542

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=271451532&repo=autoland&lineNumber=1277

Backout link: https://hg.mozilla.org/integration/autoland/rev/213b74bb0fa9d04dc84c36e834170dfde67f3d1f

[task 2019-10-16T04:53:32.496Z] 04:53:32 INFO - 'USER': 'worker',
[task 2019-10-16T04:53:32.496Z] 04:53:32 INFO - 'WORKING_DIR': '/builds/worker'}
[task 2019-10-16T04:53:32.496Z] 04:53:32 INFO - Calling ['/builds/worker/workspace/build/venv/bin/python', '-u', '/builds/worker/workspace/build/tests/web-platform/runtests.py', '--log-raw=-', '--log-raw=/builds/worker/workspace/build/blobber_upload_dir/wpt_raw.log', '--log-wptreport=/builds/worker/workspace/build/blobber_upload_dir/wptreport.json', '--log-errorsummary=/builds/worker/workspace/build/blobber_upload_dir/wpt_errorsummary.log', '--binary=/tmp', '--symbols-path=/builds/worker/workspace/build/symbols', '--stackwalk-binary=/builds/worker/workspace/build/linux64-minidump_stackwalk', '--stackfix-dir=/builds/worker/workspace/build/tests/bin', '--run-by-dir=3', '--no-pause-after-test', '--device-serial=emulator-5554', '--package-name=org.mozilla.geckoview.test', '--install-fonts', '--test-type=testharness', '--setpref=media.peerconnection.mtransport_process=false', '--setpref=network.process.enabled=false', '--stylo-threads=4', '--total-chunks=18', '--this-chunk=14', '--prefs-root=/builds/worker/workspace/build/tests/web-platform/prefs', '--processes=1', '--config=/builds/worker/workspace/build/tests/web-platform/wptrunner.ini', '--ca-cert-path=/builds/worker/workspace/build/tests/web-platform/tests/tools/certs/cacert.pem', '--host-key-path=/builds/worker/workspace/build/tests/web-platform/tests/tools/certs/web-platform.test.key', '--host-cert-path=/builds/worker/workspace/build/tests/web-platform/tests/tools/certs/web-platform.test.pem', '--certutil-binary=/builds/worker/workspace/build/hostutils/host-utils-71.0a1.en-US.linux-x86_64/certutil', '--product=firefox_android'] with output_timeout 1000
[task 2019-10-16T04:53:32.507Z] 04:53:32 CRITICAL - Test harness output was not a valid structured log message:
[task 2019-10-16T04:53:32.507Z] 04:53:32 CRITICAL - Traceback (most recent call last):
[task 2019-10-16T04:53:32.507Z] 04:53:32 CRITICAL - File "/builds/worker/workspace/build/tests/web-platform/runtests.py", line 13, in <module>
[task 2019-10-16T04:53:32.507Z] 04:53:32 CRITICAL - from wptrunner import wptrunner
[task 2019-10-16T04:53:32.507Z] 04:53:32 CRITICAL - ImportError: No module named wptrunner
[task 2019-10-16T04:53:32.515Z] 04:53:32 ERROR - Return code: 1
[task 2019-10-16T04:53:32.515Z] 04:53:32 ERROR - No checks run.
[task 2019-10-16T04:53:32.515Z] 04:53:32 ERROR - No suite end message was emitted by this harness.
[task 2019-10-16T04:53:32.515Z] 04:53:32 CRITICAL - # TBPL FAILURE #
[task 2019-10-16T04:53:32.515Z] 04:53:32 WARNING - setting return code to 2

Flags: needinfo?(dmalyshau)
Status: NEW → RESOLVED
Closed: Last month
Resolution: --- → FIXED
Target Milestone: --- → mozilla71

Cosmin, the error is completely unrelated ("EnvironmentError: http server on port 8000 failed to start") to the change. Are you sure reverting this change helped?

Flags: needinfo?(dmalyshau) → needinfo?(csabou)

Dzmitry, you are right, that looks more like a network hiccup. Sorry about the trouble. I would reland your changes in D49367 but it wasn't reviewed yet. Please wait for review before landing it.

Flags: needinfo?(csabou) → needinfo?(dmalyshau)

Cosmin, the change contains one line suggested by the reviewer in https://phabricator.services.mozilla.com/D48229 plus the auto-formatting fixes done by clang-format. I don't think it needs any additional review.

Flags: needinfo?(dmalyshau) → needinfo?(csabou)
Pushed by csabou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ab675f6a7e26
Formatting fixes in WebGPUThreading
Flags: needinfo?(csabou)
You need to log in before you can comment on or make changes to this bug.