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

Side by Side Diff: dm/api/distributor/swarming/v1/params.proto

Issue 2267143002: Add additional validation to swarming v1 distributor. (Closed) Base URL: https://chromium.googlesource.com/external/github.com/luci/luci-go@dump_all_stacks
Patch Set: rebase Created 4 years, 3 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 2016 The LUCI Authors. All rights reserved. 1 // Copyright 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 import "google/protobuf/duration.proto"; 7 import "google/protobuf/duration.proto";
8 8
9 import "github.com/luci/luci-go/dm/api/distributor/swarming/v1/isolate_ref.proto "; 9 import "github.com/luci/luci-go/dm/api/distributor/swarming/v1/isolate_ref.proto ";
10 10
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
59 repeated swarmingV1.IsolatedRef isolated = 1; 59 repeated swarmingV1.IsolatedRef isolated = 1;
60 60
61 message CipdPackage { 61 message CipdPackage {
62 string name = 1; 62 string name = 1;
63 string path = 2; 63 string path = 2;
64 string version = 3; 64 string version = 3;
65 } 65 }
66 // 0 or more cipd packages for the job. 66 // 0 or more cipd packages for the job.
67 repeated CipdPackage packages = 2; 67 repeated CipdPackage packages = 2;
68 68
69 // Optional server to use for cipd packages. If you know you're using 69 // Server to use for cipd packages. Required (though it may be specified
70 // swarming's default server, you can omit this. 70 // by the quest's designated distributor config). This must be
71 // scheme://<hostname>, e.g. "https://example.appspot.com". The only
72 // supported schemes are http and https.
71 string cipd_server = 3; 73 string cipd_server = 3;
72 } 74 }
73 Inputs inputs = 1; 75 Inputs inputs = 1;
74 76
75 // This is the "argv" to run with this job. This includes substitution 77 // This is the "argv" to run with this job. This includes substitution
76 // paramters defined by swarming's run_isolated.py script: 78 // paramters defined by swarming's run_isolated.py script:
77 // https://github.com/luci/luci-py/blob/master/client/run_isolated.py 79 // https://github.com/luci/luci-py/blob/master/client/run_isolated.py
78 // 80 //
79 // Additionally, DM provides the following substitutions: 81 // Additionally, DM provides the following substitutions:
80 // ${DM.PREVIOUS.EXECUTION.STATE:PATH} - the path to a JSONPB encoding of 82 // ${DM.PREVIOUS.EXECUTION.STATE:PATH} - the path to a JSONPB encoding of
81 // the swarming_v1.Result from the previous Execution of this Attempt. 83 // the swarming_v1.Result from the previous Execution of this Attempt.
82 // ${DM.QUEST.DATA.DESC:PATH} - the path to the 84 // ${DM.QUEST.DATA.DESC:PATH} - the path to the
83 // dm.Quest.Data.Desc JSONPB for the quest that this execution is part 85 // dm.Quest.Data.Desc JSONPB for the quest that this execution is part
84 // of. 86 // of.
85 // ${DM.EXECUTION.AUTH:PATH} - the path to the JSONPB of a 87 // ${DM.EXECUTION.AUTH:PATH} - the path to the JSONPB of a
86 // dm.Execution.Auth for the current execution. Consuming this file is 88 // dm.Execution.Auth for the current execution. Consuming this file is
87 // currently required to interact with DM, making this substitution 89 // currently required to interact with DM, making this substitution
88 // parameter a requirement. 90 // parameter a requirement.
89 // ${DM.HOST} - the hostname to use to access DM's pRPC API. 91 // ${DM.HOST} - the hostname to use to access DM's pRPC API.
90 // 92 //
91 // Command MUST be specified; specifying a command in any of the isolated 93 // Command MUST be specified; specifying a command in any of the isolated
92 // inputs WILL NOT DO ANYTHING. 94 // inputs WILL NOT DO ANYTHING.
93 repeated string command = 2; 95 repeated string command = 2;
94 96
95 // Environment variables for the executions. 97 // Environment variables for the executions.
96 map<string, string> env = 4; 98 map<string, string> env = 4;
97 } 99 }
98 Job job = 3; 100 Job job = 3;
99 } 101 }
OLDNEW
« no previous file with comments | « dm/api/distributor/swarming/v1/normalize.go ('k') | dm/api/distributor/swarming/v1/params.pb.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698