Project

General

Profile

Feature #17230

ceph_volume_client: py3 compatible

Added by Ramana Raja almost 3 years ago. Updated 6 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
07/11/2018
Due date:
% 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 fs - Backport #26850: mimic: ceph_volume_client: py3 compatible Resolved
Copied to fs - Backport #26851: luminous: ceph_volume_client: py3 compatible Resolved

History

#1 Updated by Ramana Raja almost 3 years ago

  • Assignee deleted (Ramana Raja)

#2 Updated by Ramana Raja almost 3 years ago

  • Assignee set to Ramana Raja

#3 Updated by Ramana Raja almost 3 years ago

  • Assignee deleted (Ramana Raja)

#4 Updated by Patrick Donnelly over 1 year 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 over 1 year ago

  • Status changed from New to In Progress

#6 Updated by Rishabh Dave over 1 year ago

  • Status changed from In Progress to Need Review

#7 Updated by Patrick Donnelly over 1 year 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 about 1 year 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 about 1 year 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 about 1 year ago

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

#17 Updated by Patrick Donnelly about 1 year 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 about 1 year ago

  • Status changed from Need Review to Pending Backport

#19 Updated by Patrick Donnelly about 1 year ago

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

#20 Updated by Patrick Donnelly about 1 year ago

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

#21 Updated by Nathan Cutler 10 months ago

  • Status changed from Pending Backport to Resolved

#22 Updated by Patrick Donnelly 6 months ago

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

Also available in: Atom PDF