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

Unified 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 side-by-side diff with in-line comments
Download patch
Index: go/src/infra/monitoring/messages/gatekeeper.go
diff --git a/go/src/infra/monitoring/messages/gatekeeper.go b/go/src/infra/monitoring/messages/gatekeeper.go
index 8ba9eca0fdd1a6a9e66fc65b9c642ee6c848b383..23f85c877f50b37f5346c05195044aca439d544f 100644
--- a/go/src/infra/monitoring/messages/gatekeeper.go
+++ b/go/src/infra/monitoring/messages/gatekeeper.go
@@ -9,12 +9,12 @@ package messages
// MasterConfig represents filtering configurtaion for alerts
// generated about a buildbot master.
type MasterConfig struct {
- Categories []string `json:"categories"`
TreeNotify []string `json:"tree_notify"`
SheriffClasses []string `json:"sheriff_classes"`
Builders map[string]BuilderConfig `json:"builders"`
ExcludedBuilders []string `json:"excluded_builders"`
ExcludedSteps []string `json:"excluded_steps"`
+ 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.
}
// BuilderConfig represents filtering configuration for alerts
@@ -28,6 +28,11 @@ type BuilderConfig struct {
ClosingOptional []string `json:"closing_optional"`
}
+type GatekeeperConfig struct {
+ Categories map[string]BuilderConfig `json:"categories"`
+ Masters map[string][]MasterConfig `json:"masters"`
+}
+
// TreeMasterConfig is a named group of masters. e.g. chromium, or blink.
type TreeMasterConfig struct {
BuildDB string `json:"build-db"`

Powered by Google App Engine
This is Rietveld 408576698