git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Shawn O. Pearce" <spearce@spearce.org>
To: "Olsen, Alan R" <alan.r.olsen@intel.com>
Cc: Sitaram Chamarty <sitaramc@gmail.com>,
	Matthieu Moy <Matthieu.Moy@grenoble-inp.fr>,
	"git@vger.kernel.org" <git@vger.kernel.org>
Subject: Re: Bugs in Gitosis
Date: Mon, 1 Nov 2010 10:32:02 -0700	[thread overview]
Message-ID: <20101101173202.GA22725@spearce.org> (raw)
In-Reply-To: <26E9B811E137AB4B95200FD4C950886BA96667EA@orsmsx507.amr.corp.intel.com>

"Olsen, Alan R" <alan.r.olsen@intel.com> wrote:
> 
> Gerrit does not add existing projects well. Pushing the kernel
> project into Gerrit causes one entry to approve per commit. That
> swamps the server.

Don't push to refs/for/master.  Grant yourself Push Branch +1
(or +2 if you need to create the branch) and push directly to
refs/heads/master like you would with Gitolite, Gitosis or any
other Git repository.  Gerrit won't create a change record, and
thus won't be swamped.

> Gerrit does not have a way of handling rebases
> very well.  (We have projects that have a regular consolidation on
> the end of the development trees.)

I think it handles rebases about as well as any other Git tool, you
need to enable Push +3 to permit force push/rewind of the relevant
branches, and then actually do the force push.  Any pending commit
will need to be rebased.  Which is also true for just about any
workflow except the classic Linux kernel "format-patch and email"
model.  Switching to gitolite probably won't easy the rebase pain.

FWIW, the Google kernel developers have their Gerrit instance
configured to use the cherry-pick submit type on their kernel
repositories, which makes changes submittable across rebases,
because its emulating the format-patch->email->am workflow that
is traditionally used for kernel development.

> There are also some people (me for example) who loath the Repo
> command and prefer to work using git.

I'm also among those people, as are many of Google's kernel
developers.  We just use git push to talk to Gerrit... and
that is one of the primary reasons it embeds its own SSHD.

-- 
Shawn.

  reply	other threads:[~2010-11-01 17:32 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-28 20:58 Bugs in Gitosis Olsen, Alan R
2010-10-28 21:42 ` Matthieu Moy
2010-10-28 22:22   ` Olsen, Alan R
2010-10-29  0:02     ` Sitaram Chamarty
2010-11-01 17:23       ` Olsen, Alan R
2010-11-01 17:32         ` Shawn O. Pearce [this message]
2010-10-28 23:54 ` Sitaram Chamarty
     [not found] <AANLkTinzLL-Sk1cMd_3WNogmgvr=x4gEh7LuS8guNYgB@mail.gmail.com>
2010-10-29 10:52 ` Sitaram Chamarty

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=20101101173202.GA22725@spearce.org \
    --to=spearce@spearce.org \
    --cc=Matthieu.Moy@grenoble-inp.fr \
    --cc=alan.r.olsen@intel.com \
    --cc=git@vger.kernel.org \
    --cc=sitaramc@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).