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

Side by Side Diff: tools/gardening/lib/src/buildbot_data.dart

Issue 3000493002: Improve output of current-summary gardening tool. (Closed)
Patch Set: Created 3 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 unified diff | Download patch
« no previous file with comments | « tools/gardening/lib/src/bot.dart ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2017, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2017, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 import 'dart:async'; 5 import 'dart:async';
6 import 'buildbot_structures.dart'; 6 import 'buildbot_structures.dart';
7 import 'logdog.dart' as logdog; 7 import 'logdog.dart' as logdog;
8 import 'util.dart'; 8 import 'util.dart';
9 9
10 /// Data describing the steps of the buildbots. 10 /// Data describing the steps of the buildbots.
(...skipping 789 matching lines...) Expand 10 before | Expand all | Expand 10 after
800 /// Returns the [BuildUri] corresponding to the build steps for shards in this 800 /// Returns the [BuildUri] corresponding to the build steps for shards in this
801 /// group. 801 /// group.
802 List<BuildUri> createUris(int buildNumber, {bool includeInactive: false}) { 802 List<BuildUri> createUris(int buildNumber, {bool includeInactive: false}) {
803 List<BuildUri> uriList = <BuildUri>[]; 803 List<BuildUri> uriList = <BuildUri>[];
804 for (BuildSubgroup subgroup in subgroups) { 804 for (BuildSubgroup subgroup in subgroups) {
805 if (!subgroup.isActive && !includeInactive) continue; 805 if (!subgroup.isActive && !includeInactive) continue;
806 uriList.addAll(subgroup.createUris(buildNumber)); 806 uriList.addAll(subgroup.createUris(buildNumber));
807 } 807 }
808 return uriList; 808 return uriList;
809 } 809 }
810
811 String toString() => groupName;
810 } 812 }
811 813
812 /// A group of buildbot subgroups, often shards, that share their test steps, 814 /// A group of buildbot subgroups, often shards, that share their test steps,
813 /// for instance all `dart2js-win7-ie10chrome-*-4-be` shards. 815 /// for instance all `dart2js-win7-ie10chrome-*-4-be` shards.
814 class BuildSubgroup { 816 class BuildSubgroup {
815 /// The names of the shards in this, for instance 817 /// The names of the shards in this, for instance
816 /// `dart2js-win7-ie10chrome-1-4-be`, `dart2js-win7-ie10chrome-2-4-be`, etc. 818 /// `dart2js-win7-ie10chrome-1-4-be`, `dart2js-win7-ie10chrome-2-4-be`, etc.
817 final List<String> shardNames; 819 final List<String> shardNames;
818 820
819 /// The names of the test steps for the shards in this subgroup, for instance 821 /// The names of the test steps for the shards in this subgroup, for instance
(...skipping 130 matching lines...) Expand 10 before | Expand all | Expand 10 after
950 stepNames.add(step); 952 stepNames.add(step);
951 } 953 }
952 } 954 }
953 if (stepNames.isNotEmpty) { 955 if (stepNames.isNotEmpty) {
954 subgroups[subgroup] = stepNames; 956 subgroups[subgroup] = stepNames;
955 } 957 }
956 } 958 }
957 } 959 }
958 return subgroups; 960 return subgroups;
959 } 961 }
OLDNEW
« no previous file with comments | « tools/gardening/lib/src/bot.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698