Fix some minor issues with the android bots.

* Use the same logdog_butler as Chromium instead of redefining one.
* Use luci-auth to prevent "local auth - HTTP 400" errors.

Change-Id: I2a0d1393f9f0e1e41b2bcc9a9fec2c50c19675f3
Bug: None
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/283520
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#38637}
This commit is contained in:
Jeremy Leconte
2022-11-16 08:26:53 +01:00
committed by WebRTC LUCI CQ
parent d2811761e3
commit e085366aca
7 changed files with 331 additions and 1391 deletions

File diff suppressed because it is too large Load Diff

View File

@ -145,18 +145,6 @@
}
}
},
'logdog-butler': {
'swarming': {
'cipd_packages': [{
'cipd_package':
'infra/tools/luci/logdog/butler/${platform}',
'location':
'bin',
'revision':
'git_revision:ff387eadf445b24c935f1cf7d6ddd279f8a6b04c'
}]
}
},
'mac-m1-cpu': {
'swarming': {
'dimensions': {

View File

@ -118,17 +118,6 @@
},
}
},
'logdog-butler': {
'swarming': {
'cipd_packages': [
{
"cipd_package": 'infra/tools/luci/logdog/butler/${platform}',
'location': 'bin',
'revision': 'git_revision:ff387eadf445b24c935f1cf7d6ddd279f8a6b04c',
},
],
},
},
'mac-m1-cpu': {
'swarming': {
'dimensions': {

File diff suppressed because it is too large Load Diff

View File

@ -13,8 +13,8 @@
'machines': {
'Android32 (M Nexus5X)': {
'mixins': [
'walleye', 'android-devices', 'logdog-butler',
'has_native_resultdb_integration'
'walleye', 'android-devices', 'has_native_resultdb_integration',
'chromium-tester-service-account'
],
'test_suites': {
'gtest_tests': 'android_tests',
@ -23,8 +23,8 @@
},
'Android32 (M Nexus5X)(dbg)': {
'mixins': [
'walleye', 'android-devices', 'logdog-butler',
'has_native_resultdb_integration'
'walleye', 'android-devices', 'has_native_resultdb_integration',
'chromium-tester-service-account'
],
'test_suites': {
'gtest_tests': 'android_tests',
@ -33,8 +33,8 @@
},
'Android32 (more configs)': {
'mixins': [
'walleye', 'android-devices', 'logdog-butler',
'has_native_resultdb_integration'
'walleye', 'android-devices', 'has_native_resultdb_integration',
'chromium-tester-service-account'
],
'test_suites': {
'gtest_tests': 'more_configs_tests',
@ -45,8 +45,8 @@
'Android32 Builder x86 (dbg)': {},
'Android64 (M Nexus5X)': {
'mixins': [
'walleye', 'android-devices', 'logdog-butler',
'has_native_resultdb_integration'
'walleye', 'android-devices', 'has_native_resultdb_integration',
'chromium-tester-service-account'
],
'test_suites': {
'gtest_tests': 'android_tests',
@ -55,8 +55,8 @@
},
'Android64 (M Nexus5X)(dbg)': {
'mixins': [
'walleye', 'android-devices', 'logdog-butler',
'has_native_resultdb_integration'
'walleye', 'android-devices', 'has_native_resultdb_integration',
'chromium-tester-service-account'
],
'test_suites': {
'gtest_tests': 'android_tests',
@ -402,8 +402,8 @@
'machines': {
'android_arm64_dbg': {
'mixins': [
'walleye', 'android-devices', 'logdog-butler',
'has_native_resultdb_integration'
'walleye', 'android-devices', 'has_native_resultdb_integration',
'chromium-tester-service-account'
],
'test_suites': {
'gtest_tests': 'android_tests_tryserver',
@ -412,8 +412,8 @@
},
'android_arm64_rel': {
'mixins': [
'walleye', 'android-devices', 'logdog-butler',
'has_native_resultdb_integration'
'walleye', 'android-devices', 'has_native_resultdb_integration',
'chromium-tester-service-account'
],
'test_suites': {
'gtest_tests': 'android_tests_tryserver',
@ -422,8 +422,8 @@
},
'android_arm_dbg': {
'mixins': [
'walleye', 'android-devices', 'logdog-butler',
'has_native_resultdb_integration'
'walleye', 'android-devices', 'has_native_resultdb_integration',
'chromium-tester-service-account'
],
'test_suites': {
'gtest_tests': 'android_tests_tryserver',
@ -432,8 +432,8 @@
},
'android_arm_more_configs': {
'mixins': [
'walleye', 'android-devices', 'logdog-butler',
'has_native_resultdb_integration'
'walleye', 'android-devices', 'has_native_resultdb_integration',
'chromium-tester-service-account'
],
'test_suites': {
'gtest_tests': 'more_configs_tests',
@ -441,8 +441,8 @@
},
'android_arm_rel': {
'mixins': [
'walleye', 'android-devices', 'logdog-butler',
'has_native_resultdb_integration'
'walleye', 'android-devices', 'has_native_resultdb_integration',
'chromium-tester-service-account'
],
'test_suites': {
'gtest_tests': 'android_tests_tryserver',

View File

@ -81,10 +81,11 @@ class WebRTCMetaBuildWrapper(mb.MetaBuildWrapper):
]
elif is_android:
cmdline += [
vpython_exe, '../../build/android/test_wrapper/logdog_wrapper.py',
'--target', target, '--logdog-bin-cmd', '../../bin/logdog_butler',
'--logcat-output-file', '${ISOLATED_OUTDIR}/logcats',
'--store-tombstones'
'luci-auth', 'context', '--', vpython_exe,
'../../build/android/test_wrapper/logdog_wrapper.py', '--target',
target, '--logdog-bin-cmd',
'../../.task_template_packages/logdog_butler', '--logcat-output-file',
'${ISOLATED_OUTDIR}/logcats', '--store-tombstones'
]
elif is_ios or is_fuchsia:
cmdline += [

View File

@ -324,12 +324,15 @@ class UnitTest(unittest.TestCase):
files,
['../../.vpython3', '../../testing/test_env.py', 'foo_unittests'])
self.assertEqual(command, [
'luci-auth',
'context',
'--',
'vpython3',
'../../build/android/test_wrapper/logdog_wrapper.py',
'--target',
'foo_unittests',
'--logdog-bin-cmd',
'../../bin/logdog_butler',
'../../.task_template_packages/logdog_butler',
'--logcat-output-file',
'${ISOLATED_OUTDIR}/logcats',
'--store-tombstones',
@ -363,12 +366,15 @@ class UnitTest(unittest.TestCase):
files,
['../../.vpython3', '../../testing/test_env.py', 'foo_unittests'])
self.assertEqual(command, [
'luci-auth',
'context',
'--',
'vpython3',
'../../build/android/test_wrapper/logdog_wrapper.py',
'--target',
'foo_unittests',
'--logdog-bin-cmd',
'../../bin/logdog_butler',
'../../.task_template_packages/logdog_butler',
'--logcat-output-file',
'${ISOLATED_OUTDIR}/logcats',
'--store-tombstones',