|
|
Created:
4 years ago by Geoff Lang Modified:
4 years ago CC:
chromium-reviews, jam, darin-cc_chromium.org, piman+watch_chromium.org, Jamie Madill Target Ref:
refs/pending/heads/master Project:
chromium Visibility:
Public. |
DescriptionRun the dEQP GLES 3.1 tests on Windows and Linux NVIDIA OpenGL.
BUG=angleproject:1442
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
Committed: https://crrev.com/366f3af1194430904d8d3587598f7d02213446bd
Cr-Commit-Position: refs/heads/master@{#435650}
Patch Set 1 #
Total comments: 2
Patch Set 2 : Forward args to the dEQP runner. #
Messages
Total messages: 31 (22 generated)
Description was changed from ========== Run the dEQP GLES 3.1 tests on Windows and Linux NVIDIA OpenGL. BUG=angleproject:1442 ========== to ========== Run the dEQP GLES 3.1 tests on Windows and Linux NVIDIA OpenGL. BUG=angleproject:1442 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ==========
Description was changed from ========== Run the dEQP GLES 3.1 tests on Windows and Linux NVIDIA OpenGL. BUG=angleproject:1442 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ========== to ========== Run the dEQP GLES 3.1 tests on Windows and Linux NVIDIA OpenGL. BUG=angleproject:1442 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.mac:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ==========
Description was changed from ========== Run the dEQP GLES 3.1 tests on Windows and Linux NVIDIA OpenGL. BUG=angleproject:1442 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.mac:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ========== to ========== Run the dEQP GLES 3.1 tests on Windows and Linux NVIDIA OpenGL. BUG=angleproject:1442 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.mac:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel ==========
The CQ bit was checked by geofflang@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
Description was changed from ========== Run the dEQP GLES 3.1 tests on Windows and Linux NVIDIA OpenGL. BUG=angleproject:1442 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.mac:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel ========== to ========== Run the dEQP GLES 3.1 tests on Windows and Linux NVIDIA OpenGL. BUG=angleproject:1442 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ==========
The CQ bit was unchecked by geofflang@chromium.org
The CQ bit was checked by geofflang@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
geofflang@chromium.org changed reviewers: + kbr@chromium.org
PTAL
jmadill@chromium.org changed reviewers: + jmadill@chromium.org
https://codereview.chromium.org/2542653002/diff/1/content/test/gpu/generate_b... File content/test/gpu/generate_buildbot_json.py (right): https://codereview.chromium.org/2542653002/diff/1/content/test/gpu/generate_b... content/test/gpu/generate_buildbot_json.py:866: 'shards': 4, There are more tests in the gles31 test set than gles3, but they run super quickly now because of so many skipped tests due to missing features. The gles3 tests use 12 shards -- please add a TODO here to increase the shard count as the tests become slower with more tests enabled.
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
lgtm with Jamie's concern addressed.
One additional thing Geoff: you'll need to pass the command line flags to angle_deqp_gtest.cpp in angle_deqp_tests_main.cc https://cs.chromium.org/chromium/src/gpu/angle_deqp_tests_main.cc I don't see that they have been added yet.
Updated the main file to pass the args. https://codereview.chromium.org/2542653002/diff/1/content/test/gpu/generate_b... File content/test/gpu/generate_buildbot_json.py (right): https://codereview.chromium.org/2542653002/diff/1/content/test/gpu/generate_b... content/test/gpu/generate_buildbot_json.py:866: 'shards': 4, On 2016/11/30 18:38:37, Jamie Madill wrote: > There are more tests in the gles31 test set than gles3, but they run super > quickly now because of so many skipped tests due to missing features. The gles3 > tests use 12 shards -- please add a TODO here to increase the shard count as the > tests become slower with more tests enabled. Done.
lgtm! I like that you didn't import the header.
The CQ bit was checked by geofflang@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
The CQ bit was checked by geofflang@chromium.org
The patchset sent to the CQ was uploaded after l-g-t-m from kbr@chromium.org Link to the patchset: https://codereview.chromium.org/2542653002/#ps20001 (title: "Forward args to the dEQP runner.")
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
CQ is committing da patch. Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480615384300650, "parent_rev": "152179aa181ba20d3da8a1825a15dd7fe17d964d", "commit_rev": "1d51aaf9880287bff5a0ba38b6d39079c851ec89"}
Message was sent while issue was closed.
Description was changed from ========== Run the dEQP GLES 3.1 tests on Windows and Linux NVIDIA OpenGL. BUG=angleproject:1442 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ========== to ========== Run the dEQP GLES 3.1 tests on Windows and Linux NVIDIA OpenGL. BUG=angleproject:1442 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ==========
Message was sent while issue was closed.
Committed patchset #2 (id:20001)
Message was sent while issue was closed.
Description was changed from ========== Run the dEQP GLES 3.1 tests on Windows and Linux NVIDIA OpenGL. BUG=angleproject:1442 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ========== to ========== Run the dEQP GLES 3.1 tests on Windows and Linux NVIDIA OpenGL. BUG=angleproject:1442 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Committed: https://crrev.com/366f3af1194430904d8d3587598f7d02213446bd Cr-Commit-Position: refs/heads/master@{#435650} ==========
Message was sent while issue was closed.
Patchset 2 (id:??) landed as https://crrev.com/366f3af1194430904d8d3587598f7d02213446bd Cr-Commit-Position: refs/heads/master@{#435650} |