On Mon, 17 Oct 2016, Alfredo Deza wrote: > On Fri, Oct 14, 2016 at 3:53 PM, Gregory Farnum <gfarnum@xxxxxxxxxx> wrote: > > +1 this idea > > > > On Fri, Oct 14, 2016 at 12:03 PM, Sage Weil <sage@xxxxxxxxxxxx> wrote: > >> That seems okay for now. I think this might have gotten lost in the > >> earlier thread, though: > >> > >> Why don't we take this opportunity to create a ceph-ci.git and lock down > >> ceph.git? > >> > >> - ceph.git would contain only official branches (master, jewel, etc.) > >> - ceph-ci.git would have all the branches that need to get build for > >> ci purposes. > >> - ci would build everything in ceph.git and ceph-ci.git that doesn't have > >> a / in it > >> - if you don't want a branch built, don't push it to either one; push it > >> to your private clone (which is what you should be doing anyway!) > > But nobody does and it is not enforceable :( We can restrict access to ceph.git, which will force people to change their habits... and hope that they do so for the better. > >> - restrict access to ceph.git to core developers (and trusted robots) > >> - grant access to ceph-ci to a broader set of developers (and robots) > > > > Except for this. Until/unless we find something requiring ceph.git > > access, we should lock down push access and only let code go in via > > Github PRs. Give ceph-ci access to everybody who's allowed in to > > sepia. Why open any gates preemptively if we don't need to? :) > > This sounds very similar to the problem we have today. Building > everything in ceph.git and ceph-ci.git would > get us right where we are right now (although not from the start). > > Not that we can't adapt these rules and settings as we move forward though. Yeah. One other thought I had: we can move all the current extraneous branches in ceph.git to ceph-archive.git or something so that ceph-ci.git starts out fresh and clean... sage -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
