Project

General

Profile

Bug #46496

pybind/mgr/volumes: subvolume operations throw exception if volume doesn't exist

Added by Kotresh Hiremath Ravishankar over 3 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Category:
-
Target version:
% Done:

0%

Source:
Development
Tags:
Backport:
octopus,nautilus
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
mgr/volumes
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Most of the volume/subvolume operations throw exception if the volume doesn't exist.

e.g.

$ bin/ceph fs subvolume ls non-existing-volume
  • DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
    2020-07-13T12:31:51.061+0530 7fee71f28700 -1 WARNING: all dangerous and experimental features are enabled.
    2020-07-13T12:31:51.077+0530 7fee71f28700 -1 WARNING: all dangerous and experimental features are enabled.
    Error EINVAL: Traceback (most recent call last):
    File "/root/sandbox/kotresh-ceph/ceph/src/pybind/mgr/mgr_module.py", line 1171, in _handle_command
    return self.handle_command(inbuf, cmd)
    File "/root/sandbox/kotresh-ceph/ceph/src/pybind/mgr/volumes/module.py", line 330, in handle_command
    return handler(inbuf, cmd)
    File "/root/sandbox/kotresh-ceph/ceph/src/pybind/mgr/volumes/module.py", line 390, in _cmd_fs_subvolume_ls
    group_name=cmd.get('group_name', None))
    File "/root/sandbox/kotresh-ceph/ceph/src/pybind/mgr/volumes/fs/volume.py", line 272, in list_subvolumes
    with open_volume(self, volname) as fs_handle:
    File "/usr/lib/python3.6/contextlib.py", line 81, in enter
    return next(self.gen)
    File "/root/sandbox/kotresh-ceph/ceph/src/pybind/mgr/volumes/fs/operations/volume.py", line 133, in open_volume
    with open_filesystem(vc, volname) as fs_handle:
    File "/usr/lib/python3.6/contextlib.py", line 81, in enter
    return next(self.gen)
    File "/root/sandbox/kotresh-ceph/ceph/src/pybind/mgr/mgr_util.py", line 260, in open_filesystem
    fs_handle = fsc.connection_pool.get_fs_handle(fs_name)
    File "/root/sandbox/kotresh-ceph/ceph/src/pybind/mgr/mgr_util.py", line 192, in get_fs_handle
    conn = CephfsConnectionPool.Connection(self.mgr, fs_name)
    File "/root/sandbox/kotresh-ceph/ceph/src/pybind/mgr/mgr_util.py", line 62, in init
    self.fs_id = self.get_fs_id()
    File "/root/sandbox/kotresh-ceph/ceph/src/pybind/mgr/mgr_util.py", line 70, in get_fs_id
    -errno.ENOENT, "FS '{0}' not found".format(self.fs_name))
    mgr_util.CephfsConnectionException: -2 (FS 'a' not found)

Related issues

Copied to CephFS - Backport #46792: octopus: pybind/mgr/volumes: subvolume operations throw exception if volume doesn't exist Rejected
Copied to CephFS - Backport #46793: nautilus: pybind/mgr/volumes: subvolume operations throw exception if volume doesn't exist Rejected

History

#1 Updated by Kotresh Hiremath Ravishankar over 3 years ago

  • Status changed from New to In Progress
  • Assignee set to Kotresh Hiremath Ravishankar

#2 Updated by Kotresh Hiremath Ravishankar over 3 years ago

  • Subject changed from pybind/mgr/volumes: Volume/subvolume operations throw exception if volume doesn't exist to pybind/mgr/volumes: Subvolume operations throw exception if volume doesn't exist

#3 Updated by Kotresh Hiremath Ravishankar over 3 years ago

  • Pull request ID set to 36064

#4 Updated by Kotresh Hiremath Ravishankar over 3 years ago

  • Status changed from In Progress to Fix Under Review

#5 Updated by Patrick Donnelly over 3 years ago

  • Subject changed from pybind/mgr/volumes: Subvolume operations throw exception if volume doesn't exist to pybind/mgr/volumes: subvolume operations throw exception if volume doesn't exist
  • Target version set to v16.0.0
  • Source set to Development
  • Backport set to octopus,nautilus

#6 Updated by Patrick Donnelly over 3 years ago

  • Status changed from Fix Under Review to Pending Backport

#7 Updated by Nathan Cutler over 3 years ago

  • Copied to Backport #46792: octopus: pybind/mgr/volumes: subvolume operations throw exception if volume doesn't exist added

#8 Updated by Nathan Cutler over 3 years ago

  • Copied to Backport #46793: nautilus: pybind/mgr/volumes: subvolume operations throw exception if volume doesn't exist added

#9 Updated by Nathan Cutler over 3 years ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF