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

Side by Side Diff: go/src/infra/monitoring/messages/gatekeeper.go

Issue 1874053002: [alerts-dispatcher] update to build again :) also fixes a concurrent map access (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: make tests compile. still don't pass though Created 4 years, 8 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // These structs are used for parsing gatekeeper.json files. 5 // These structs are used for parsing gatekeeper.json files.
6 6
7 package messages 7 package messages
8 8
9 // MasterConfig represents filtering configurtaion for alerts 9 // MasterConfig represents filtering configurtaion for alerts
10 // generated about a buildbot master. 10 // generated about a buildbot master.
11 type MasterConfig struct { 11 type MasterConfig struct {
12 Categories []string `json:"categories"`
13 TreeNotify []string `json:"tree_notify"` 12 TreeNotify []string `json:"tree_notify"`
14 SheriffClasses []string `json:"sheriff_classes"` 13 SheriffClasses []string `json:"sheriff_classes"`
15 Builders map[string]BuilderConfig `json:"builders"` 14 Builders map[string]BuilderConfig `json:"builders"`
16 ExcludedBuilders []string `json:"excluded_builders"` 15 ExcludedBuilders []string `json:"excluded_builders"`
17 ExcludedSteps []string `json:"excluded_steps"` 16 ExcludedSteps []string `json:"excluded_steps"`
17 Categories []string `json:"categories"`
martiniss 2016/04/11 21:33:34 Move this back up, so no diff?
seanmccullough 2016/04/11 23:11:08 Done.
18 } 18 }
19 19
20 // BuilderConfig represents filtering configuration for alerts 20 // BuilderConfig represents filtering configuration for alerts
21 // generated about a buildbot builder. 21 // generated about a buildbot builder.
22 type BuilderConfig struct { 22 type BuilderConfig struct {
23 ExcludedSteps []string `json:"excluded_steps"` 23 ExcludedSteps []string `json:"excluded_steps"`
24 ForgivingSteps []string `json:"forgiving_steps"` 24 ForgivingSteps []string `json:"forgiving_steps"`
25 ForgivingOptional []string `json:"forgiving_optional"` 25 ForgivingOptional []string `json:"forgiving_optional"`
26 SheriffClasses []string `json:"sheriff_classes"` 26 SheriffClasses []string `json:"sheriff_classes"`
27 ClosingSteps []string `json:"closing_steps"` 27 ClosingSteps []string `json:"closing_steps"`
28 ClosingOptional []string `json:"closing_optional"` 28 ClosingOptional []string `json:"closing_optional"`
29 } 29 }
30 30
31 type GatekeeperConfig struct {
32 Categories map[string]BuilderConfig `json:"categories"`
33 Masters map[string][]MasterConfig `json:"masters"`
34 }
35
31 // TreeMasterConfig is a named group of masters. e.g. chromium, or blink. 36 // TreeMasterConfig is a named group of masters. e.g. chromium, or blink.
32 type TreeMasterConfig struct { 37 type TreeMasterConfig struct {
33 BuildDB string `json:"build-db"` 38 BuildDB string `json:"build-db"`
34 Masters []string `json:"masters"` 39 Masters []string `json:"masters"`
35 } 40 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698