Revert "cbuildbot_config: temporarily mark 'auron-paladin' not important"
This reverts commit 01f6ee56a1b55576a4d047fcfbdb58d1f97f4273.
CL:427325 marked auron-paladin not important due to chromium:678643,
which is now fixed. Builds on auron-paladin have been passing in the
last 50 runs, so this CL marks auron-paladin as important again by
reverting CL:427325.
BUG=chromium:681267
TEST=`./cbuildbot/chromeos_config_unittest`
Change-Id: I0619b39d655cbe33f58276053ff76a0fdd2f5866
Reviewed-on: https://chromium-review.googlesource.com/430824
Commit-Ready: Ben Chan <benchan@chromium.org>
Tested-by: Ben Chan <benchan@chromium.org>
Reviewed-by: Dan Shi <dshi@chromium.org>
diff --git a/cbuildbot/chromeos_config.py b/cbuildbot/chromeos_config.py
index f9339f7..264f419 100644
--- a/cbuildbot/chromeos_config.py
+++ b/cbuildbot/chromeos_config.py
@@ -719,9 +719,6 @@
'lakitu_next-paladin',
'poppy-paladin',
- # Flaky Paladins (see TODO below in _paladin_important_boards)
- 'auron-paladin',
-
# Firmware Builders.
'link-depthcharge-full-firmware',
@@ -2093,12 +2090,7 @@
_paladin_important_boards = frozenset([
'amd64-generic',
'arm-generic',
- # TODO(benchan): update-engine unit tests has been hanging on
- # 'auron-paradin' more frequently (crbug.com/678643) than other builders.
- # Temporarily mark 'auron-paladin' not important to reduce CQ failures as
- # other builders like 'samus-paladin' provide similar coverage. Mark
- # 'auron-paladin' important once the issue is resolved.
- # 'auron',
+ 'auron',
'beaglebone',
'butterfly',
'cyan',
diff --git a/cbuildbot/config_dump.json b/cbuildbot/config_dump.json
index e895126..876deda 100644
--- a/cbuildbot/config_dump.json
+++ b/cbuildbot/config_dump.json
@@ -2604,7 +2604,6 @@
"auron"
],
"description": "Commit Queue (internal)",
- "important": false,
"internal": true,
"manifest": "official.xml",
"manifest_repo_url": "https://chrome-internal-review.googlesource.com/chromeos/manifest-internal",
