BrowserOnly/MemlogBrowserTest.EndToEnd/0 fails on chromeos try builder in suite mus_browser_tests |
|||||||
Issue descriptionI'm not sure what's different with mus_browser_tests, but it's been periodically failing on the try servers. I don't see failures on the main waterfall, nor do I know when it started failing. Here's the stack from https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Ftryserver.chromium.linux%2Flinux_chromium_chromeos_rel_ng%2F509809%2F%2B%2Frecipes%2Fsteps%2Fmus_browser_tests__with_patch_%2F0%2Flogs%2FBrowserOnly__x2f_MemlogBrowserTest.EndToEnd__x2f_0%2F0 #0 0x0000036f29ac base::debug::StackTrace::StackTrace() #1 0x000003d00c28 content::(anonymous namespace)::DumpStackTraceSignalHandler() #2 0x7fb8c0ffdcb0 <unknown> #3 0x7fb8c43cc95b __libc_send #4 0x000001ff23ab mojo::edk::PlatformChannelWrite() #5 0x0000055c2512 profiling::MemlogSenderPipe::Send() #6 0x0000055c2ba1 profiling::(anonymous namespace)::DoSend() #7 0x0000055c2cb0 profiling::(anonymous namespace)::HookAlloc() #8 0x000003795664 operator new() #9 0x000004be38b9 IPC::Message::EnsureMessageAttachmentSet() #10 0x000004bf9161 mojo::internal::UnmappedNativeStructSerializerImpl::SerializeMessageContents() #11 0x000001f3a10d mojo::internal::Serializer<>::Serialize() #12 0x000001f399a9 mojo::internal::ArraySerializer<>::SerializeElements() #13 0x000001f3984f mojo::internal::Serializer<>::Serialize() #14 0x000001f36dad mojo::internal::Serializer<>::Serialize() #15 0x000001f34a3a viz::mojom::CompositorFrameSinkProxy::SubmitCompositorFrame() #16 0x000001fce53d viz::ClientLayerTreeFrameSink::SubmitCompositorFrame() #17 0x000004cb2e36 cc::LayerTreeHostImpl::DrawLayers() #18 0x000004d497c8 cc::SingleThreadProxy::DoComposite() #19 0x000004d4a480 cc::SingleThreadProxy::ScheduledActionDrawIfPossible() #20 0x000004d59834 cc::Scheduler::DrawIfPossible() #21 0x000004d56a1b cc::Scheduler::ProcessScheduledActions() #22 0x000004d566e0 cc::Scheduler::OnBeginImplFrameDeadline() #23 0x0000006b2677 _ZN4base8internal7InvokerINS0_9BindStateIMN12_GLOBAL__N_116SimpleHttpServer10ConnectionEFvvEJNS_7WeakPtrIS5_EEEEEFvvEE7RunOnceEPNS0_13BindStateBaseE #24 0x0000006b2677 _ZN4base8internal7InvokerINS0_9BindStateIMN12_GLOBAL__N_116SimpleHttpServer10ConnectionEFvvEJNS_7WeakPtrIS5_EEEEEFvvEE7RunOnceEPNS0_13BindStateBaseE #25 0x0000036f3148 base::debug::TaskAnnotator::RunTask() #26 0x0000037a70dc base::internal::IncomingTaskQueue::RunTask() #27 0x000003713453 base::MessageLoop::RunTask() #28 0x000003713a5e base::MessageLoop::DoWork() #29 0x000003716279 base::MessagePumpLibevent::Run() #30 0x000003712f62 base::MessageLoop::Run() #31 0x00000373cc4a base::RunLoop::Run() #32 0x00000152ee6f (anonymous namespace)::MemlogBrowserTest_EndToEnd_Test::RunTestOnMainThread() #33 0x000003d009a3 content::BrowserTestBase::ProxyRunTestOnMainThreadLoop() #34 0x00000382529c ChromeBrowserMainParts::PreMainMessageLoopRunImpl() #35 0x00000382412d ChromeBrowserMainParts::PreMainMessageLoopRun() #36 0x0000016a4b82 chromeos::ChromeBrowserMainPartsChromeos::PreMainMessageLoopRun() #37 0x0000022172a1 content::BrowserMainLoop::PreMainMessageLoopRun() #38 0x000002606d57 content::StartupTaskRunner::RunAllTasksNow() #39 0x00000221562f content::BrowserMainLoop::CreateStartupTasks() #40 0x000002219bc0 content::BrowserMainRunnerImpl::Initialize() #41 0x000002212c37 content::BrowserMain() #42 0x00000369a06c content::ContentMainRunnerImpl::Run() #43 0x00000556854e service_manager::Main() #44 0x000003698a44 content::ContentMain() #45 0x000003d00663 content::BrowserTestBase::SetUp() #46 0x0000037e3e76 InProcessBrowserTest::SetUp() #47 0x0000019eaa81 testing::Test::Run() #48 0x0000019eb570 testing::TestInfo::Run() #49 0x0000019eba57 testing::TestCase::Run() #50 0x0000019f1fe7 testing::internal::UnitTestImpl::RunAllTests() #51 0x0000019f1c67 testing::UnitTest::Run() #52 0x0000037f92f2 base::TestSuite::Run() #53 0x0000036e5ac9 ChromeTestSuiteRunner::RunTestSuite() #54 0x000003d322bf content::LaunchTests() #55 0x0000036e5e91 LaunchChromeTests() #56 0x0000036e36ed RunMashBrowserTests() #57 0x0000036e3397 main #58 0x7fb8c0fe8f45 __libc_start_main I will disable the test on mus_browser_tests in the short term. https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_chromeos_rel_ng/builds/509787 is the first failure I see in the 200 runs on the trybot.
,
Sep 14 2017
While this bots build target_os=chromeos, they run on desktop linux boxs. I don't think they use a special version of libc, but I could be wrong. Dirk could likely answer if the chromeos bots on the chrome waterfall are configured in a different way than normal linux bots.
,
Sep 14 2017
I would not expect there to be a difference between the desktop linux and desktop cros builds, but thomasanderson@ can probably say for sure.
,
Sep 14 2017
The builds are definitely different, but the environments are the same -- they should be using the same libc version.
,
Sep 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f21d5fac1be532f9319a11217ba212883a8937eb commit f21d5fac1be532f9319a11217ba212883a8937eb Author: Scott Violet <sky@chromium.org> Date: Thu Sep 14 23:46:46 2017 Disables BrowserOnly/MemlogBrowserTest.EndToEnd/0 It's flaky on the chromeos try bots in the suite mus_browser_tests. BUG= 765414 TEST=test only change TBR=brettw@chromium.org Change-Id: I56fe3b09fff6100b144b9607219e2f8344e50aa9 Reviewed-on: https://chromium-review.googlesource.com/667935 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#502103} [modify] https://crrev.com/f21d5fac1be532f9319a11217ba212883a8937eb/testing/buildbot/filters/mus.browser_tests.filter
,
Oct 3 2017
When we added new tests in this suite - it still caused mus_browser_Test failures...we really need to look into this: https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.chromiumos%2FLinux_ChromiumOS_Tests__1_%2F45306%2F%2B%2Frecipes%2Fsteps%2Fmus_browser_tests%2F0%2Flogs%2FAllProcesses__x2f_MemlogBrowserTest.EndToEndTracing__x2f_0%2F0
,
Oct 4 2017
The memlog parser thread was using LOG(), which invokes localtime, which is overridden in the Linux sandbox to synchronously communicate with the browser process, which was waiting for the memlog parser thread to clear the pipe. Deadlock. https://cs.chromium.org/chromium/src/content/zygote/zygote_main_linux.cc?q=time_override&sq=package:chromium&l=259
,
Oct 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0ba08cb3faee37bbeed8249f6f29d75145cc3c34 commit 0ba08cb3faee37bbeed8249f6f29d75145cc3c34 Author: Erik Chen <erikchen@chromium.org> Date: Tue Oct 10 17:59:34 2017 Create a new sandbox type for the profiling service. The new "profiling" sandbox type is a variant of the utility sandbox type. The only difference is that on Linux, localtime will use UTC instead of brokering a synchronous IPC to the browser process. This is needed to prevent deadlocks between the browser process and the profiling service, when profiling is enabled. For more details, see Issue 772503 . Bug: 772503 , 765414 Change-Id: Id16171ea26155aed254567c61dbec05f635305a8 Reviewed-on: https://chromium-review.googlesource.com/707918 Reviewed-by: Robert Sesek <rsesek@chromium.org> Reviewed-by: Brett Wilson <brettw@chromium.org> Commit-Queue: Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#507713} [modify] https://crrev.com/0ba08cb3faee37bbeed8249f6f29d75145cc3c34/chrome/profiling/profiling_manifest.json [modify] https://crrev.com/0ba08cb3faee37bbeed8249f6f29d75145cc3c34/content/app/DEPS [modify] https://crrev.com/0ba08cb3faee37bbeed8249f6f29d75145cc3c34/content/app/content_main_runner.cc [modify] https://crrev.com/0ba08cb3faee37bbeed8249f6f29d75145cc3c34/content/browser/utility_process_host_impl.cc [modify] https://crrev.com/0ba08cb3faee37bbeed8249f6f29d75145cc3c34/content/common/sandbox_linux/sandbox_seccomp_bpf_linux.cc [modify] https://crrev.com/0ba08cb3faee37bbeed8249f6f29d75145cc3c34/content/common/sandbox_mac.mm [modify] https://crrev.com/0ba08cb3faee37bbeed8249f6f29d75145cc3c34/content/zygote/zygote_main.h [modify] https://crrev.com/0ba08cb3faee37bbeed8249f6f29d75145cc3c34/content/zygote/zygote_main_linux.cc [modify] https://crrev.com/0ba08cb3faee37bbeed8249f6f29d75145cc3c34/services/service_manager/sandbox/sandbox_type.cc [modify] https://crrev.com/0ba08cb3faee37bbeed8249f6f29d75145cc3c34/services/service_manager/sandbox/sandbox_type.h [modify] https://crrev.com/0ba08cb3faee37bbeed8249f6f29d75145cc3c34/services/service_manager/sandbox/switches.cc [modify] https://crrev.com/0ba08cb3faee37bbeed8249f6f29d75145cc3c34/services/service_manager/sandbox/switches.h [modify] https://crrev.com/0ba08cb3faee37bbeed8249f6f29d75145cc3c34/testing/buildbot/filters/mus.browser_tests.filter
,
Oct 31 2017
,
Oct 31 2017
Issue 761342 has been merged into this issue.
,
Jan 22 2018
,
Jan 23 2018
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by erikc...@chromium.org
, Sep 14 2017