New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 672903 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
OOO until 2019-01-24
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Windows GPU Optional builder does not seem to source json correctly for AMD

Project Member Reported by jmad...@chromium.org, Dec 9 2016

Issue description

After  issue 641080  is fixed, the generator should be correctly placing the angle_deqp_gles3_d3d11_tests in the Win7 Release (AMD) section. However, looking at the try jobs on https://codereview.chromium.org/2566643003/, it was testing this target on the optional bot. This should be in a different json section: Optional Win7 Release (AMD).


 

Comment 1 by kbr@chromium.org, Dec 9 2016

Please try running the generate_buildbot_json.py script locally and see if any diffs show up in the JSON file.

If multiple people CQ changes to the generated JSON files at the same time, they're usually applied incorrectly because the file is so repetitive.

Labels: TE-NeedsTriageHelp

Comment 3 by kbr@chromium.org, Dec 21 2016

Owner: jmad...@chromium.org
Status: Assigned (was: Unconfirmed)
Jamie, please re-confirm this. I don't think there's a bug in the JSON parsing, but rather that a patch was applied incorrectly.

I can confirm this is still happening.

https://luci-milo.appspot.com/buildbot/tryserver.chromium.win/win_optional_gpu_tests_rel/6194

See any build from the optional builders (search the above log for angle_deqp_gles3). You will see they *only* pull in the AMD configs, and don't even test NVIDIA, which we have had on the ANGLE CQ for a long time.

Any advice?

Comment 5 by kbr@chromium.org, Dec 21 2016

Thanks Jamie for pointing this out.

One strangeness in the configuration of win_optional_gpu_tests_rel is that it mirrors the (not actually existing) "Optional Win7 Release (NVIDIA)", but also "Win7 Release (AMD)", so any test that goes on the Win7 Release (AMD) bot will also go on the optional GPU tryserver. This is set up here:

https://cs.chromium.org/chromium/build/scripts/slave/recipe_modules/chromium_tests/trybots.py?l=586

I'm still digging through the JSON file to figure out what's going on for the NVIDIA side of things.

Comment 6 by kbr@chromium.org, Dec 21 2016

Owner: kbr@chromium.org
OK, looking at the report, the main problem is the abovementioned one. Taking this.

Comment 7 by kbr@chromium.org, Dec 21 2016

Note that this will have the side-effect of ceasing to run the WebGL 1.0 conformance tests on Win/AMD, as well as many of the other tests that are only specified for the main waterfall bots and not the "optional" tryservers. However, since this bot was already running the WebGL 2.0 tests on Win/AMD, the loss of coverage should be minimal.

Comment 8 by kbr@chromium.org, Dec 21 2016

Labels: -TE-NeedsTriageHelp
Project Member

Comment 9 by bugdroid1@chromium.org, Dec 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/450af73ed85c75c5cd58bb59302d9a5db18dc996

commit 450af73ed85c75c5cd58bb59302d9a5db18dc996
Author: Kenneth Russell <kbr@chromium.org>
Date: Wed Dec 21 21:57:36 2016

Change win_optional_gpu_tests_rel to mirror optional AMD bot.

It was previously mirroring the waterfall bot "Win7 Release (AMD)".
Make it mirror the virtual bot "Optional Win7 Release (AMD)" as
expected.

This will have the side-effect of making this bot stop running the
WebGL 1.0 conformance tests, but since it's already running the WebGL
2.0 conformance tests, the loss of coverage should be minimal. (Note
that win_chromium_rel_ng does not run tests on Win/AMD because there
isn't enough capacity.)

BUG= 672903 

Change-Id: I59ce9978e195a2526866e920661d02af104e36b0

TBR=dpranke@chromium.org

Change-Id: I59ce9978e195a2526866e920661d02af104e36b0
Reviewed-on: https://chromium-review.googlesource.com/422941
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>

[modify] https://crrev.com/450af73ed85c75c5cd58bb59302d9a5db18dc996/scripts/slave/recipe_modules/chromium_tests/trybots.py

Comment 10 by kbr@chromium.org, Dec 21 2016

Status: Fixed (was: Assigned)
This is fixed and will take effect in the next runs of win_optional_gpu_tests_rel. It looks like the AMD bots behind this are being fixed in Issue 673475 so when they come back online let's look at them and verify.

Sign in to add a comment