All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mykola Golub <mgolub@mirantis.com>
To: dillaman@redhat.com
Cc: Victor Denisov <vdenisov@mirantis.com>,
	ceph-devel <ceph-devel@vger.kernel.org>
Subject: Re: Cinder doesn't work with the current rados python bindings
Date: Wed, 14 Dec 2016 16:12:52 +0200	[thread overview]
Message-ID: <20161214141251.GB8416@gmail.com> (raw)
In-Reply-To: <CA+aFP1DesvY2=+0yVyNwpAcLZTq44QEFKgyXc5vHvWpi-i9wFw@mail.gmail.com>

On Wed, Dec 14, 2016 at 08:39:03AM -0500, Jason Dillaman wrote:
> In the short and medium term, there isn't any way to get multiple
> versions of OpenStack to stop directly parsing librbd configuration
> options and switch to new API methods. The only reason they are
> overriding the default image features is to ensure layering support is
> enabled. In the C/C++ APIs, we have a clean way [1] to add individual
> features while keeping the default feature set, but this isn't
> currently exposed in the Python API.
> 
> For the Kraken release, I think the only two real options are to (1)
> add internal support for a config validator that takes the human
> friendly, comma delimited feature names and automatically converts
> them to the integer that has previously been expected, or (2) revert
> the change and go back to using a non-user friendly bitmask.

(3) Hardcode rbd_default_features in common/config_opts.h to its
numerical value (currently it should work both way). It would make
cinder (and other users who parse rbd_default_features) happy until it
is explicitly set in the config using a new way (as a list of feature
names). Document this restriction -- cinder users who want to change
rbd_default_features in ceph.conf would need to continue to use a
numeric value.

This may be used as workaround on currently affected installations.
Victor, could you please try to set rbd_default_features in ceph.conf
to its numeric value (61) and see if it fixes the issue?

-- 
Mykola Golub

  reply	other threads:[~2016-12-14 14:13 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
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 [this message]
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=20161214141251.GB8416@gmail.com \
    --to=mgolub@mirantis.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=dillaman@redhat.com \
    --cc=vdenisov@mirantis.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.