blob: a854a01b99b208ea703c665f13e09a4fd39d8539 [file] [log] [blame]
// Copyright 2020 The LUCI Authors.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
package ledcli
import (
"context"
"net/http"
"net/url"
"strconv"
"strings"
"google.golang.org/grpc"
"google.golang.org/grpc/codes"
"github.com/maruel/subcommands"
"go.chromium.org/luci/auth"
bbpb "go.chromium.org/luci/buildbucket/proto"
gerritapi "go.chromium.org/luci/common/api/gerrit"
"go.chromium.org/luci/common/errors"
gerritpb "go.chromium.org/luci/common/proto/gerrit"
"go.chromium.org/luci/led/job"
)
func editCrCLCmd(opts cmdBaseOptions) *subcommands.Command {
return &subcommands.Command{
UsageLine: "edit-cr-cl [-remove|-no-implicit-clear] URL_TO_CHANGELIST",
ShortDesc: "sets Chromium CL-related properties on this JobDefinition (for experimenting with tryjob recipes)",
LongDesc: `This allows you to edit a JobDefinition for some tryjob recipe
(e.g. chromium_tryjob), and associate a changelist with it, as if the recipe
was triggered via Gerrit.
Recognized URLs:
https://<gerrit_host>/c/<path/to/project>/+/<change>
https://<gerrit_host>/c/<path/to/project>/+/<change>/<patchset>
If you provide a CL missing <patchset> AND <gerrit_host> has public read access,
this will fill in the patchset from the latest version of the change. Otherwise
this will fail and ask you to provide the full CL/patchset url.
By default, when adding a CL, this will clear all existing CLs on the job, unless
you pass -no-implicit-clear. Most jobs (as of 2020Q2) only expect one CL, so we
did this implicit clearing behavior for CLI ergonomic reasons.
`,
CommandRun: func() subcommands.CommandRun {
ret := &cmdEditCl{}
ret.initFlags(opts)
return ret
},
}
}
type cmdEditCl struct {
cmdBase
gerritChange *bbpb.GerritChange
remove bool
noImplicitClear bool
}
func (c *cmdEditCl) initFlags(opts cmdBaseOptions) {
c.Flags.BoolVar(&c.remove, "remove", false, "If provided, will remove the given CL instead of adding it.")
c.Flags.BoolVar(&c.noImplicitClear, "no-implicit-clear", false,
"If provided, will not clear existing CLs when adding a new one.")
c.cmdBase.initFlags(opts)
}
func (c *cmdEditCl) jobInput() bool { return true }
func (c *cmdEditCl) positionalRange() (min, max int) { return 1, 1 }
type patchsetResolver func(host string, change int64) (ps int64, err error)
func parseCrChangeListURL(clURL string, resolvePatchset patchsetResolver) (*bbpb.GerritChange, error) {
p, err := url.Parse(clURL)
if err != nil {
return nil, errors.Annotate(err, "URL_TO_CHANGELIST").Err()
}
if !strings.HasSuffix(p.Hostname(), "-review.googlesource.com") {
return nil, errors.New("only *-review.googlesource.com URLs are supported")
}
var toks []string
if trimPath := strings.Trim(p.Path, "/"); len(trimPath) > 0 {
toks = strings.Split(trimPath, "/")
}
if len(toks) == 0 {
// https://<gerrit_host>/#/c/<change>
// https://<gerrit_host>/#/c/<change>/<patchset>
return nil, errors.Reason("old/empty gerrit URL: %q", clURL).Err()
} else if toks[0] != "c" {
return nil, errors.Reason("Unknown changelist URL format: %q", clURL).Err()
}
toks = toks[1:] // remove "c"
// toks == v --------------------------------v
// https://<gerrit_host>/c/<change>
// https://<gerrit_host>/c/<change>/<patchset>
// https://<gerrit_host>/c/<project/path>/+/<change>
// https://<gerrit_host>/c/<project/path>/+/<change>/<patchset>
var projectToks []string
var changePatchsetToks []string
for i, tok := range toks {
if tok == "+" {
projectToks, changePatchsetToks = toks[:i], toks[i+1:]
break
}
}
if len(projectToks) == 0 {
return nil, errors.Reason("gerrit URL missing project: %q", clURL).Err()
}
if len(changePatchsetToks) == 0 {
return nil, errors.Reason("gerrit URL missing change/patchset: %q", clURL).Err()
}
ret := &bbpb.GerritChange{
Host: p.Hostname(),
Project: strings.Join(projectToks, "/"),
}
ret.Change, err = strconv.ParseInt(changePatchsetToks[0], 10, 64)
if err != nil {
return nil, errors.Reason("gerrit URL parsing change %q from %q", changePatchsetToks[0], clURL).Err()
}
if len(changePatchsetToks) > 1 {
ret.Patchset, err = strconv.ParseInt(changePatchsetToks[1], 10, 64)
if err != nil {
return nil, errors.Reason("gerrit URL parsing patchset %q from %q", changePatchsetToks[1], clURL).Err()
}
} else {
ret.Patchset, err = resolvePatchset(ret.Host, ret.Change)
if err != nil {
return nil, errors.Annotate(
err, "resolving patchset from Gerrit Url %q", clURL).Err()
}
}
return ret, nil
}
func gerritResolver(ctx context.Context) patchsetResolver {
return func(host string, change int64) (int64, error) {
// TODO(crbug/1211623): allow authentication for internal hosts.
gc, err := gerritapi.NewRESTClient(http.DefaultClient, host, false)
if err != nil {
return 0, errors.Annotate(err, "creating new gerrit client").Err()
}
ci, err := gc.GetChange(ctx, &gerritpb.GetChangeRequest{
Number: change,
Options: []gerritpb.QueryOption{
gerritpb.QueryOption_CURRENT_REVISION,
},
})
if grpc.Code(err) == codes.Unauthenticated {
return 0, errors.Annotate(err,
"Gerrit host %q requires authentication and no patchset was provided. "+
"Please include the patchset you want in your URL (or add a patchset "+
"`0` to ignore this).", host,
).Err()
}
if err != nil {
return 0, errors.Annotate(err, "GetChange").Err()
}
// There's only one.
for _, rd := range ci.Revisions {
return int64(rd.Number), nil
}
panic("impossible")
}
}
func (c *cmdEditCl) validateFlags(ctx context.Context, positionals []string, _ subcommands.Env) (err error) {
if c.remove && c.noImplicitClear {
return errors.New("cannot specify both -remove and -no-implicit-clear")
}
c.gerritChange, err = parseCrChangeListURL(positionals[0], gerritResolver(ctx))
return errors.Annotate(err, "invalid URL_TO_CHANGESET").Err()
}
func (c *cmdEditCl) execute(ctx context.Context, _ *http.Client, _ auth.Options, inJob *job.Definition) (out interface{}, err error) {
return inJob, inJob.HighLevelEdit(func(je job.HighLevelEditor) {
if c.remove {
je.RemoveGerritChange(c.gerritChange)
} else {
if !c.noImplicitClear {
je.ClearGerritChanges()
}
je.AddGerritChange(c.gerritChange)
}
// wipe out all the old properties
je.Properties(map[string]string{
"blamelist": "",
"buildbucket": "",
"issue": "",
"patch_gerrit_url": "",
"patch_issue": "",
"patch_project": "",
"patch_ref": "",
"patch_repository_url": "",
"patch_set": "",
"patch_storage": "",
"patchset": "",
"repository": "",
"rietveld": "",
}, true)
})
}
func (c *cmdEditCl) Run(a subcommands.Application, args []string, env subcommands.Env) int {
return c.doContextExecute(a, c, args, env)
}