Re: Master needs newer version of boost on centos?

["Matt W. Benjamin" <matt@linuxbox.com>]

Hi,

I think the offset,length change is probably not correct, since we need
to span the range actually being written.

Also we think that overlapping intervals are possible.

Matt

----- "Sage Weil" <sage@inktank.com> wrote:

> On Mon, 17 Feb 2014, Matt W. Benjamin wrote:
> > Hi,
> > 
> > I don't know, that's an interesting catch.  I guess it's not really
> feasible to do
> > anything accept ack this, for now.
> 
> Probably given that we need to be able to build, but really it comes 
> down to one questions and a sanity check:
> 
> - Does the (start,end) -> (start, length) fix look right?  AFAICS it's
> 
> coming straight from the caller, so it should only break something if
> the 
> external caller is also broken.
> 
> - Is span supposed to contain overlapping intervals?  If so, then our
> 
> internal type is not sufficient.
> 
> sage
> 
> 
> 
> > 
> > Matt
> > 
> > ----- "Sage Weil" <sage@inktank.com> wrote:
> > 
> > > On Mon, 17 Feb 2014, Matt W. Benjamin wrote:
> > > > Hi Sage,
> > > > 
> > > > I originally used this, and it was replaced with the Boost
> version
> > > as part
> > > > of a bugfix.
> > > 
> > > I wonder if the bug was our internal class asserting because of an
> 
> > > overlapping interval due to the (start, end) thing?
> > > 
> > > s
> > > > 
> > > > Matt
> > > > 
> > > > ----- "Sage Weil" <sage@inktank.com> wrote:
> > > > 
> > > > > Hi Matt,
> > > > > 
> > > > > I pushed wip-interval-set, which replaces the boost
> interval_set
> > > with
> > > > > our 
> > > > > internal type.  As far as I can see it is only being used for
> 
> > > > > intersects(), which we support.  As long as the interval_set
> isn't
> > > 
> > > > > supposed to handle overlapping sets (ours doesn't) this should
> be
> > > > > fine.
> > > > > 
> > > > > I did notice one discrepancy where one user was passing in
> (start,
> > > 
> > > > > length) and the other was using (start, end).  I switched it
> to
> > > > > (start, 
> > > > > offset) (since that's what our class does) but didn't verify
> the
> > > > > semantics 
> > > > > were what was originally intended... can you take a look?
> > > > > 
> > > > > 	https://github.com/ceph/ceph/pull/1253
> > > > > 
> > > > > Thanks!
> > > > > sage
> > > > > 
> > > > > 
> > > > > On Fri, 14 Feb 2014, Matt W. Benjamin wrote:
> > > > > 
> > > > > > yarg, sorry
> > > > > > 
> > > > > > ----- "Josh Durgin" <josh.durgin@inktank.com> wrote:
> > > > > > 
> > > > > > > ICL was added in 1.46 according to
> > > > > http://www.boost.org/doc/libs/.
> > > > > > > This also fails on debian wheezy with only 1.42
> available:
> > > > > > > 
> > > > > > > http://tracker.ceph.com/issues/7422
> > > > > > > 
> > > > > > > On 02/14/2014 04:26 PM, Matt W. Benjamin wrote:
> > > > > > > > according to google, 1.41 ahould have it.
> > > > > > > >
> > > > > > > > ----- "Sage Weil" <sage@inktank.com> wrote:
> > > > > > > >
> > > > > > > >> On Fri, 14 Feb 2014, Matt W. Benjamin wrote:
> > > > > > > >>> but we haven't built on centos, to my knowledge...
> > > > > > > >>
> > > > > > > >> I think the question is really: is there an
> interval_set
> > > in
> > > > > boost
> > > > > > > 1.41
> > > > > > > >>
> > > > > > > >> that you can use, can you use something else instead,
> or do
> > > we
> > > > > need
> > > > > > > to
> > > > > > > >> do
> > > > > > > >> a boost backport in order to build for centos/rhel
> 6.x?
> > > > > > > >>
> > > > > > > >> sage
> > > > > > > >>
> > > > > > > >>>
> > > > > > > >>> ----- "Matt W. Benjamin" <matt@linuxbox.com> wrote:
> > > > > > > >>>
> > > > > > > >>>> oh, ok.  I think they need an extra boost rpm?
> > > > > > > >>>>
> > > > > > > >>>> ----- "Sage Weil" <sage@inktank.com> wrote:
> > > > > > > >>>>
> > > > > > > >>>>> Hey Matt,
> > > > > > > >>>>>
> > > > > > > >>>>> This is in the libcephfs code I merged yesterday. 
> Any
> > > > > ideas?
> > > > > > > >>>>>
> > > > > > > >>>>> Thanks!
> > > > > > > >>>>> sage
> > > > > > > >>>>>
> > > > > > > >>>>>
> > > > > > > >>>>> On Fri, 14 Feb 2014, Sandon Van Ness wrote:
> > > > > > > >>>>>
> > > > > > > >>>>>>
> > > > > > > >>>>>> The last couple builds of master have been failing
> on
> > > the
> > > > > > > >> centos
> > > > > > > >>>>> gitbuilder.
> > > > > > > >>>>>> here is the build log:
> > > > > > > >>>>>>
> > > > > > > >>>>>>
> > > > > > > >>>>>
> > > > > > > >>>>
> > > > > > > >>
> > > > > > >
> > > > >
> > >
> http://gitbuilder.sepia.ceph.com/gitbuilder-centos6-amd64/log.cgi?log=f47062d8a66b94743888aaea13e0766579379395
> > > > > > > >>>>>>
> > > > > > > >>>>>> Its running into:
> > > > > > > >>>>>>
> > > > > > > >>>>>> CXX mds/Capability.lo
> > > > > > > >>>>>> CXX mds/Dumper.lo
> > > > > > > >>>>>> In file included from client/Client.h:38,
> > > > > > > >>>>>> from libcephfs.cc:21:
> > > > > > > >>>>>>
> > > > > > > >>>>>>       error: client/barrier.h:25:38:
> > > > > > > >> boost/icl/interval_set.hpp:
> > > > > > > >>>> No
> > > > > > > >>>>> such
> > > > > > > >>>>>>       file or directory
> > > > > > > >>>>>>
> > > > > > > >>>>>> It looks like its not effecting fedora gitbuilders
> > > that
> > > > > have
> > > > > > > >> a
> > > > > > > >>>> newer
> > > > > > > >>>>> version
> > > > > > > >>>>>> of boost but on centos/rhel the version is only
> > > 1.41.0-18.
> > > > > Is
> > > > > > > >>>> there
> > > > > > > >>>>> something
> > > > > > > >>>>>> we can do to get around this without back-porting
> or
> > > will
> > > > > > > >>>>> backporting a newer
> > > > > > > >>>>>> boost for rhel/centos be necessary?
> > > > > > > >>>>>> --
> > > > > > > >>>>>> To unsubscribe from this list: send the line
> > > "unsubscribe
> > > > > > > >>>>> ceph-devel" in
> > > > > > > >>>>>> the body of a message to majordomo@vger.kernel.org
> > > > > > > >>>>>> More majordomo info at
> > > > > > > >>>> http://vger.kernel.org/majordomo-info.html
> > > > > > > >>>>>>
> > > > > > > >>>>>>
> > > > > > > >>>>
> > > > > > > >>>> --
> > > > > > > >>>> Matt Benjamin
> > > > > > > >>>> The Linux Box
> > > > > > > >>>> 206 South Fifth Ave. Suite 150
> > > > > > > >>>> Ann Arbor, MI  48104
> > > > > > > >>>>
> > > > > > > >>>> http://linuxbox.com
> > > > > > > >>>>
> > > > > > > >>>> tel.  734-761-4689
> > > > > > > >>>> 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@vger.kernel.org
> > > > > > > >>>> More majordomo info at
> > > > > > > >> http://vger.kernel.org/majordomo-info.html
> > > > > > > >>>
> > > > > > > >>> --
> > > > > > > >>> Matt Benjamin
> > > > > > > >>> The Linux Box
> > > > > > > >>> 206 South Fifth Ave. Suite 150
> > > > > > > >>> Ann Arbor, MI  48104
> > > > > > > >>>
> > > > > > > >>> http://linuxbox.com
> > > > > > > >>>
> > > > > > > >>> tel.  734-761-4689
> > > > > > > >>> 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@vger.kernel.org
> > > > > > > More majordomo info at 
> > > > > http://vger.kernel.org/majordomo-info.html
> > > > > > 
> > > > > > -- 
> > > > > > Matt Benjamin
> > > > > > The Linux Box
> > > > > > 206 South Fifth Ave. Suite 150
> > > > > > Ann Arbor, MI  48104
> > > > > > 
> > > > > > http://linuxbox.com
> > > > > > 
> > > > > > tel.  734-761-4689 
> > > > > > fax.  734-769-8938 
> > > > > > cel.  734-216-5309 
> > > > > > 
> > > > > >
> > > > 
> > > > -- 
> > > > Matt Benjamin
> > > > The Linux Box
> > > > 206 South Fifth Ave. Suite 150
> > > > Ann Arbor, MI  48104
> > > > 
> > > > http://linuxbox.com
> > > > 
> > > > tel.  734-761-4689 
> > > > fax.  734-769-8938 
> > > > cel.  734-216-5309 
> > > > 
> > > >
> > 
> > -- 
> > Matt Benjamin
> > The Linux Box
> > 206 South Fifth Ave. Suite 150
> > Ann Arbor, MI  48104
> > 
> > http://linuxbox.com
> > 
> > tel.  734-761-4689 
> > 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@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > 
> >

-- 
Matt Benjamin
The Linux Box
206 South Fifth Ave. Suite 150
Ann Arbor, MI  48104

http://linuxbox.com

tel.  734-761-4689 
fax.  734-769-8938 
cel.  734-216-5309