Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(546)

Unified Diff: scripts/slave/recipes/chromium.py

Issue 2207263002: Add logcats link in builbot page for android swarming tasks (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Fix mismatch links on task_id Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipes/chromium.py
diff --git a/scripts/slave/recipes/chromium.py b/scripts/slave/recipes/chromium.py
index 63dc555c4de468604051e1d756dd8557c9ce7ea1..5ceb56508356dbc16f9e6c0a0bc553a071b871cb 100644
--- a/scripts/slave/recipes/chromium.py
+++ b/scripts/slave/recipes/chromium.py
@@ -946,8 +946,112 @@ def GenTests(api):
],
'cipd_packages': [
{
- 'location': '{$HOME}/logdog',
- 'cipd_package': 'infra/logdog/linux-386',
+ 'location': 'bin',
+ 'cipd_package': 'logdog/butler/linux-386',
+ 'revision': 'git_revision:deadbeef',
+ },
+ ],
+ },
+ 'override_compile_targets': [
+ 'chrome_public_test_apk'
+ ],
+ 'override_isolate_target': 'chrome_public_test_apk',
+ }
+ ],
+ },
+ })
+ )
+ )
+
+ yield (
+ api.test('dynamic_swarmed_output_links_trigger_test') +
+ api.properties.generic(mastername='chromium.linux',
+ buildername='Android Tests',
+ parent_buildername='Android Builder') +
+ api.properties(swarm_hashes={
+ 'chrome_public_test_apk': 'cccccccccccccccccc',
+ }) +
+ api.override_step_data(
+ 'read test spec (chromium.linux.json)',
+ api.json.output({
+ 'Android Tests': {
+ 'gtest_tests': [
+ {
+ 'test': 'chrome_public_test_apk',
+ 'swarming': {
+ 'can_use_on_swarming_builders': True,
+ 'dimension_sets': [
+ {
+ 'build.id': 'KTU84P',
+ 'product.board': 'hammerhead',
+ },
+ ],
+ 'cipd_packages': [
+ {
+ 'location': 'bin',
+ 'cipd_package': 'logdog/butler/linux-386',
+ 'revision': 'git_revision:deadbeef',
+ },
+ {
+ 'location': 'bin',
+ 'cipd_package': 'some/other/package',
+ 'revision': 'git_revision:deadbeef',
+ },
+ ],
+ 'output_links': [
+ {
+ 'name': 'shard_index:${SHARD_INDEX}logcats',
+ 'link': [
+ 'https://luci-logdog.appspot.com/v/?s',
+ '=android%2Fswarming%2Flogcats%2F',
+ '${TASK_ID}%2F%2B%2Funified_logcats',
+ ],
+ },
+ ],
+ },
+ 'override_compile_targets': [
+ 'chrome_public_test_apk'
+ ],
+ 'override_isolate_target': 'chrome_public_test_apk',
+ }
+ ],
+ },
+ })
+ )
+ )
+
+ yield (
+ api.test('dynamic_swarmed_no_output_trigger_test') +
+ api.properties.generic(mastername='chromium.linux',
+ buildername='Android Tests',
+ parent_buildername='Android Builder') +
+ api.properties(swarm_hashes={
+ 'chrome_public_test_apk': 'cccccccccccccccccc',
+ }) +
+ api.override_step_data(
+ 'read test spec (chromium.linux.json)',
+ api.json.output({
+ 'Android Tests': {
+ 'gtest_tests': [
+ {
+ 'test': 'chrome_public_test_apk',
+ 'swarming': {
+ 'can_use_on_swarming_builders': True,
+ 'dimension_sets': [
+ {
+ 'build.id': 'KTU84P',
+ 'product.board': 'hammerhead',
+ },
+ ],
+ 'cipd_packages': [
+ {
+ 'location': 'bin',
+ 'cipd_package': 'some/other/package',
+ 'revision': 'git_revision:deadbeef',
+ },
+ {
+ 'location': 'bin',
+ 'cipd_package': 'another/package',
'revision': 'git_revision:deadbeef',
},
],

Powered by Google App Engine
This is Rietveld 408576698