All of lore.kernel.org
 help / color / mirror / Atom feed
From: Victor Denisov <vdenisov@mirantis.com>
To: Jason Dillaman <dillaman@redhat.com>
Cc: ceph-devel <ceph-devel@vger.kernel.org>
Subject: Re: Cinder doesn't work with the current rados python bindings
Date: Tue, 13 Dec 2016 16:10:58 -0800	[thread overview]
Message-ID: <CA+hcxJTBWk0VXYBdL7zLxWJTU6VfrT8RWL7W3e53ycHYUwqXyQ@mail.gmail.com> (raw)
In-Reply-To: <CA+aFP1Bz_=HPVnJTJa29NJMmvzX7_gSA3EEB2n5VZLctBU4tKA@mail.gmail.com>

Yes. This is exactly the line I mean.
Is there any chance I can do it myself?
What kind of change is required?
I can unblock the cinder guy as well as contribute.
Let me know if I can be useful.

Thanks.
V.

On Tue, Dec 13, 2016 at 1:38 PM, Jason Dillaman <jdillama@redhat.com> wrote:
> Are you referring to rbd_default_features [1]? If so, that was a
> change with unintentional consequences in the master branch that I
> will need to fix before the final Kraken release.
>
> [1] https://github.com/openstack/cinder/blob/9fcb3bf1ad00913654872858d0cec839a6a2f0c8/cinder/volume/drivers/rbd.py#L152
>
> On Tue, Dec 13, 2016 at 3:29 PM, Victor Denisov <vdenisov@mirantis.com> wrote:
>> Hello,
>>
>> Who knows where pybind/rados/rados.pyx came from?
>> I talked to a guy in the cinder development team - he tried to build
>> his code against the latest python bindings from master.
>> It looks like the api's changed. Rados.conf_get('features') used to
>> return an integer.
>> It returns a string right now.
>>
>> I looked into the git history and I see that pybind.pyx has been where
>> it's right now only since February this year. I presume cinder's using
>> an older version.
>>
>> Can someone advise me on how I can track down the changes in python bindings.
>>
>> Thank you in advance.
>>
>> Victor.
>> --
>> 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
>
>
>
> --
> Jason

  reply	other threads:[~2016-12-14  0:11 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-13 20:29 Cinder doesn't work with the current rados python bindings Victor Denisov
2016-12-13 21:38 ` Jason Dillaman
2016-12-14  0:10   ` Victor Denisov [this message]
2016-12-14  0:24     ` Jason Dillaman
2016-12-14  0:26       ` Victor Denisov
2016-12-14  9:44       ` Mykola Golub
2016-12-14 13:39         ` Jason Dillaman
2016-12-14 14:12           ` Mykola Golub
2016-12-14 14:19             ` Jason Dillaman
2016-12-14  9:17 ` Alexey Sheplyakov

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=CA+hcxJTBWk0VXYBdL7zLxWJTU6VfrT8RWL7W3e53ycHYUwqXyQ@mail.gmail.com \
    --to=vdenisov@mirantis.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=dillaman@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.