Project

General

Profile

Bug #54237

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 10 months ago. Updated about 2 months 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

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.pyx Resolved
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.pyx Resolved

History

#1 Updated by Sarthak Gupta 10 months 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

#2 Updated by Patrick Donnelly 10 months 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

#3 Updated by Ernesto Puerta 9 months ago

  • Pull request ID set to 45104

#4 Updated by Ernesto Puerta 9 months ago

  • Status changed from New to Fix Under Review

#5 Updated by Venky Shankar 9 months ago

  • Status changed from Fix Under Review to Pending Backport

#6 Updated by Backport Bot 9 months 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

#7 Updated by Backport Bot 9 months 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

#8 Updated by Backport Bot 4 months ago

  • Tags set to backport_processed

#9 Updated by Konstantin Shalygin about 2 months 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

Also available in: Atom PDF