Bug #6052
closed
ceph cli doesn't respect CEPH_ARGS
Added by Sage Weil over 10 years ago.
Updated over 10 years ago.
Description
Date: Mon, 19 Aug 2013 13:28:44 +0200
From: ?ystein L?nning Nerhus <nerhus@vx.no>
To: "ceph-users@lists.ceph.com" <ceph-users@lists.ceph.com>
Subject: [ceph-users] dumpling "ceph" cli tool breaks openstack cinder
Parts/Attachments:
1.1 OK ~25 lines Text (charset: ISO-8859-1)
1.2 Shown ~24 lines Text (charset: ISO-8859-1)
2 Shown 5 lines Text
----------------------------------------
[ The following text is in the "iso-8859-1" character set. ]
[ Your display is set for the "ANSI_X3.4-1968" character set. ]
[ Some characters may be displayed incorrectly. ]
Hi,
I just noticed that in dumpling the "ceph" cli tool no longer utilises the
"CEPH_ARGS" environment variable. This is used by openstack cinder to specifiy
the cephx user. Ref: http://ceph.com/docs/next/rbd/rbd-openstack/#configure-op>
ph [ceph.com]
I modifiied this line in /usr/share/pyshared/cinder/volume/driver.py
< stdout, _ = self._execute('ceph', 'fsid')
> stdout, _ = self._execute('ceph', '--id', 'volumes', 'fsid')
For my particular setup this seems to be sufficient as a quick workaround. Is
there a proper way to do this with the new tool?
Note: This only hit when i tried to create a volume from an image (i'm using
copy on write cloning). creating a fresh volume didnt invoke the "ceph fsid"
command in the openstack script, so i guess some openstack users will not be
affected.
Thanks,
Files
- Assignee set to Sage Weil
- Status changed from New to Fix Under Review
- Status changed from Fix Under Review to Resolved
Well, this commit didn't solve the issue for me. Running CEPH_ARGS="--id cinder" ceph -s
doesn't change the client id to cinder
, while ceph --id cinder -s
does.
The attached patch makes ceph
behave as expected.
Please ignore the first patch, it didn't work if CEPH_ARGS
wasn't set.
Benoît Knecht wrote:
Please ignore the first patch, it didn't work if CEPH_ARGS
wasn't set.
Ah, I see why. In this case we should probably revert my original fix too so that the options aren't parsed twice. Thanks!
Sage Weil wrote:
Ah, I see why. In this case we should probably revert my original fix too so that the options aren't parsed twice. Thanks!
Do you want me to resubmit the patch with the revert of your fix included? I can also submit a pull request on Github, if that's the preferred procedure.
Benoît Knecht wrote:
Sage Weil wrote:
Ah, I see why. In this case we should probably revert my original fix too so that the options aren't parsed twice. Thanks!
Do you want me to resubmit the patch with the revert of your fix included? I can also submit a pull request on Github, if that's the preferred procedure.
No need, already pushed to wip-ceph-cli-env. Care to test?
Thanks!
Sage Weil wrote:
Benoît Knecht wrote:
Do you want me to resubmit the patch with the revert of your fix included? I can also submit a pull request on Github, if that's the preferred procedure.
No need, already pushed to wip-ceph-cli-env.
I see, great!
Care to test?
It'll have to wait until tomorrow, but I will. I'll report back here; don't hesitate to ping me if somehow I forget...
Alright, I just tested it (the wip-ceph-cli-env
branch), and it works fine.
Thanks for having taken care of this so quickly.
Also available in: Atom
PDF