New issue
Advanced search Search tips

Issue 880078 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug

Blocked on:
issue 851213

Blocking:
issue 609629



Sign in to add a comment

"gpu_tests.context_lost_integration_test.ContextLostIntegrationTest.ContextLost_WorkerRAFAfterGPUCrash_OOPD" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Sep 3

Issue description

"gpu_tests.context_lost_integration_test.ContextLostIntegrationTest.ContextLost_WorkerRAFAfterGPUCrash_OOPD" is flaky.

This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label.

We have detected 3 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNydQsSBUZsYWtlImpncHVfdGVzdHMuY29udGV4dF9sb3N0X2ludGVncmF0aW9uX3Rlc3QuQ29udGV4dExvc3RJbnRlZ3JhdGlvblRlc3QuQ29udGV4dExvc3RfV29ya2VyUkFGQWZ0ZXJHUFVDcmFzaF9PT1BEDA.

Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs
 
Detected 3 new flakes for test/step "gpu_tests.context_lost_integration_test.ContextLostIntegrationTest.ContextLost_WorkerRAFAfterGPUCrash_OOPD". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNydQsSBUZsYWtlImpncHVfdGVzdHMuY29udGV4dF9sb3N0X2ludGVncmF0aW9uX3Rlc3QuQ29udGV4dExvc3RJbnRlZ3JhdGlvblRlc3QuQ29udGV4dExvc3RfV29ya2VyUkFGQWZ0ZXJHUFVDcmFzaF9PT1BEDA. This message was posted automatically by the chromium-try-flakes app.
Blocking: 851213
Blocking: -851213
Blockedon: 851213
Blocking: 609629
Components: -Tests>Flaky Internals>GPU>Testing Internals>Services>Viz Internals>GPU>Internals
Labels: -Pri-1 OS-Android Pri-3
https://chromium-review.googlesource.com/1204598 has landed which should suppress these failures, as well as those of gpu_tests.context_lost_integration_test.ContextLostIntegrationTest.ContextLost_WorkerRAFAfterGPUCrash:

https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNycAsSBUZsYWtlImVncHVfdGVzdHMuY29udGV4dF9sb3N0X2ludGVncmF0aW9uX3Rlc3QuQ29udGV4dExvc3RJbnRlZ3JhdGlvblRlc3QuQ29udGV4dExvc3RfV29ya2VyUkFGQWZ0ZXJHUFVDcmFzaAw

It looks like Chrome on Android doesn't handle the GPU process crashing very well because several of the context_loss tests are marked as skipped (Issue 609629). We should try to improve the situation here though this isn't high priority.

Status: Available (was: Untriaged)
Project Member

Comment 7 by bugdroid1@chromium.org, Sep 5

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d9878f42b7788b02a711e88a9df5e14e67ad1839

commit d9878f42b7788b02a711e88a9df5e14e67ad1839
Author: Kenneth Russell <kbr@chromium.org>
Date: Tue Sep 04 21:44:43 2018

Suppress WorkerRAFAfterGPUCrash failures on Android.

Mark the following two tests as failing:
  ContextLost_WorkerRAFAfterGPUCrash
  ContextLost_WorkerRAFAfterGPUCrash_OOPD

Tbr: fserb@chromium.org
No-Try: True
Bug:  851213 , 880078
Change-Id: Ia3baefd5b7bfc1f565626925de28befd3e3f4163
Reviewed-on: https://chromium-review.googlesource.com/1204598
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#588654}
[modify] https://crrev.com/d9878f42b7788b02a711e88a9df5e14e67ad1839/content/test/gpu/gpu_tests/context_lost_expectations.py

Status: Fixed (was: Available)
THe original flake doesn't seem to still be happening. \o/
Labels: -Sheriff-Chromium
Status: Available (was: Fixed)
Right, but the failures are only suppressed, not actually fixed.

Sign in to add a comment