Project

General

Profile

Actions

Bug #54237

closed

pybind/cephfs: Add mapping for Ernno 13:Permission Denied and adding path (in msg) while raising exception from opendir() in cephfs.pyx

Added by Sarthak Gupta about 2 years ago. Updated over 1 year ago.

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

100%

Source:
Development
Tags:
Backport:
quincy,pacific
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
cephfs.pyx
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

opendir() in src/pybind/cephfs/cephfs.pyx returns a generic cephfs.OSError: Errno 13 for this tracker issue https://tracker.ceph.com/issues/51611, which can be to tricky to catch if we want to provide a more verbose error to the user.

Hence adding a mapping for Errno13 (CEPHFS_EACCES) in errno_to_exception and raising a custom Exception like PermissionDenied(OSError) can be useful in exception handling.


Related issues 2 (0 open2 closed)

Copied to CephFS - Backport #54577: pacific: pybind/cephfs: Add mapping for Ernno 13:Permission Denied and adding path (in msg) while raising exception from opendir() in cephfs.pyxResolvedSarthak GuptaActions
Copied to CephFS - Backport #54578: quincy: pybind/cephfs: Add mapping for Ernno 13:Permission Denied and adding path (in msg) while raising exception from opendir() in cephfs.pyxResolvedSarthak GuptaActions
Actions #1

Updated by Sarthak Gupta about 2 years ago

  • Subject changed from mgr/dashboard: Add mapping for Ernno 13:Permission denied (CEPHFS_EACCES) in errno_to_exception in cephfs.pyx to mgr/dashboard: Add mapping for Ernno 13:Permission Denied and adding path (in msg) while raising exception from opendir() in cephfs.pyx
Actions #2

Updated by Patrick Donnelly about 2 years ago

  • Project changed from Dashboard to CephFS
  • Subject changed from mgr/dashboard: Add mapping for Ernno 13:Permission Denied and adding path (in msg) while raising exception from opendir() in cephfs.pyx to pybind/cephfs: Add mapping for Ernno 13:Permission Denied and adding path (in msg) while raising exception from opendir() in cephfs.pyx
  • Assignee set to Sarthak Gupta
  • Target version set to v18.0.0
  • Source set to Development
  • Backport set to quincy,pacific
  • Component(FS) cephfs.pyx added
Actions #3

Updated by Ernesto Puerta about 2 years ago

  • Pull request ID set to 45104
Actions #4

Updated by Ernesto Puerta about 2 years ago

  • Status changed from New to Fix Under Review
Actions #5

Updated by Venky Shankar about 2 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions #6

Updated by Backport Bot about 2 years ago

  • Copied to Backport #54577: pacific: pybind/cephfs: Add mapping for Ernno 13:Permission Denied and adding path (in msg) while raising exception from opendir() in cephfs.pyx added
Actions #7

Updated by Backport Bot about 2 years ago

  • Copied to Backport #54578: quincy: pybind/cephfs: Add mapping for Ernno 13:Permission Denied and adding path (in msg) while raising exception from opendir() in cephfs.pyx added
Actions #8

Updated by Backport Bot over 1 year ago

  • Tags set to backport_processed
Actions #9

Updated by Konstantin Shalygin over 1 year ago

  • Tracker changed from Feature to Bug
  • Status changed from Pending Backport to Resolved
  • % Done changed from 0 to 100
  • Tags deleted (backport_processed)
  • Regression set to No
  • Severity set to 3 - minor
Actions

Also available in: Atom PDF