| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 823587: Add some ZK debug scripts https://review.opendev.org/c/zuul/zuul/+/823587 | 00:59 | |
| @y2kenny:matrix.org | The password configuration for Gerrit connection is used to determine if git clone over http or ssh ( https://zuul-ci.org/docs/zuul/drivers/gerrit.html#attr-%3Cgerrit%20connection%3E.password ) and also for REST API access (such as line comments). Unfortunately, I am in a situation where the server disabled git over http. Is there a way to have both REST access while using ssh for git? If not, would you folks be open to a patch for such a feature? (Perhaps an additional connection configuration 'force_git_over_ssh' defaulting to false?) | 16:17 |
|---|---|---|
| @jim:acmegating.com | Kenny Ho: i think the answers are "no" and "yes" respectively :) and the default connection flag sounds good. could probably drop the word "force". | 16:26 |
| @y2kenny:matrix.org | corvus: great, I will dig into the code. | 16:30 |
| @mhuin:matrix.org | I'm not sure all REST features of gerrit are covered by ssh, but maybe the covered subset is enough for what zuul needs | 16:32 |
| @y2kenny:matrix.org | mhu: definitely not all REST are covered by ssh, that's why I want to have something to force git to use ssh even when http password is set such that I can use REST and ssh at the same time | 16:34 |
| @jim:acmegating.com | Kenny Ho: a workaround which may work: put the password only on the scheduler zuul.conf, and omit it n mergers and executors. | 16:41 |
| @jim:acmegating.com | * Kenny Ho: a workaround which may work: put the password only on the scheduler zuul.conf, and omit it on mergers and executors. | 16:41 |
| @y2kenny:matrix.org | corvus: oh... interesting... I hadn't thought about that (everything share the same zuul.conf on my setup currently.) | 16:43 |
| @jim:acmegating.com | yeah, it's typical and intended that zuul.conf can be universal. it's probably still worth adding the feature and flag. but if that works, it might be a quick fix (depending on if deployment tooling allows that) | 16:44 |
| @jim:acmegating.com | * yeah, it's typical and intended that zuul.conf can be universal. it's probably still worth adding the feature and flag so that it can be universal in this case. but if the split works, it might be a quick fix (depending on if deployment tooling allows that) | 16:45 |
| @y2kenny:matrix.org | My deployment will allow it but I will implement the feature anyway. Sharing zuul.conf seems cleaner and my need for REST access is not super urgent (just trying to inline comments checkpatch.pl for kernel commits.) | 16:46 |
| @jim:acmegating.com | sounds good. fyi there are other benefits for rest too including better file lists on merge commits. i expect rest to continue to improve while ssh api stays the same based on gerrit dev priorities. so regardless of how it's done, having the scheduler use rest is best. executors and mergers don't use it, they only use git (via http or ssh, it makes no difference to them). | 16:49 |
| @fungicide:matrix.org | in part because the gerrit community's gerrit deployment doesn't even make its ssh api accessible to users, so their incentive for improving it is fairly nonexistent | 17:33 |
| @clarkb:matrix.org | Talking to other Gerrit users they also seem to treat gerrit as a five 9s service and ssh event stream and other ssh attached functionality doesn't work well with active active gerrit servers. I think I managed to make a case that for those of us where that constraint doesn't exist simplicity is a feature in itself, but ya it is very often overlooked | 18:34 |
| @fungicide:matrix.org | to be fair, ssh functionality could be made redundantly available to satisfy that need, it just hasn't been made so | 18:53 |
| @fungicide:matrix.org | and it would almost certainly require a different approach from http redundancy, so i can understand them not wanting to have to do it all twice | 18:56 |
| -@gerrit:opendev.org- Kenny Ho proposed: [zuul/zuul] 823732: Add git_over_ssh option for Gerrit connection https://review.opendev.org/c/zuul/zuul/+/823732 | 20:50 | |
| -@gerrit:opendev.org- Kenny Ho proposed: [zuul/zuul] 823732: Add git_over_ssh option for Gerrit connection https://review.opendev.org/c/zuul/zuul/+/823732 | 22:44 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!