Project

General

Profile

Actions
Copy link

Fix #44171

open

pybind/cephfs: audit for unimplemented bindings for libcephfs

Added by Patrick Donnelly about 4 years ago. Updated almost 2 years ago.

Status:
Need More Info
Priority:
Normal
Assignee:
Kotresh Hiremath Ravishankar
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:
33406
Crash signature (v1):
Crash signature (v2):

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".

Actions
Copy link
 #1

Updated by Patrick Donnelly about 4 years ago

  • Assignee set to Kotresh Hiremath Ravishankar
Actions
Copy link
 #2

Updated by Kotresh Hiremath Ravishankar about 4 years ago

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

Actions
Copy link
 #3

Updated by Patrick Donnelly about 4 years ago

  • Status changed from New to In Progress
  • Pull request ID set to 33406
Actions
Copy link
 #4

Updated by Kotresh Hiremath Ravishankar about 4 years ago

  • Status changed from In Progress to Fix Under Review
Actions
Copy link
 #5

Updated by Patrick Donnelly almost 4 years ago

  • Status changed from Fix Under Review to Resolved
Actions
Copy link
 #6

Updated by Ramana Raja almost 4 years ago

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

Actions
Copy link
 #7

Updated by Ramana Raja almost 4 years ago

  • Status changed from Resolved to Need More Info
Actions
Copy link
 #8

Updated by Patrick Donnelly almost 4 years 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.

Actions
Copy link
 #9

Updated by Patrick Donnelly over 3 years ago

  • Target version changed from v16.0.0 to v17.0.0
  • Backport set to pacific,octopus,nautilus
Actions
Copy link
 #10

Updated by Patrick Donnelly almost 2 years ago

  • Target version deleted (v17.0.0)
Actions
Copy link

Also available in: Atom PDF