New issue
Advanced search Search tips

Issue 864018 link

Starred by 1 user
Status: Fixed
Owner:
tmathmeyer@chromium.org
Closed: Jul 19
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

"PipelineIntegrationTest.PipelineStoppedWhileVideoRestartPending" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Jul 16 
"PipelineIntegrationTest.PipelineStoppedWhileVideoRestartPending" 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=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNySgsSBUZsYWtlIj9QaXBlbGluZUludGVncmF0aW9uVGVzdC5QaXBlbGluZVN0b3BwZWRXaGlsZVZpZGVvUmVzdGFydFBlbmRpbmcM.

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

Comment 1 by xiaoche...@chromium.org, Jul 16 

Flaky crashes also seen on Linux and Mac besides Fuchsia:

https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=media_unittests%20(with%20patch)&tests=PipelineIntegrationTest.PipelineStoppedWhileVideoRestartPending

It seems to be platform-independent. I'm going to disable it on all platforms.

Comment 2 by xiaoche...@chromium.org, Jul 16 

Stack trace on Linux for reference:

Received signal 11 SEGV_MAPERR 000000000000
0   media_unittests                     0x000000010b7b04dc base::debug::StackTrace::StackTrace(unsigned long) + 28
1   media_unittests                     0x000000010b7b0311 base::debug::(anonymous namespace)::StackDumpSignalHandler(int, __siginfo*, void*) + 2401
2   libsystem_platform.dylib            0x00007fff70760f5a _sigtramp + 26
3   ???                                 0x0000000110004080 0x0 + 4563419264
4   media_unittests                     0x000000010b65898e base::internal::Invoker<base::internal::BindState<void (media::PipelineIntegrationTestBase::*)(base::TimeDelta, base::OnceCallback<void ()>), base::internal::UnretainedWrapper<media::PipelineIntegrationTestBase>, base::TimeDelta, base::RepeatingCallback<void ()> >, void ()>::RunOnce(base::internal::BindStateBase*) + 94
5   media_unittests                     0x000000010b6c8471 base::debug::TaskAnnotator::RunTask(char const*, base::PendingTask*) + 321
6   media_unittests                     0x000000010b6f40e9 base::internal::IncomingTaskQueue::RunTask(base::PendingTask*) + 121
7   media_unittests                     0x000000010b6f8537 base::MessageLoop::RunTask(base::PendingTask*) + 599
8   media_unittests                     0x000000010b6f88fa base::MessageLoop::DeferOrRunPendingTask(base::PendingTask) + 186
9   media_unittests                     0x000000010b6f913f base::MessageLoop::DoDelayedWork(base::TimeTicks*) + 1247
10  media_unittests                     0x000000010b6fc41c base::MessagePumpDefault::Run(base::MessagePump::Delegate*) + 108
11  media_unittests                     0x000000010b6f7f14 base::MessageLoop::Run(bool) + 132
12  media_unittests                     0x000000010b731399 base::RunLoop::Run() + 249
13  media_unittests                     0x000000010b73194a base::RunLoop::RunUntilIdle() + 106
14  media_unittests                     0x000000010b64c75b media::PipelineIntegrationTestBase::~PipelineIntegrationTestBase() + 155
15  media_unittests                     0x000000010a98de62 media::PipelineIntegrationTest_PipelineStoppedWhileVideoRestartPending_Test::~PipelineIntegrationTest_PipelineStoppedWhileVideoRestartPending_Test() + 18

Comment 3 by xiaoche...@chromium.org, Jul 16

Components: Internals>Media
Owner: sande...@chromium.org
Status: Assigned (was: Untriaged)
sandersd@: Could you take a look?
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 16 

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

commit a9267df4fc628d1cb267db20b1e091dc02e5020d
Author: Xiaocheng Hu <xiaochengh@chromium.org>
Date: Mon Jul 16 20:10:11 2018

Disable PipelineIntegrationTest.PipelineStoppedWhileVideoRestartPending

Change-Id: I9c3d675b9b694090a99f20d49b963d8dba6d038a
Tbr: sandersd@chromium.org
NoTry: True
Bug:  864018 
Reviewed-on: https://chromium-review.googlesource.com/1138417
Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#575398}
[modify] https://crrev.com/a9267df4fc628d1cb267db20b1e091dc02e5020d/media/test/pipeline_integration_test.cc

Comment 5 by xiaoche...@chromium.org, Jul 16

Labels: -Sheriff-Chromium

Comment 6 by sande...@chromium.org, Jul 16

Owner: tmathmeyer@chromium.org

Comment 7 by tmathmeyer@chromium.org, Jul 18

Status: Started (was: Assigned)
Project Member

Comment 8 by bugdroid1@chromium.org, Jul 19 

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

commit a5e4085f3c0a5ece7041002055d6ca493009517a
Author: Ted Meyer <tmathmeyer@chromium.org>
Date: Thu Jul 19 03:13:10 2018

Reenable test with fix

fuchsia should no longer be racing and taking a page fault on IP.

Bug:  864018 
Change-Id: I07a21cbc80944822ceb3dcc12cfc39f8978d677c
Reviewed-on: https://chromium-review.googlesource.com/1141353
Commit-Queue: Ted Meyer <tmathmeyer@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#576344}
[modify] https://crrev.com/a5e4085f3c0a5ece7041002055d6ca493009517a/media/test/pipeline_integration_test.cc
[modify] https://crrev.com/a5e4085f3c0a5ece7041002055d6ca493009517a/media/test/pipeline_integration_test_base.cc

Comment 9 by tmathmeyer@chromium.org, Jul 19

Status: Fixed (was: Started)
►

Sign in to add a comment