Austin Eng | cc2516a | 2023-10-17 20:57:54 +0000 | [diff] [blame] | 1 | // Copyright 2021 The Dawn & Tint Authors |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 2 | // |
Austin Eng | cc2516a | 2023-10-17 20:57:54 +0000 | [diff] [blame] | 3 | // Redistribution and use in source and binary forms, with or without |
| 4 | // modification, are permitted provided that the following conditions are met: |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 5 | // |
Austin Eng | cc2516a | 2023-10-17 20:57:54 +0000 | [diff] [blame] | 6 | // 1. Redistributions of source code must retain the above copyright notice, this |
| 7 | // list of conditions and the following disclaimer. |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 8 | // |
Austin Eng | cc2516a | 2023-10-17 20:57:54 +0000 | [diff] [blame] | 9 | // 2. Redistributions in binary form must reproduce the above copyright notice, |
| 10 | // this list of conditions and the following disclaimer in the documentation |
| 11 | // and/or other materials provided with the distribution. |
| 12 | // |
| 13 | // 3. Neither the name of the copyright holder nor the names of its |
| 14 | // contributors may be used to endorse or promote products derived from |
| 15 | // this software without specific prior written permission. |
| 16 | // |
| 17 | // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" |
| 18 | // AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE |
| 19 | // IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE |
| 20 | // DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE |
| 21 | // FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL |
| 22 | // DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR |
| 23 | // SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER |
| 24 | // CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, |
| 25 | // OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE |
| 26 | // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 27 | |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 28 | // Package gerrit provides helpers for obtaining information from Tint's gerrit instance |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 29 | package gerrit |
| 30 | |
| 31 | import ( |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 32 | "context" |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 33 | "flag" |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 34 | "fmt" |
Austin Eng | 260ae98 | 2023-08-21 21:22:22 +0000 | [diff] [blame] | 35 | "log" |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 36 | "net/url" |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 37 | "strconv" |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 38 | "strings" |
| 39 | |
Ben Clayton | 1a8d078 | 2023-05-16 14:36:37 +0000 | [diff] [blame] | 40 | "dawn.googlesource.com/dawn/tools/src/container" |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 41 | "github.com/andygrunwald/go-gerrit" |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 42 | "go.chromium.org/luci/auth" |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 43 | ) |
| 44 | |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 45 | // Gerrit is the interface to gerrit |
| 46 | type Gerrit struct { |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 47 | client *gerrit.Client |
| 48 | authenticated bool |
| 49 | } |
| 50 | |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 51 | // Patchset refers to a single gerrit patchset |
| 52 | type Patchset struct { |
| 53 | // Gerrit host |
| 54 | Host string |
| 55 | // Gerrit project |
| 56 | Project string |
| 57 | // Change ID |
| 58 | Change int |
| 59 | // Patchset ID |
| 60 | Patchset int |
Ben Clayton | 5209a81 | 2021-07-30 16:20:46 +0000 | [diff] [blame] | 61 | } |
| 62 | |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 63 | // ChangeInfo is an alias to gerrit.ChangeInfo |
| 64 | type ChangeInfo = gerrit.ChangeInfo |
| 65 | |
Ben Clayton | 1a8d078 | 2023-05-16 14:36:37 +0000 | [diff] [blame] | 66 | // LatestPatchset returns the latest Patchset from the ChangeInfo |
| 67 | func LatestPatchset(change *ChangeInfo) Patchset { |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 68 | u, _ := url.Parse(change.URL) |
| 69 | ps := Patchset{ |
| 70 | Host: u.Host, |
| 71 | Project: change.Project, |
| 72 | Change: change.Number, |
| 73 | Patchset: change.Revisions[change.CurrentRevision].Number, |
| 74 | } |
| 75 | return ps |
| 76 | } |
| 77 | |
| 78 | // RegisterFlags registers the command line flags to populate p |
| 79 | func (p *Patchset) RegisterFlags(defaultHost, defaultProject string) { |
| 80 | flag.StringVar(&p.Host, "host", defaultHost, "gerrit host") |
| 81 | flag.StringVar(&p.Project, "project", defaultProject, "gerrit project") |
| 82 | flag.IntVar(&p.Change, "cl", 0, "gerrit change id") |
| 83 | flag.IntVar(&p.Patchset, "ps", 0, "gerrit patchset id") |
| 84 | } |
| 85 | |
Ben Clayton | 526087b | 2022-04-29 15:15:43 +0000 | [diff] [blame] | 86 | // RefsChanges returns the gerrit 'refs/changes/X/Y/Z' string for the patchset |
| 87 | func (p Patchset) RefsChanges() string { |
| 88 | // https://gerrit-review.googlesource.com/Documentation/intro-user.html |
| 89 | // A change ref has the format refs/changes/X/Y/Z where X is the last two |
| 90 | // digits of the change number, Y is the entire change number, and Z is the |
| 91 | // patch set. For example, if the change number is 263270, the ref would be |
| 92 | // refs/changes/70/263270/2 for the second patch set. |
| 93 | shortChange := fmt.Sprintf("%.2v", p.Change) |
| 94 | shortChange = shortChange[len(shortChange)-2:] |
| 95 | return fmt.Sprintf("refs/changes/%v/%v/%v", shortChange, p.Change, p.Patchset) |
| 96 | } |
| 97 | |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 98 | // New returns a new Gerrit instance. If credentials are not provided, then |
| 99 | // New() will automatically attempt to load them from the gitcookies file. |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 100 | func New(ctx context.Context, opts auth.Options, url string) (*Gerrit, error) { |
| 101 | http, err := auth.NewAuthenticator(ctx, auth.InteractiveLogin, opts).Client() |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 102 | if err != nil { |
Ben Clayton | 5209a81 | 2021-07-30 16:20:46 +0000 | [diff] [blame] | 103 | return nil, fmt.Errorf("couldn't create gerrit client: %w", err) |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 104 | } |
| 105 | |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 106 | client, err := gerrit.NewClient(url, http) |
| 107 | if err != nil { |
| 108 | return nil, fmt.Errorf("couldn't create gerrit client: %w", err) |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 109 | } |
| 110 | |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 111 | return &Gerrit{client, true}, nil |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 112 | } |
| 113 | |
Ben Clayton | dededb1 | 2023-05-18 11:30:07 +0000 | [diff] [blame] | 114 | // QueryExtraData holds extra data to query for with QueryChangesWith() |
| 115 | type QueryExtraData struct { |
| 116 | Labels bool |
| 117 | Messages bool |
| 118 | CurrentRevision bool |
| 119 | DetailedAccounts bool |
Ben Clayton | 4cdc6bf | 2023-06-01 00:40:30 +0000 | [diff] [blame] | 120 | Submittable bool |
Ben Clayton | dededb1 | 2023-05-18 11:30:07 +0000 | [diff] [blame] | 121 | } |
| 122 | |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 123 | // QueryChanges returns the changes that match the given query strings. |
| 124 | // See: https://gerrit-review.googlesource.com/Documentation/user-search.html#search-operators |
Ben Clayton | dededb1 | 2023-05-18 11:30:07 +0000 | [diff] [blame] | 125 | func (g *Gerrit) QueryChangesWith(extras QueryExtraData, queries ...string) (changes []gerrit.ChangeInfo, query string, err error) { |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 126 | changes = []gerrit.ChangeInfo{} |
Ben Clayton | dededb1 | 2023-05-18 11:30:07 +0000 | [diff] [blame] | 127 | query = strings.Join(queries, "+") |
| 128 | |
| 129 | changeOpts := gerrit.ChangeOptions{} |
| 130 | if extras.Labels { |
| 131 | changeOpts.AdditionalFields = append(changeOpts.AdditionalFields, "LABELS") |
| 132 | } |
| 133 | if extras.Messages { |
| 134 | changeOpts.AdditionalFields = append(changeOpts.AdditionalFields, "MESSAGES") |
| 135 | } |
| 136 | if extras.CurrentRevision { |
| 137 | changeOpts.AdditionalFields = append(changeOpts.AdditionalFields, "CURRENT_REVISION") |
| 138 | } |
| 139 | if extras.DetailedAccounts { |
| 140 | changeOpts.AdditionalFields = append(changeOpts.AdditionalFields, "DETAILED_ACCOUNTS") |
| 141 | } |
Ben Clayton | 4cdc6bf | 2023-06-01 00:40:30 +0000 | [diff] [blame] | 142 | if extras.Submittable { |
| 143 | changeOpts.AdditionalFields = append(changeOpts.AdditionalFields, "SUBMITTABLE") |
| 144 | } |
Ben Clayton | dededb1 | 2023-05-18 11:30:07 +0000 | [diff] [blame] | 145 | |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 146 | for { |
| 147 | batch, _, err := g.client.Changes.QueryChanges(&gerrit.QueryChangeOptions{ |
Ben Clayton | dededb1 | 2023-05-18 11:30:07 +0000 | [diff] [blame] | 148 | QueryOptions: gerrit.QueryOptions{Query: []string{query}}, |
| 149 | Skip: len(changes), |
| 150 | ChangeOptions: changeOpts, |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 151 | }) |
| 152 | if err != nil { |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 153 | return nil, "", err |
Ben Clayton | f6660aa | 2021-07-20 20:25:38 +0000 | [diff] [blame] | 154 | } |
| 155 | |
| 156 | changes = append(changes, *batch...) |
| 157 | if len(*batch) == 0 || !(*batch)[len(*batch)-1].MoreChanges { |
| 158 | break |
| 159 | } |
| 160 | } |
| 161 | return changes, query, nil |
| 162 | } |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 163 | |
Ben Clayton | dededb1 | 2023-05-18 11:30:07 +0000 | [diff] [blame] | 164 | // QueryChanges returns the changes that match the given query strings. |
| 165 | // See: https://gerrit-review.googlesource.com/Documentation/user-search.html#search-operators |
| 166 | func (g *Gerrit) QueryChanges(queries ...string) (changes []gerrit.ChangeInfo, query string, err error) { |
| 167 | return g.QueryChangesWith(QueryExtraData{}, queries...) |
| 168 | } |
| 169 | |
Ben Clayton | 4cdc6bf | 2023-06-01 00:40:30 +0000 | [diff] [blame] | 170 | // ChangesSubmittedTogether returns the changes that want to be submitted together |
| 171 | // See: https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#submitted-together |
| 172 | func (g *Gerrit) ChangesSubmittedTogether(changeID string) (changes []gerrit.ChangeInfo, err error) { |
| 173 | info, _, err := g.client.Changes.ChangesSubmittedTogether(changeID) |
| 174 | if err != nil { |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 175 | return nil, err |
Ben Clayton | 4cdc6bf | 2023-06-01 00:40:30 +0000 | [diff] [blame] | 176 | } |
| 177 | return *info, nil |
| 178 | } |
| 179 | |
Ben Clayton | dededb1 | 2023-05-18 11:30:07 +0000 | [diff] [blame] | 180 | func (g *Gerrit) AddLabel(changeID, revisionID, message, label string, value int) error { |
| 181 | _, _, err := g.client.Changes.SetReview(changeID, revisionID, &gerrit.ReviewInput{ |
| 182 | Message: message, |
| 183 | Labels: map[string]string{label: fmt.Sprint(value)}, |
| 184 | }) |
| 185 | if err != nil { |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 186 | return err |
Ben Clayton | dededb1 | 2023-05-18 11:30:07 +0000 | [diff] [blame] | 187 | } |
| 188 | return nil |
| 189 | } |
| 190 | |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 191 | // Abandon abandons the change with the given changeID. |
| 192 | func (g *Gerrit) Abandon(changeID string) error { |
| 193 | _, _, err := g.client.Changes.AbandonChange(changeID, &gerrit.AbandonInput{}) |
| 194 | if err != nil { |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 195 | return err |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 196 | } |
| 197 | return nil |
| 198 | } |
| 199 | |
| 200 | // CreateChange creates a new change in the given project and branch, with the |
| 201 | // given subject. If wip is true, then the change is constructed as |
| 202 | // Work-In-Progress. |
| 203 | func (g *Gerrit) CreateChange(project, branch, subject string, wip bool) (*ChangeInfo, error) { |
| 204 | change, _, err := g.client.Changes.CreateChange(&gerrit.ChangeInput{ |
| 205 | Project: project, |
| 206 | Branch: branch, |
| 207 | Subject: subject, |
| 208 | WorkInProgress: wip, |
| 209 | }) |
| 210 | if err != nil { |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 211 | return nil, err |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 212 | } |
Ben Clayton | 9cae32e | 2024-03-19 19:23:19 +0000 | [diff] [blame] | 213 | if change.URL == "" { |
| 214 | base := g.client.BaseURL() |
| 215 | change.URL = fmt.Sprintf("%vc/%v/+/%v", base.String(), change.Project, change.Number) |
| 216 | } |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 217 | return change, nil |
| 218 | } |
| 219 | |
Austin Eng | 9d14b16 | 2022-06-10 16:25:43 +0000 | [diff] [blame] | 220 | // EditFiles replaces the content of the files in the given change. It deletes deletedFiles. |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 221 | // If newCommitMsg is not an empty string, then the commit message is replaced |
| 222 | // with the string value. |
Austin Eng | 9d14b16 | 2022-06-10 16:25:43 +0000 | [diff] [blame] | 223 | func (g *Gerrit) EditFiles(changeID, newCommitMsg string, files map[string]string, deletedFiles []string) (Patchset, error) { |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 224 | if newCommitMsg != "" { |
| 225 | resp, err := g.client.Changes.ChangeCommitMessageInChangeEdit(changeID, &gerrit.ChangeEditMessageInput{ |
| 226 | Message: newCommitMsg, |
| 227 | }) |
| 228 | if err != nil && resp.StatusCode != 409 { // 409 no changes were made |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 229 | return Patchset{}, err |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 230 | } |
| 231 | } |
| 232 | for path, content := range files { |
| 233 | resp, err := g.client.Changes.ChangeFileContentInChangeEdit(changeID, path, content) |
| 234 | if err != nil && resp.StatusCode != 409 { // 409 no changes were made |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 235 | return Patchset{}, err |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 236 | } |
| 237 | } |
Austin Eng | 9d14b16 | 2022-06-10 16:25:43 +0000 | [diff] [blame] | 238 | for _, path := range deletedFiles { |
| 239 | resp, err := g.client.Changes.DeleteFileInChangeEdit(changeID, path) |
| 240 | if err != nil && resp.StatusCode != 409 { // 409 no changes were made |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 241 | return Patchset{}, err |
Austin Eng | 9d14b16 | 2022-06-10 16:25:43 +0000 | [diff] [blame] | 242 | } |
| 243 | } |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 244 | |
| 245 | resp, err := g.client.Changes.PublishChangeEdit(changeID, "NONE") |
| 246 | if err != nil && resp.StatusCode != 409 { // 409 no changes were made |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 247 | return Patchset{}, err |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 248 | } |
| 249 | |
Ben Clayton | 1a8d078 | 2023-05-16 14:36:37 +0000 | [diff] [blame] | 250 | return g.LatestPatchset(changeID) |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 251 | } |
| 252 | |
Ben Clayton | 1a8d078 | 2023-05-16 14:36:37 +0000 | [diff] [blame] | 253 | // LatestPatchset returns the latest patchset for the change. |
| 254 | func (g *Gerrit) LatestPatchset(changeID string) (Patchset, error) { |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 255 | change, _, err := g.client.Changes.GetChange(changeID, &gerrit.ChangeOptions{ |
| 256 | AdditionalFields: []string{"CURRENT_REVISION"}, |
| 257 | }) |
| 258 | if err != nil { |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 259 | return Patchset{}, err |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 260 | } |
| 261 | ps := Patchset{ |
| 262 | Host: g.client.BaseURL().Host, |
| 263 | Project: change.Project, |
| 264 | Change: change.Number, |
| 265 | Patchset: change.Revisions[change.CurrentRevision].Number, |
| 266 | } |
| 267 | return ps, nil |
| 268 | } |
| 269 | |
Ben Clayton | 1a8d078 | 2023-05-16 14:36:37 +0000 | [diff] [blame] | 270 | // AddHashtags adds the given hashtags to the change |
| 271 | func (g *Gerrit) AddHashtags(changeID string, tags container.Set[string]) error { |
| 272 | _, resp, err := g.client.Changes.SetHashtags(changeID, &gerrit.HashtagsInput{ |
| 273 | Add: tags.List(), |
| 274 | }) |
| 275 | if err != nil && resp.StatusCode != 409 { // 409: already ready |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 276 | return err |
Ben Clayton | 1a8d078 | 2023-05-16 14:36:37 +0000 | [diff] [blame] | 277 | } |
| 278 | return nil |
| 279 | } |
| 280 | |
Ben Clayton | 526087b | 2022-04-29 15:15:43 +0000 | [diff] [blame] | 281 | // CommentSide is an enumerator for specifying which side code-comments should |
| 282 | // be shown. |
| 283 | type CommentSide int |
| 284 | |
| 285 | const ( |
Ben Clayton | ea7d7fe | 2024-02-28 00:23:17 +0000 | [diff] [blame] | 286 | // Left is used to specify that code comments should appear on the parent change |
Ben Clayton | 526087b | 2022-04-29 15:15:43 +0000 | [diff] [blame] | 287 | Left CommentSide = iota |
Ben Clayton | ea7d7fe | 2024-02-28 00:23:17 +0000 | [diff] [blame] | 288 | // Right is used to specify that code comments should appear on the new change |
Ben Clayton | 526087b | 2022-04-29 15:15:43 +0000 | [diff] [blame] | 289 | Right |
| 290 | ) |
| 291 | |
| 292 | // FileComment describes a single comment on a file |
| 293 | type FileComment struct { |
| 294 | Path string // The file path |
| 295 | Side CommentSide // Which side the comment should appear |
| 296 | Line int // The 1-based line number for the comment |
| 297 | Message string // The comment message |
| 298 | } |
| 299 | |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 300 | // Comment posts a review comment on the given patchset. |
Ben Clayton | 526087b | 2022-04-29 15:15:43 +0000 | [diff] [blame] | 301 | // If comments is an optional list of file-comments to include in the comment. |
| 302 | func (g *Gerrit) Comment(ps Patchset, msg string, comments []FileComment) error { |
| 303 | input := &gerrit.ReviewInput{ |
| 304 | Message: msg, |
| 305 | } |
| 306 | if len(comments) > 0 { |
| 307 | input.Comments = map[string][]gerrit.CommentInput{} |
| 308 | for _, c := range comments { |
| 309 | ci := gerrit.CommentInput{ |
| 310 | Line: c.Line, |
| 311 | // Updated: &gerrit.Timestamp{Time: time.Now()}, |
| 312 | Message: c.Message, |
| 313 | } |
| 314 | if c.Side == Left { |
| 315 | ci.Side = "PARENT" |
| 316 | } else { |
| 317 | ci.Side = "REVISION" |
| 318 | } |
| 319 | input.Comments[c.Path] = append(input.Comments[c.Path], ci) |
| 320 | } |
| 321 | } |
| 322 | _, _, err := g.client.Changes.SetReview(strconv.Itoa(ps.Change), strconv.Itoa(ps.Patchset), input) |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 323 | if err != nil { |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 324 | return err |
Ben Clayton | 53ddabe | 2022-04-12 16:13:31 +0000 | [diff] [blame] | 325 | } |
| 326 | return nil |
| 327 | } |
Ben Clayton | 526087b | 2022-04-29 15:15:43 +0000 | [diff] [blame] | 328 | |
| 329 | // SetReadyForReview marks the change as ready for review. |
Austin Eng | 260ae98 | 2023-08-21 21:22:22 +0000 | [diff] [blame] | 330 | func (g *Gerrit) SetReadyForReview(changeID, message, reviewer string) error { |
Ben Clayton | 526087b | 2022-04-29 15:15:43 +0000 | [diff] [blame] | 331 | resp, err := g.client.Changes.SetReadyForReview(changeID, &gerrit.ReadyForReviewInput{ |
| 332 | Message: message, |
| 333 | }) |
| 334 | if err != nil && resp.StatusCode != 409 { // 409: already ready |
Austin Eng | 0dae5ce | 2023-08-17 22:07:36 +0000 | [diff] [blame] | 335 | return err |
Ben Clayton | 526087b | 2022-04-29 15:15:43 +0000 | [diff] [blame] | 336 | } |
Austin Eng | 260ae98 | 2023-08-21 21:22:22 +0000 | [diff] [blame] | 337 | if reviewer != "" { |
Austin Eng | 260ae98 | 2023-08-21 21:22:22 +0000 | [diff] [blame] | 338 | log.Printf("Got reviewer %s", reviewer) |
Austin Eng | 260ae98 | 2023-08-21 21:22:22 +0000 | [diff] [blame] | 339 | _, resp, err = g.client.Changes.AddReviewer(changeID, &gerrit.ReviewerInput{ |
| 340 | Reviewer: reviewer, |
| 341 | }) |
| 342 | if err != nil && resp.StatusCode != 409 { // 409: already ready |
| 343 | return err |
| 344 | } |
| 345 | } |
Ben Clayton | 526087b | 2022-04-29 15:15:43 +0000 | [diff] [blame] | 346 | return nil |
| 347 | } |