Re: Widening reach for ... code?
Thanh ha <zxiiro@...>
See inline. (Include's Robert's fixes) On Mon, Aug 24, 2020 at 4:06 PM Robert Varga <nite@...> wrote: Hello, Either way would require modifying JJB to support the new SCM system GitLab and the trigger mechanism would be quite a bit different than what we have with Gerrit (PRs vs Gerrit Trigger) so this wouldn't be a simple change of backends. Concern here for me is who would lead this effort. I think we'd need LF support considering the community does not have the available cycles to do this. 2) Patch review process. Let's not kid ourselves, GitLab and GitHub The cherry-pick option in GitHub (likely GitLab too but I'm not familiar with it) is CLI with a command like `git cherry-pick master` or similar. Personally I like this method but that's because I'm in the weeds of Git often, I can see the value in an easy button for this like Gerrit has. Sadly, the GitHub / GitLab model is via PRs (basically branches) which are not equivalent to patch reviews like Gerrit. So a change of system would require all the committers to unlearn Gerrit. 3) The straight-up choice of GitLab -- I think this is something the I think this would be better received if the LF did the background work to make this work for the community, however I feel LF is no longer actively engaged in LFN project's day to day development cycle. Meaning LF is going to drop GitLab on us and tell us to do the legwork to make all of our projects work with that. This kind of transition would require a significant amount of work for the community to take on to transition the CI bits and adjust our workflows. Considering there's a lack of people to lead this kind of work... I worry that this is an exercise in frustration. With that out of the way, I would like to make a counter-proposal: This is possible as I've used GerritHub (http://gerrithub.io/) before and it worked (although over 8 years ago when I tried it), it was quite clunky but maybe it's better now. I'm not sure if GerritHub is proprietary or if there's available a Gerrit plugin for this feature. Not sure how hard it would be to replicate this feature but at least it seems to be possible. Regards, Thanh I mean we already have a read-only mirror on GitHub for each and every |
|