From: Joao Eduardo Luis <joao@suse.de>
To: Sage Weil <sweil@redhat.com>, ceph-devel@vger.kernel.org
Subject: Re: ceph-ci.git?
Date: Sat, 23 May 2015 10:34:49 +0100 [thread overview]
Message-ID: <556049B9.9080906@suse.de> (raw)
In-Reply-To: <alpine.DEB.2.00.1505172130440.8350@cobra.newdream.net>
On 18/05/15 05:40, Sage Weil wrote:
> As the number of people contributing code grows, we've added more and more
> people to the github ceph org who have write access to ceph.git. Those
> people can merge pull requests and can also push branches directly to the
> repo.
>
> We also use ceph.git as a source for the test build infrastrucure
> (gitbuilders) to generate packages for QA or hot fixes and make check
> tests. This leads to an every-growing body of wip-* branches in the repo
> (which is annoying), and also means that in order to build something to
> test in QA you also get the ability to (say) push directly to master.
>
> How about we instead
>
> - create a second repo named something like ceph-ci.git (that's the best
> I can come up with at the moment)
> - add this as a second source for all gitbuilders (they can poll a list)
> - move all wip-* branches here
> - create a new github team with contributing developers who can push to
> this repo and are trusted not to wreak havoc on the builders
> - remove all the cruft from ceph.git, so that it's just master, next, the
> stable branches, release tags, and anything else similarly important.
> - restrict ceph.git write access to core developers
>
> This will improve security somewhat and reduce the risk of an accidental
> push to an important branch.
>
> It may also reduce the risk associated with accidental force pushes
> (something we've hemmed and hawed about recently) by limiting the circle
> of people who can write to ceph.git and also changing workflows so that it
> is almost never used directly...
I think this is a great idea!
'ceph-ci' also looks like a simple enough name conveying its purpose,
even though the 'continuous' part is not necessarily true (but it's
trivial to type, so it gets my vote).
-Joao
next prev parent reply other threads:[~2015-05-23 9:34 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-18 4:40 ceph-ci.git? Sage Weil
2015-05-18 8:09 ` ceph-ci.git? Loic Dachary
2015-05-18 14:02 ` ceph-ci.git? Justin Erenkrantz
2015-05-18 16:29 ` ceph-ci.git? John Spray
2015-05-18 17:10 ` ceph-ci.git? Loic Dachary
2015-05-23 9:34 ` Joao Eduardo Luis [this message]
2015-05-24 15:49 ` ceph-ci.git? Sage Weil
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=556049B9.9080906@suse.de \
--to=joao@suse.de \
--cc=ceph-devel@vger.kernel.org \
--cc=sweil@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.