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

Re: [Ceph-maintainers] Updated ceph backport



Hi

Alessio, I put you on CC because you are the leveldb maintainer and
backporter. The problem is that ceph currently is not built on mips and
mipsel because the leveldb backport is unavailable there.

Dmitry Smirnov <onlyjob@debian.org> writes:

> On Thu, 16 Oct 2014 10:50:24 Gaudenz Steinlin wrote:
>> As google-perftools are optional for ceph, the best fix for this is
>> probably to remove the dependency in the backport for the architectures
>> not supported in wheezy. This simple patch would probably do it (not
>> tested):
>> 
>> diff --git a/debian/control b/debian/control
>> index 96d98fa..f022982 100644
>> --- a/debian/control
>> +++ b/debian/control
>> @@ -26,7 +26,7 @@ Build-Depends: debhelper (>= 9~),
>>                 libexpat1-dev,
>>                 libfcgi-dev,
>>                 libfuse-dev,
>> -               libgoogle-perftools-dev [i386 amd64 powerpc armhf ppc64el],
>> +               libgoogle-perftools-dev [i386 amd64],
>>                 libkeyutils-dev,
>>                 libleveldb-dev,
>>                 libnss3-dev,
>> diff --git a/debian/rules b/debian/rules
>> index b3b0ccb..9941375 100755
>> --- a/debian/rules
>> +++ b/debian/rules
>> @@ -26,7 +26,7 @@ ifeq ($(DEB_HOST_ARCH), armel)
>>  endif
>> 
>>  # Only selected architectures support gperftools
>> -gperftools_archs = i386 amd64 powerpc armhf ppc64el
>> +gperftools_archs = i386 amd64
>>  ifneq (,$(filter $(DEB_HOST_ARCH), $(gperftools_archs)))
>>    extraopts += --with-tcmalloc
>>  else
>> 
>
> Noted, thank you. I'll consider applying this patch for next upload to 
> backports.
>
>
>> If there are no objections I can push this patch to the repository.
>
> If you insist but IMHO it would be best to apply it as part of next backport 
> which will be staged in "wheezy-backports" branch. Let's wait for 0.80.7-1 to 
> migrate to "testing" first, shall we?

I meant applying to the wheezy-backports branch in git. I have done that
now. Uploading to backports would be the next step. I'm not sure why you
really want to wait for 0.80.7 to reach testing. I would prefer an
upload soon so we can check if our fixes really make the package build
on more architectures.

The final goal of my push to have this in backports is that I would also
like to have the rbd backend in qemu activated in the backport. As
Michael pointed out he would like to have a list of supported
architectures for that.

The current patch leaves us with build problems for the backport on
s390, sparc, mips and mipsel. Mipsel built in the past, but latest leveldb does
not build there anymore. Of these only mips and mipsel are release
architectures for jessie. So I think we should just not support these
with the backport. I don't see the point of backporting to architectures
that won't be released anymore and I doubt there is a large enough
userbase for ceph or rbd on mips and mipsel to be worth the effort.
Or does the leveldb maintainer see a way to fix the build failures?

Michael would you be willing to enable the rbd backend on the qemu
backport for all architectures except mips and mipsel?

Gaudenz

Attachment: pgpTvWOWC3HYZ.pgp
Description: PGP signature


Reply to: