Project

General

Profile

Fix #44171

pybind/cephfs: audit for unimplemented bindings for libcephfs

Added by Patrick Donnelly 11 months ago. Updated about 10 hours ago.

Status:
Need More Info
Priority:
Normal
Category:
-
Target version:
% Done:

0%

Source:
Development
Tags:
Backport:
pacific,octopus,nautilus
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
cephfs.pyx
Labels (FS):
task(easy)
Pull request ID:
Crash signature:

Description

Recently we've added some missing bindings:

commit 5b33fefcde39724bb746f6af66e741012dbb2f0a HEAD
Author: Varsha Rao <varao@redhat.com>
Date:   Wed Jan 8 19:55:57 2020 +0530

    pybind: Add listxattr

    Signed-off-by: Varsha Rao <varao@redhat.com>
---
 src/pybind/cephfs/cephfs.pyx | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

or

https://github.com/ceph/ceph/pull/33120/files#diff-e236831d9ec122aa7331671da4adff33

Audit the python binding to make sure we have complete 1-1 mapping of python bindings to libcephfs functions in "src/include/cephfs/libcephfs.h".

History

#1 Updated by Patrick Donnelly 11 months ago

  • Assignee set to Kotresh Hiremath Ravishankar

#2 Updated by Kotresh Hiremath Ravishankar 11 months ago

The initial PR is sent. It doesn't cover all the missing bindings yet.

#3 Updated by Patrick Donnelly 11 months ago

  • Status changed from New to In Progress
  • Pull request ID set to 33406

#4 Updated by Kotresh Hiremath Ravishankar 9 months ago

  • Status changed from In Progress to Fix Under Review

#5 Updated by Patrick Donnelly 7 months ago

  • Status changed from Fix Under Review to Resolved

#6 Updated by Ramana Raja 6 months ago

Wouldn't backporting this to nautilus, and octopus be useful?

#7 Updated by Ramana Raja 6 months ago

  • Status changed from Resolved to Need More Info

#8 Updated by Patrick Donnelly 6 months ago

Ramana Raja wrote:

Wouldn't backporting this to nautilus, and octopus be useful?

I think only if a function is considered necessary for some other backport.

#9 Updated by Patrick Donnelly about 10 hours ago

  • Target version changed from v16.0.0 to v17.0.0
  • Backport set to pacific,octopus,nautilus

Also available in: Atom PDF