Project

General

Profile

Feature #17230

ceph_volume_client: py3 compatible

Added by Ramana Raja over 2 years ago. Updated about 1 month ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Manila
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):
task(intern)
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?) New 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 over 2 years ago

  • Assignee deleted (Ramana Raja)

#2 Updated by Ramana Raja about 2 years ago

  • Assignee set to Ramana Raja

#3 Updated by Ramana Raja about 2 years ago

  • Assignee deleted (Ramana Raja)

#4 Updated by Patrick Donnelly 7 months 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 7 months ago

  • Status changed from New to In Progress

#6 Updated by Rishabh Dave 7 months ago

  • Status changed from In Progress to Need Review

#7 Updated by Patrick Donnelly 7 months 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 6 months 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 6 months 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 5 months ago

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

#17 Updated by Patrick Donnelly 5 months 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 4 months ago

  • Status changed from Need Review to Pending Backport

#19 Updated by Patrick Donnelly 4 months ago

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

#20 Updated by Patrick Donnelly 4 months ago

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

#21 Updated by Nathan Cutler about 1 month ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF