New issue
Advanced search Search tips

Issue 683408 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

AudioAndroidInputTest.StartInputStreamCallbacks{,NonDefaultParameters}/0 flaky on KK Tablet

Project Member Reported by jbudorick@chromium.org, Jan 20 2017

Issue description

Cc: tommi@chromium.org
Status: Available (was: Untriaged)
Looks like timeouts.

+cc tommi as closest owner.

Comment 2 by awdf@chromium.org, Apr 6 2017

Cc: braveyao@chromium.org henrika@chromium.org
Still flakily timing out on KitKat tablet tester:

https://uberchromegw.corp.google.com/i/chromium.android/builders/KitKat%20Tablet%20Tester/builds/7207 

I  275.757s Main  Will run 1 tests on 7 devices: 04996ae8, 057ccb85, 05837e65, 0597cb68, 05984fda, 0599f782, 0a587045
I  275.758s TimeoutThread-1-for-run_tests_on_device(04996ae8)  [host]> /b/c/b/KitKat_Tablet_Tester/src/third_party/android_tools/sdk/platform-tools/adb -s 04996ae8 shell '( pm dump org.chromium.native_test | grep dataDir=; echo "PIPESTATUS: ${PIPESTATUS[@]}" );echo %$?'
I  276.660s run_tests_on_device(04996ae8)  flags:
I  276.662s TimeoutThread-1-for-run_tests_on_device(04996ae8)  [host]> /b/c/b/KitKat_Tablet_Tester/src/third_party/android_tools/sdk/platform-tools/adb -s 04996ae8 push /tmp/tmpC6AR2E /data/local/tmp/temp_file-5af7090038154.sh
I  276.669s TimeoutThread-1-for-run_tests_on_device(04996ae8)  Large shell command will be run from file: ( p=org.chromium.native_test;am instrument -w -e "$p".NativeTestInstrumentationTestRunner.ShardNanoTimeout 120000000000 -e "$p".NativeTestInstrumentationTestRunner.NativeTestActivity "$p".NativeUnitTestActivity -e "$p".NativeTestInstrumentationTestRunner.Test AudioAndroidInputTest/AudioAndroidInputTest.StartInputStreamCallbacks/0 -e "$p".NativeTestInstrumentationTestRunner.StdoutFile /storage/emulated/legacy/temp_file-be6c8426ef19c.gtest_out "$p"/"$p".NativeTestInstrumentationTestRunner )>/data/local/tmp/te ...
I  276.669s TimeoutThread-1-for-run_tests_on_device(04996ae8)  [host]> /b/c/b/KitKat_Tablet_Tester/src/third_party/android_tools/sdk/platform-tools/adb -s 04996ae8 shell '( sh /data/local/tmp/temp_file-5af7090038154.sh );echo %$?'
I  336.738s run_tests_on_device(04996ae8)  Still working on StartInstrumentation(04996ae8, org.chromium.native_test/org.chromium.native_test.NativeTestInstrumentationTestRunner, retries=0, extras={'org.chromium.native_test.NativeTestInstrumentationTestRunner.ShardNanoTimeout': 120000000000, 'org.chromium.native_test.NativeTestInstrumentationTestRunner.NativeTestActivity': 'org.chromium.native_test.NativeUnitTestActivity', 'org.chromium.native_test.NativeTestInstrumentationTestRunner.Test': 'AudioAndroidInputTest/AudioAndroidInputTest.StartInputStreamCallbacks/0', 'org.chromium.native_test.NativeTestInstrumentationTestRunner.StdoutFile': '/storage/emulated/legacy/temp_file-be6c8426ef19c.gtest_out'}, timeout=120.0, raw=False)
C  396.715s run_tests_on_device(04996ae8)  Timed out. Dumping threads.
C  396.715s run_tests_on_device(04996ae8)  ********************************************************************************
C  396.715s run_tests_on_device(04996ae8)  Stack dump for thread 'TimeoutThread-1-for-run_tests_on_device(04996ae8)'
C  396.715s run_tests_on_device(04996ae8)  ********************************************************************************
C  396.715s run_tests_on_device(04996ae8)  File: "/usr/lib/python2.7/threading.py", line 783, in __bootstrap
C  396.715s run_tests_on_device(04996ae8)    self.__bootstrap_inner()
C  396.715s run_tests_on_device(04996ae8)  File: "/usr/lib/python2.7/threading.py", line 810, in __bootstrap_inner
C  396.715s run_tests_on_device(04996ae8)    self.run()
C  396.716s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/utils/reraiser_thread.py", line 81, in run
C  396.716s run_tests_on_device(04996ae8)    self._ret = self._func(*self._args, **self._kwargs)
C  396.716s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/utils/timeout_retry.py", line 152, in <lambda>
C  396.716s run_tests_on_device(04996ae8)    child_thread = reraiser_thread.ReraiserThread(lambda: func(*args, **kwargs),
C  396.716s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/decorators.py", line 47, in impl
C  396.716s run_tests_on_device(04996ae8)    return f(*args, **kwargs)
C  396.716s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/device_utils.py", line 1114, in StartInstrumentation
C  396.716s run_tests_on_device(04996ae8)    large_output=True)
C  396.716s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/decorators.py", line 51, in timeout_retry_wrapper
C  396.716s run_tests_on_device(04996ae8)    return impl()
C  396.716s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/decorators.py", line 47, in impl
C  396.716s run_tests_on_device(04996ae8)    return f(*args, **kwargs)
C  396.717s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/device_utils.py", line 960, in RunShellCommand
C  396.717s run_tests_on_device(04996ae8)    output = handle_large_output(cmd, large_output)
C  396.717s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/device_utils.py", line 925, in handle_large_output
C  396.717s run_tests_on_device(04996ae8)    handle_large_command(cmd)
C  396.717s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/device_utils.py", line 917, in handle_large_command
C  396.717s run_tests_on_device(04996ae8)    return handle_check_return('sh %s' % script.name_quoted)
C  396.717s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/device_utils.py", line 902, in handle_check_return
C  396.717s run_tests_on_device(04996ae8)    return run(cmd)
C  396.717s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/device_utils.py", line 898, in run
C  396.717s run_tests_on_device(04996ae8)    return self.adb.Shell(cmd)
C  396.717s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/sdk/adb_wrapper.py", line 485, in Shell
C  396.718s run_tests_on_device(04996ae8)    output = self._RunDeviceAdbCmd(args, timeout, retries, check_error=False)
C  396.718s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/sdk/adb_wrapper.py", line 282, in _RunDeviceAdbCmd
C  396.718s run_tests_on_device(04996ae8)    check_error=check_error)
C  396.718s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/decorators.py", line 51, in timeout_retry_wrapper
C  396.718s run_tests_on_device(04996ae8)    return impl()
C  396.718s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/decorators.py", line 47, in impl
C  396.718s run_tests_on_device(04996ae8)    return f(*args, **kwargs)
C  396.718s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/sdk/adb_wrapper.py", line 249, in _RunAdbCmd
C  396.718s run_tests_on_device(04996ae8)    timeout_retry.CurrentTimeoutThreadGroup().GetRemainingTime())
C  396.718s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/utils/cmd_helper.py", line 315, in GetCmdStatusAndOutputWithTimeout
C  396.719s run_tests_on_device(04996ae8)    for data in _IterProcessStdout(process, timeout=timeout):
C  396.719s run_tests_on_device(04996ae8)  File: "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/utils/cmd_helper.py", line 271, in _IterProcessStdout
C  396.719s run_tests_on_device(04996ae8)    [child_fd], [], [], iter_aware_poll_interval)
C  396.719s run_tests_on_device(04996ae8)  ********************************************************************************
E  396.719s run_tests_on_device(04996ae8)  gtest shard timed out.
Traceback (most recent call last):
  File "/b/c/b/KitKat_Tablet_Tester/src/build/android/pylib/local/device/local_device_gtest_run.py", line 170, in Run
    self._component, extras=extras, raw=False, **kwargs)
  File "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/android/decorators.py", line 57, in timeout_retry_wrapper
    retry_if_func=retry_if_func)
  File "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/utils/timeout_retry.py", line 159, in Run
    error_log_func=error_log_func)
  File "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/utils/reraiser_thread.py", line 186, in JoinAll
    self._JoinAll(watcher, timeout)
  File "/b/c/b/KitKat_Tablet_Tester/src/third_party/catapult/devil/devil/utils/reraiser_thread.py", line 151, in _JoinAll
    (len(alive_threads), len(self._threads)))
