Feature #17230
closedceph_volume_client: py3 compatible
100%
Description
Manila drivers and their CIs are encouraged to be py3 compatible. Manila's cephfs_native driver uses ceph_volume_client library. It would be useful to make the library py3 compatible especially since there's been active work going on in making Ceph py3 compatible [1]. The tests and testing framework of ceph_volume_client are also being made py3 compatible [2]3. So it might be easy to test the py3 compatibility of the volume client.
[1] http://www.spinics.net/lists/ceph-devel/msg32349.html
[2] https://github.com/ceph/teuthology/pull/931
[3] https://github.com/ceph/ceph-qa-suite/pull/1129
On quick glance, it seems like ceph_volume_client needs to ensure that the buffer or the xattr value it writes or sets respectively using cython cephfs bindings needs to be byte type. There could be other changes needed.
Updated by Patrick Donnelly almost 6 years ago
- Assignee set to Rishabh Dave
- Target version set to v13.0.0
- Source changed from other to Development
- Labels (FS) task(intern) added
Updated by Patrick Donnelly almost 6 years ago
- Status changed from New to In Progress
Updated by Rishabh Dave almost 6 years ago
- Status changed from In Progress to Fix Under Review
Updated by Patrick Donnelly almost 6 years ago
- Priority changed from Normal to Urgent
- Target version changed from v13.0.0 to v14.0.0
- Backport set to mimic,luminous
Updated by Nathan Cutler almost 6 years ago
Question for all, and particularly for Patrick: why is a luminous backport of this needed, and is it worth the risk?
Updated by Patrick Donnelly almost 6 years ago
There are high-level pushes to py3 in future (minor-)releases of Ceph/Openstack. RHCS 3.X (Luminous) will need to be py3 compatible.
Updated by Patrick Donnelly almost 6 years ago
Note: I've deleted the backport issues as this hasn't been merged to master yet.
Updated by Patrick Donnelly almost 6 years ago
- Related to Bug #24920: teuthology is not installing python3-cephfs/python3-rados/etc. (Ubuntu) or python34-cephfs (CentOS) or (what else?) added
Updated by Patrick Donnelly over 5 years ago
- Status changed from Fix Under Review to Pending Backport
Updated by Patrick Donnelly over 5 years ago
- Copied to Backport #26850: mimic: ceph_volume_client: py3 compatible added
Updated by Patrick Donnelly over 5 years ago
- Copied to Backport #26851: luminous: ceph_volume_client: py3 compatible added
Updated by Nathan Cutler over 5 years ago
- Status changed from Pending Backport to Resolved
Updated by Patrick Donnelly about 5 years ago
- Category deleted (
87) - Labels (FS) Manila added
- Labels (FS) deleted (
task(intern))