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

Side by Side Diff: third_party/cq_client/cq.proto

Issue 1697053003: Roll cq_client. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: Created 4 years, 10 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 | Annotate | Revision Log
« no previous file with comments | « third_party/cq_client/README.md ('k') | third_party/cq_client/cq.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 syntax = "proto2"; 1 syntax = "proto2";
2 2
3 // This message describes a Commit Queue configuration. The config file cq.cfg 3 // This message describes a Commit Queue configuration. The config file cq.cfg
4 // should be stored in the config directory located on the branch that this CQ 4 // should be stored in the config directory located on the branch that this CQ
5 // should commit to. 5 // should commit to.
6 message Config { 6 message Config {
7 // Required. Version of the config format. 7 // Required. Version of the config format.
8 optional int32 version = 1; 8 optional int32 version = 1;
9 9
10 // Required. Name of the CQ. May only contain characters [a-zA-Z0-9_]. It is 10 // Required. Name of the CQ. May only contain characters [a-zA-Z0-9_]. It is
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
88 // repo url provided in `git_repo_url` above, then 88 // repo url provided in `git_repo_url` above, then
89 // https://chromium-review.googlesource.com/#/admin/projects/infra/infra should 89 // https://chromium-review.googlesource.com/#/admin/projects/infra/infra should
90 // show general properties of your project. 90 // show general properties of your project.
91 // 91 //
92 // Also, 92 // Also,
93 // https://chromium-review.googlesource.com/#/admin/projects/infra/infra,access 93 // https://chromium-review.googlesource.com/#/admin/projects/infra/infra,access
94 // should show ACLs for refs in your project, but you may need to be admin to 94 // should show ACLs for refs in your project, but you may need to be admin to
95 // see it. This will come handy to enable and customize the CQ-related workflows 95 // see it. This will come handy to enable and customize the CQ-related workflows
96 // for your project. 96 // for your project.
97 message Gerrit { 97 message Gerrit {
98 // If set, tells CQ to set score on a given label to mark result of CQ run. 98 // Optional. If set, tells CQ vote on a given label to mark result of CQ run.
99 // Typically, this is Commit-Queue-Verified label. 99 // The vote is either -1 if failed or 1 on success, and will be given on
100 // If not set, CQ will just try to hit submit button. 100 // non-dry runs only.
101 // This vote can then be used in Gerrit's rule for submitting issues, so as to
102 // require CQ run. CQ will attempt to submit issue only after setting this
103 // label.
101 optional string cq_verified_label = 1; 104 optional string cq_verified_label = 1;
102 } 105 }
103 106
104 // Verifiers are various types of checks that a Commit Queue performs on a CL. 107 // Verifiers are various types of checks that a Commit Queue performs on a CL.
105 // All verifiers must pass in order for a CL to be landed. Configuration file 108 // All verifiers must pass in order for a CL to be landed. Configuration file
106 // describes types of verifiers that should be applied to each CL and their 109 // describes types of verifiers that should be applied to each CL and their
107 // parameters. 110 // parameters.
108 message Verifiers { 111 message Verifiers {
109 // This verifier is used to ensure that an LGTM was posted to the code review 112 // This verifier is used to ensure that an LGTM was posted to the code review
110 // site from a valid project committer. 113 // site from a valid project committer.
111 // This verifier is not supported with Gerrit. 114 // This verifier is not supported with Gerrit.
112 optional ReviewerLgtmVerifier reviewer_lgtm = 1; 115 optional ReviewerLgtmVerifier reviewer_lgtm = 1;
113 116
114 // This verifier is used to check tree status before committing a CL. If the 117 // This verifier is used to check tree status before committing a CL. If the
115 // tree is closed, then the verifier will wait until it is reopened. 118 // tree is closed, then the verifier will wait until it is reopened.
116 optional TreeStatusLgtmVerifier tree_status = 2; 119 optional TreeStatusLgtmVerifier tree_status = 2;
117 120
118 // This verifier triggers a set of tryjobs that are to be run on builders on 121 // This verifier triggers a set of tryjobs that are to be run on builders on
119 // Buildbot. It automatically retries failed try-jobs and only allows CL to 122 // Buildbot. It automatically retries failed try-jobs and only allows CL to
120 // land if each builder has succeeded in the latest retry. If a given tryjob 123 // land if each builder has succeeded in the latest retry. If a given tryjob
121 // result is too old (>1 day) it is ignored. 124 // result is too old (>1 day) it is ignored.
122 optional TryJobVerifier try_job = 3; 125 optional TryJobVerifier try_job = 3;
123 126
124 // This verifier is used to ensure that the author has signed Google's 127 // This verifier is used to ensure that the author has signed Google's
125 // Contributor License Agreement. 128 // Contributor License Agreement.
126 optional SignCLAVerifier sign_cla = 4; 129 optional SignCLAVerifier sign_cla = 4;
127 130
128 message ReviewerLgtmVerifier { 131 message ReviewerLgtmVerifier {
129 // Required. Name of the chrome-infra-auth group, which contains the 132 // Required. Name of the chrome-infra-auth group, which contains the list of
130 // list of identities authorized to approve (lgtm) a CL. This list is also 133 // identities authorized to approve (lgtm) a CL and trigger CQ run or dry
131 // known as a committer list and often corresponds to the list of accounts 134 // run.
132 // that have direct commit/push access to the repository. Some older lists
133 // are still stored in the CQ source code, but are being moved to the new
134 // location (https://crbug.com/511311).
135 optional string committer_list = 1; 135 optional string committer_list = 1;
136 136
137 // Number of seconds to wait for LGTM on CQ. Default value is 0. 137 // Number of seconds to wait for LGTM on CQ. Default value is 0.
138 optional int32 max_wait_secs = 2; 138 optional int32 max_wait_secs = 2;
139 139
140 // Message to be posted to code review site when no LGTM is found. Default 140 // Message to be posted to code review site when no LGTM is found. Default
141 // value is "No LGTM from a valid reviewer yet. Only full committers are " 141 // value is "No LGTM from a valid reviewer yet. Only full committers are "
142 // "accepted.\nEven if an LGTM may have been provided, it was from a " 142 // "accepted.\nEven if an LGTM may have been provided, it was from a "
143 // "non-committer,\n_not_ a full super star committer.\nSee " 143 // "non-committer,\n_not_ a full super star committer.\nSee "
144 // "http://www.chromium.org/getting-involved/become-a-committer\nNote that " 144 // "http://www.chromium.org/getting-involved/become-a-committer\nNote that "
145 // "this has nothing to do with OWNERS files." 145 // "this has nothing to do with OWNERS files."
146 optional string no_lgtm_msg = 3; 146 optional string no_lgtm_msg = 3;
147
148 // Optional, but recommended. Name of the chrome-infra-auth group,
149 // which contains the list of identities authorized to trigger CQ dry run.
150 // This is usually the same group as tryjob-access.
151 optional string dry_run_access_list = 4;
147 } 152 }
148 153
149 message TreeStatusLgtmVerifier { 154 message TreeStatusLgtmVerifier {
150 // Required. URL of the project tree status app. 155 // Required. URL of the project tree status app.
151 optional string tree_status_url = 1; 156 optional string tree_status_url = 1;
152 } 157 }
153 158
154 message TryJobVerifier { 159 message TryJobVerifier {
155 message Builder { 160 message Builder {
156 // Name of the builder. 161 // Name of the builder.
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
198 optional int32 timeout_retry_weight = 5; 203 optional int32 timeout_retry_weight = 5;
199 } 204 }
200 205
201 // Provides project specific trybot retry configuration. This overrides the 206 // Provides project specific trybot retry configuration. This overrides the
202 // defaults used in the CQ. 207 // defaults used in the CQ.
203 optional TryJobRetryConfig try_job_retry_config = 2; 208 optional TryJobRetryConfig try_job_retry_config = 2;
204 } 209 }
205 210
206 message SignCLAVerifier {} 211 message SignCLAVerifier {}
207 } 212 }
OLDNEW
« no previous file with comments | « third_party/cq_client/README.md ('k') | third_party/cq_client/cq.pb.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698