CommandTimeoutError: Timed out waiting for 1 of 1 threads.

looks live a revival of https://bugs.chromium.org/p/chromium/issues/detail?id=337867. cc-ing the people who worked on it last time.

Owner: braveyao@chromium.org
Brave, are you able to check this out? Not sure why we see this again now given work in https://bugs.chromium.org/p/chromium/issues/detail?id=337867.
I flashed a N7 with same version as the bots, razor-userdebug 4.4.4 KTU84Z and run the media_unittests dozens of times. So far tests passed all runs.
Same result on a N5 KK device.

According to the bots, how often it'll reproduce? (Sorry I don't quite understand that result dashboards)
Project Member

Comment 5 by bugdroid1@chromium.org, May 3 2017

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

commit da7c2682b226765b94f4470933a134109854b0a0
Author: johnme <johnme@chromium.org>
Date: Wed May 03 15:35:39 2017

[Audio] Mark AudioAndroidInputTest.StartInputStreamCallbacks* flaky

Marks AudioAndroidInputTest.StartInputStreamCallbacks and
AudioAndroidInputTest.StartInputStreamCallbacksNonDefaultParameters
as flaky. They've been failing on chromium.android/KitKat Tablet Tester,
chromium.linux/Android Tests and chromium.linux/Android Tests (dbg) due
to timeouts, sometimes turning the build red.

BUG= 683408 
TBR=braveyao@chromium.org
NOTRY=true
SKIPTREECHECKS=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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

Review-Url: https://codereview.chromium.org/2852323004
Cr-Commit-Position: refs/heads/master@{#468983}

[modify] https://crrev.com/da7c2682b226765b94f4470933a134109854b0a0/media/audio/android/audio_android_unittest.cc

Project Member

Comment 6 by sheriffbot@chromium.org, May 3 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: dalecur...@chromium.org
Labels: -Pri-2 Pri-3
Owner: dalecur...@chromium.org
Status: Assigned (was: Untriaged)
I don't think that any changes will (or can) be done in the existing OpenSL ES based audio layer in Chrome to fix this issue. Dale probably knows more. Perhaps it would be a good idea to to try to re-enable the test if/when a more stable Android audio layer is in use.

Dale, given that you are more active in this area than I am currently, I don't feel confident closing the issue. Reassigning to you for final decision.
Probably worth some look to make sure it's not a bug in our code. If/when we switch to oboe for SLES/AAudio it might help resolve these.
Status: WontFix (was: Assigned)

Sign in to add a comment