From 3105c841d483c225de7453294a838ad023b1d8ab Mon Sep 17 00:00:00 2001
From: Daco Harkes <dacoharkes@google.com>
Date: Thu, 1 Feb 2024 11:20:47 +0000
Subject: [PATCH] [tool] find_builders skip flaky builders

Bug: https://github.com/dart-lang/sdk/issues/51170
Change-Id: I5fbc97838b78f8efdab3a16c907976afaac43d4d
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/349660
Auto-Submit: Daco Harkes <dacoharkes@google.com>
Commit-Queue: Daco Harkes <dacoharkes@google.com>
Commit-Queue: William Hesse <whesse@google.com>
Reviewed-by: William Hesse <whesse@google.com>
---
 tools/find_builders.dart | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/tools/find_builders.dart b/tools/find_builders.dart
index 8c4157a7862..67118ed0af1 100755
--- a/tools/find_builders.dart
+++ b/tools/find_builders.dart
@@ -75,7 +75,9 @@ Iterable<String> _filterConfigurations(Set<String> configs) {
 }
 
 Iterable<String> _filterBuilders(Iterable<String> builders) {
-  return builders.where((b) => !_ciOnlyBuilders.contains(b));
+  return builders.where(
+    (b) => !_ciOnlyBuilders.contains(b) && !_denyListedBuilders.contains(b),
+  );
 }
 
 const _ciOnlyBuilders = {
@@ -83,6 +85,13 @@ const _ciOnlyBuilders = {
   'vm-linux-release-arm64',
 };
 
+const _denyListedBuilders = {
+  // TODO(https://dartbug.com/51170): Reenable this.
+  'vm-aot-win-debug-x64c',
+  // TODO(https://dartbug.com/51170): Reenable this.
+  'vm-win-debug-x64c',
+};
+
 Stream<Map<String, dynamic>> _configurationDocuments() async* {
   String? nextPageToken;
   do {