RE: ceph, XioMessenger issue

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



It's merged now.  In the future if you have a lingering PR like this 
please harass someone on IRC on ceph-devel.  No reason to be blocked for 4 
months!  :)

sage


On Thu, 3 Nov 2016, Avner Ben Hanoch wrote:

> HI Matt,
> 
> Thanks for your suggestion, we'll consider it.
> In general, we don't submit PRs in parallel because we want control on their acceptance order since many times one Xio PR is based on the other.
> 
> Regards,
> Avner
> 
> > -----Original Message-----
> > From: Matt Benjamin [mailto:mbenjamin@xxxxxxxxxx]
> > Sent: Thursday, November 03, 2016 14:50
> > Subject: Re: ceph, XioMessenger issue
> > 
> > Hi Avner,
> > 
> > Please make your work visible by creating PRs for your pipeline of commits.  If
> > many commits are of a similar type, such as this 1-line cleanup, please
> > consider batching them into a smaller number of PRs.
> > 
> > Regards,
> > 
> > Matt
> > 
> > ----- Original Message -----
> > > From: "Avner Ben Hanoch" <avnerb@xxxxxxxxxxxx>
> > > To: "Sage Weil" <sweil@xxxxxxxxxx>, "Marov Aleksey"
> > > <Marov.A@xxxxxxxxxx>
> > > Cc: ceph-devel@xxxxxxxxxxxxxxx
> > > Sent: Thursday, November 3, 2016 3:27:40 AM
> > > Subject: RE: ceph, XioMessenger issue
> > >
> > > We (in Mellanox) build with Xio on a regular basis.  We have a bunch
> > > of commits in the pipe that we are submitting 1 after the other.
> > > Currently, the pipe is blocked by this PR
> > > https://github.com/ceph/ceph/pull/9444
> > > After we complete releasing new version here we'll struggle with the
> > > needed merges
> > >
> > > > -----Original Message-----
> > > > From: ceph-devel-owner@xxxxxxxxxxxxxxx [mailto:ceph-devel-
> > > > owner@xxxxxxxxxxxxxxx] On Behalf Of Sage Weil
> > > > Sent: Wednesday, November 02, 2016 22:07
> > > > To: Marov Aleksey
> > > >
> > > >
> > > > It sounds like nobody else has hit this yet except you, so
> > > > apparently others also are building with[out] xio in their dev
> > > > environments.  :(
> > > --
> > > 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
> > >
> > 
> > --
> > Matt Benjamin
> > Red Hat, Inc.
> > 315 West Huron Street, Suite 140A
> > Ann Arbor, Michigan 48103
> > 
> > http://www.redhat.com/en/technologies/storage
> > 
> > tel.  734-707-0660
> > fax.  734-769-8938
> > cel.  734-216-5309
> 
--
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



[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux