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

Side by Side Diff: milo/common/config/settings.proto

Issue 2784073002: Milo: Fix naming of pubsub settings (Closed)
Patch Set: Created 3 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
« no previous file with comments | « milo/appengine/buildbot/pubsub.go ('k') | milo/common/config/settings.pb.go » ('j') | 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) 2016 The LUCI Authors. All rights reserved. 1 // Copyright (c) 2016 The LUCI Authors. All rights reserved.
2 // Use of this source code is governed under the Apache License, Version 2.0 2 // Use of this source code is governed under the Apache License, Version 2.0
3 // that can be found in the LICENSE file. 3 // that can be found in the LICENSE file.
4 4
5 syntax = "proto3"; 5 syntax = "proto3";
6 6
7 package config; 7 package config;
8 8
9 // Settings represents the format for the global (service) config for Milo. 9 // Settings represents the format for the global (service) config for Milo.
10 message Settings { 10 message Settings {
11 message Buildbot { 11 message Buildbot {
12 // internal_reader is the infra-auth group that is allowed to read internal 12 // internal_reader is the infra-auth group that is allowed to read internal
13 // buildbot data. 13 // buildbot data.
14 string internal_reader = 1; 14 string internal_reader = 1;
15 15
16 // public_topic is the name of the pubsub topic where public builds come in 16 // public_subscription is the name of the pubsub topic where public builds c ome in
17 // from 17 // from
18 string public_topic = 2; 18 string public_subscription = 2;
19 19
20 // internal_topic is the name of the pubsub topic where internal builds 20 // internal_subscription is the name of the pubsub topic where internal buil ds
21 // come in from 21 // come in from
22 string internal_topic = 3; 22 string internal_subscription = 3;
23 } 23 }
24 Buildbot buildbot = 1; 24 Buildbot buildbot = 1;
25 25
26 message Buildbucket { 26 message Buildbucket {
27 // name is the user friendly name of the Buildbucket instance we're pointing to. 27 // name is the user friendly name of the Buildbucket instance we're pointing to.
28 string name = 1; 28 string name = 1;
29 29
30 // host is the hostname of the buildbucket instance we're pointing to (sans schema). 30 // host is the hostname of the buildbucket instance we're pointing to (sans schema).
31 string host = 2; 31 string host = 2;
32 } 32 }
33 Buildbucket buildbucket = 2; 33 Buildbucket buildbucket = 2;
34 34
35 message Swarming { 35 message Swarming {
36 // default_host is the hostname of the swarming host Milo defaults to, if 36 // default_host is the hostname of the swarming host Milo defaults to, if
37 // none is specified. Default host is implicitly an allowed host. 37 // none is specified. Default host is implicitly an allowed host.
38 string default_host = 1; 38 string default_host = 1;
39 39
40 // allowed_hosts is a whitelist of hostnames of swarming instances 40 // allowed_hosts is a whitelist of hostnames of swarming instances
41 // that Milo is allowed to talk to. This is specified here for security 41 // that Milo is allowed to talk to. This is specified here for security
42 // reasons, because Milo will hand out its oauth2 token to a swarming host. 42 // reasons, because Milo will hand out its oauth2 token to a swarming host.
43 repeated string allowed_hosts = 2; 43 repeated string allowed_hosts = 2;
44 } 44 }
45 Swarming swarming = 3; 45 Swarming swarming = 3;
46 } 46 }
OLDNEW
« no previous file with comments | « milo/appengine/buildbot/pubsub.go ('k') | milo/common/config/settings.pb.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698