Project

General

Profile

Feature #17230

ceph_volume_client: py3 compatible

Added by Ramana Raja over 7 years ago. Updated about 5 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
% Done:

100%

Source:
Development
Tags:
Backport:
mimic,luminous
Reviewed:
Affected Versions:
Component(FS):
VolumeClient
Labels (FS):
Manila
Pull request ID:

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.


Subtasks

Bug #24870: ceph-debug-docker: python3 libraries not installed in docker imageResolvedPatrick Donnelly


Related issues

Related to devops - Bug #24920: teuthology is not installing python3-cephfs/python3-rados/etc. (Ubuntu) or python34-cephfs (CentOS) or (what else?) Resolved 07/13/2018
Copied to CephFS - Backport #26850: mimic: ceph_volume_client: py3 compatible Resolved
Copied to CephFS - Backport #26851: luminous: ceph_volume_client: py3 compatible Resolved

History

#1 Updated by Ramana Raja over 7 years ago

  • Assignee deleted (Ramana Raja)

#2 Updated by Ramana Raja over 7 years ago

  • Assignee set to Ramana Raja

#3 Updated by Ramana Raja over 7 years ago

  • Assignee deleted (Ramana Raja)

#4 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

#5 Updated by Patrick Donnelly almost 6 years ago

  • Status changed from New to In Progress

#6 Updated by Rishabh Dave almost 6 years ago

  • Status changed from In Progress to Fix Under Review

#7 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

#14 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?

#15 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.

#16 Updated by Patrick Donnelly over 5 years ago

Note: I've deleted the backport issues as this hasn't been merged to master yet.

#17 Updated by Patrick Donnelly over 5 years ago

  • Related to Bug #24920: teuthology is not installing python3-cephfs/python3-rados/etc. (Ubuntu) or python34-cephfs (CentOS) or (what else?) added

#18 Updated by Patrick Donnelly over 5 years ago

  • Status changed from Fix Under Review to Pending Backport

#19 Updated by Patrick Donnelly over 5 years ago

  • Copied to Backport #26850: mimic: ceph_volume_client: py3 compatible added

#20 Updated by Patrick Donnelly over 5 years ago

  • Copied to Backport #26851: luminous: ceph_volume_client: py3 compatible added

#21 Updated by Nathan Cutler over 5 years ago

  • Status changed from Pending Backport to Resolved

#22 Updated by Patrick Donnelly about 5 years ago

  • Category deleted (87)
  • Labels (FS) Manila added
  • Labels (FS) deleted (task(intern))

Also available in: Atom PDF