Updated cq_client

third_party/cq_client/cq.proto

 syntax = "proto2";
 
 // This message describes a Commit Queue configuration. The config file cq.cfg
 // should be stored in the config directory located on the branch that this CQ
 // should commit to.
 message Config {
   // Required. Version of the config format.
   optional int32 version = 1;
 
   // Required. Name of the CQ. May only contain characters [a-zA-Z0-9_]. It is
@@ skipping 21 matching lines @@
   // commit_burst_delay. Default value is 4.
   optional int32 max_commit_burst = 7;
 
   // Defines whether a CQ is used in production. Allows to disable CQ for a
   // given branch. Default is true.
   optional bool in_production = 8;
 
   // Configuration options for Rietveld code review.
   optional Rietveld rietveld = 9;
 
+  // EXPERIMENTAL! Configuration options for Gerrit code review.
+  // TODO(tandrii): update this doc (GERRIT).
+  optional Gerrit gerrit = 15;
+
   // This can be used to override the Git repository URL used to checkout and
   // commit changes on CQ host. This should only be used in case, when the
   // source repository is not supported by luci-config (e.g. GitHub).
   optional string git_repo_url = 10;
 
   // Target ref to commit to. This can be used to specify a different ref than
   // the one where the luci config is located. This is useful, e.g. for projects
   // that use gnumbd where CQ should commit into a pending ref.
   optional string target_ref = 11;
 
   // Deprecated. URL of the SVN repository. We are deprecating SVN support.
   optional string svn_repo_url = 12;
-
-  // Deprecated. Should be set to true, when the project's SVN repository does
-  // not have server-side hooks configured.
-  optional bool server_hooks_missing = 13;
-
-  // Deprecated. Specifies a list of verifiers that are run on a local checkout
-  // with patch applied. The only remaining use case for this is PRESUBMIT_CHECK
-  // verifier, which we are deprecating as well.
-  optional Verifiers verifiers_with_patch = 14;
 }
 
 message Rietveld {
   // Required. URL of the codereview site.
   optional string url = 1;
 
   // List of regular expressions used to check if CL's base URL should be
   // processed by this CQ. This may be useful if a single branch has multiple
   // sub-directories that are handled by different CQs. When no regular
   // expressions are specified, the regular expression '.*', which matches any
   // directory, is used.
   repeated string project_bases = 2;
 }
 
+// Gerrit CQ is EXPERIMENTAL! See http://crbug.com/493899 for more info.
+//
+// Unlike Rietveld, Gerrit doesn't need a separate url.
+// Instead, the git_repo_url must be specified on the Gerrit instance,
+// and CQ will deduce Gerrit url from it.
+//
+// TODO(tandrii): support Rietveld and Gerrit at the same time.
+// This basically requires to start two CQ instances, instead of one.
+//
+// For example, if https://chromium.googlesource.com/infra/infra.git is your
+// repo url provided in `git_repo_url` above, then
+// https://chromium-review.googlesource.com/#/admin/projects/infra/infra should
+// show general properties of your project.
+//
+// Also,
+// https://chromium-review.googlesource.com/#/admin/projects/infra/infra,access
+// should show ACLs for refs in your project, but you may need to be admin to
+// see it. This will come handy to enable and customize the CQ-related workflows
+// for your project.
+message Gerrit {
+  // If set, tells CQ to set score on a given label to mark result of CQ run.
+  // Typically, this is Commit-Queue-Verified label.
+  // If not set, CQ will just try to hit submit button.
+  optional string cq_verified_label = 1;
+}
+
 // Verifiers are various types of checks that a Commit Queue performs on a CL.
 // All verifiers must pass in order for a CL to be landed. Configuration file
 // describes types of verifiers that should be applied to each CL and their
 // parameters.
 message Verifiers {
   // This verifier is used to ensure that an LGTM was posted to the code review
   // site from a valid project committer.
+  // This verifier is not supported with Gerrit.
   optional ReviewerLgtmVerifier reviewer_lgtm = 1;
 
   // This verifier is used to check tree status before committing a CL. If the
   // tree is closed, then the verifier will wait until it is reopened.
   optional TreeStatusLgtmVerifier tree_status = 2;
 
   // This verifier triggers a set of tryjobs that are to be run on builders on
   // Buildbot. It automatically retries failed try-jobs and only allows CL to
   // land if each builder has succeeded in the latest retry. If a given tryjob
   // result is too old (>1 day) it is ignored.
@@ skipping 29 matching lines @@
     optional string tree_status_url = 1;
   }
 
   message TryJobVerifier {
     message Builder {
       // Name of the builder.
       optional string name = 1;
 
       // When true, the builder is triggered by CQ. Otherwise, it is expected to
       // be triggered from another tryjob. Default value is true.
-      optional bool triggered = 2;
+      optional bool triggered_by_cq = 2;
 
       // When this field is present, it marks given builder as experimental. It
       // is only executed on a given percentage of the CLs and the outcome does
       // not affect the decicion whether a CL can land or not. This is typically
       // used to test new builders and estimate their capacity requirements.
       optional float experiment_percentage = 4;
     }
 
     message Bucket {
       // Name of the bucket. This is typically the same as a master name without
@@ skipping 25 matching lines @@
       optional int32 timeout_retry_weight = 5;
     }
 
     // Provides project specific trybot retry configuration. This overrides the
     // defaults used in the CQ.
     optional TryJobRetryConfig try_job_retry_config = 2;
   }
 
   message SignCLAVerifier {}
 }