| -@gerrit:opendev.org- Joseph Kostreva proposed: [zuul/zuul] 923183: Add commentable_files variable to GerritChange https://review.opendev.org/c/zuul/zuul/+/923183 | 16:05 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 18:11 | |
| - [zuul/zuul] 937212: Use empty sparse checkouts in mergers https://review.opendev.org/c/zuul/zuul/+/937212 | ||
| - [zuul/zuul] 937571: Parallelize workspace repo restoration https://review.opendev.org/c/zuul/zuul/+/937571 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 19:04 | |
| - [zuul/zuul] 937384: Use a TreeCache for job request queues https://review.opendev.org/c/zuul/zuul/+/937384 | ||
| - [zuul/zuul] 937385: Reduce ZK lock contention in executor https://review.opendev.org/c/zuul/zuul/+/937385 | ||
| - [zuul/zuul] 937386: Disable cache event log https://review.opendev.org/c/zuul/zuul/+/937386 | ||
| - [zuul/zuul] 937387: Make executor sensor messages more useful https://review.opendev.org/c/zuul/zuul/+/937387 | ||
| - [zuul/zuul] 937469: Use rendezvous hashing for executor distribution https://review.opendev.org/c/zuul/zuul/+/937469 | ||
| - [zuul/zuul] 937470: Use Rendezvous hashing for merger jobs https://review.opendev.org/c/zuul/zuul/+/937470 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 21:40 | |
| - [zuul/zuul] 937384: Use a TreeCache for job request queues https://review.opendev.org/c/zuul/zuul/+/937384 | ||
| - [zuul/zuul] 937385: Reduce ZK lock contention in executor https://review.opendev.org/c/zuul/zuul/+/937385 | ||
| - [zuul/zuul] 937386: Disable cache event log https://review.opendev.org/c/zuul/zuul/+/937386 | ||
| - [zuul/zuul] 937387: Make executor sensor messages more useful https://review.opendev.org/c/zuul/zuul/+/937387 | ||
| - [zuul/zuul] 937469: Use rendezvous hashing for executor distribution https://review.opendev.org/c/zuul/zuul/+/937469 | ||
| - [zuul/zuul] 937470: Use Rendezvous hashing for merger jobs https://review.opendev.org/c/zuul/zuul/+/937470 | ||
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 937654: Push and approve Gerrit ACL update in one command https://review.opendev.org/c/zuul/zuul/+/937654 | 21:42 | |
| @clarkb:matrix.org | corvus: fungi ^ I mentioned the 3.11 testing we're doing and this shortcut was suggested to me on the Gerrit discord | 21:42 |
|---|---|---|
| @clarkb:matrix.org | I am indifferent if we actually want to use it but I Figured I'd push it up so we test that it actually works and also are aware it may be an option | 21:43 |
| @jim:acmegating.com | Clark: neat trick; happy to drop the extra use of ssh there | 21:53 |
| @fungicide:matrix.org | huh, yeah i always forget about the push options, even though git-review makes significant use of them | 22:20 |
| @clarkb:matrix.org | corvus: https://zuul.opendev.org/t/zuul/build/a1642b1b6bc94cc4a213192cd1870cc4/log/job-output.txt#1474 this looks like gerrit may not have started? Which I think happens before the bit I modified | 23:41 |
| @clarkb:matrix.org | looking at the gerrit container logs it does seem to have been running though so I'm nto sure what happened there | 23:42 |
| @clarkb:matrix.org | maybe it is related to my change somehow (timing?) | 23:43 |
| @jim:acmegating.com | https://zuul.opendev.org/t/zuul/build/a1642b1b6bc94cc4a213192cd1870cc4/log/container_logs/gerritconfig.log | 23:50 |
| @jim:acmegating.com | Clark: i read line 44 there (with some effort) as saying that's a no-go | 23:51 |
| @jim:acmegating.com | HEAD -> refs/for/refs/meta/config%l=Code-Review+2,submit (prohibited by Gerrit: not permitted: update by submit on refs/meta/config)" | 23:51 |
| @clarkb:matrix.org | huh like its saying we can't do the submit push to refs/meta/config specifically? | 23:52 |
| @jim:acmegating.com | yeah i've never seen that error, but that sure sounds like what it's trying to say | 23:52 |
| @fungicide:matrix.org | doesn't look familiar to me either, but yeah that's the only interpretation i can come up with | 23:52 |
| @jim:acmegating.com | like they specifically made an error message to say you can't do the thing the folks in the gerrit discord said to try. :) | 23:53 |
| @fungicide:matrix.org | i know you can set label votes that way, so it's gotta be the submit specifically | 23:53 |
| @jim:acmegating.com | right, i assume it's closing another "must-use-code-review" loophole, so requires bootstrapping that perm. | 23:54 |
| @jim:acmegating.com | i just hope they don't require a captcha next. :) | 23:55 |
| @clarkb:matrix.org | https://issues.gerritcodereview.com/issues/40003142 I think this discussion explains it | 23:55 |
| @clarkb:matrix.org | the permissions for clicking the submit button are different than the permissions for using the submit on push option I guess | 23:55 |
| @clarkb:matrix.org | and the default perms for admin must only allow the click the button option? | 23:55 |
| @jim:acmegating.com | sounds right | 23:55 |
| @fungicide:matrix.org | le sigh | 23:57 |
| @clarkb:matrix.org | I haev passed this back along into discord | 23:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!