Project

General

Profile

Actions

Bug #55762

open

mgr/volumes: Handle internal metadata directories under '/volumes' properly.

Added by Kotresh Hiremath Ravishankar almost 2 years ago. Updated over 1 year ago.

Status:
Pending Backport
Priority:
Normal
Category:
Correctness/Safety
Target version:
% Done:

0%

Source:
Tags:
backport_processed
Backport:
quincy, pacific
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

1) The subvolumegroup ls lists `_index` directory as subvolumegroup

# bin/ceph fs subvolumegroup ls a
*** DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
2022-05-25T12:28:24.565+0530 7f2eb7c03700 -1 WARNING: all dangerous and experimental features are enabled.
2022-05-25T12:28:24.573+0530 7f2eb7c03700 -1 WARNING: all dangerous and experimental features are enabled.
[
    {
        "name": "_nogroup" 
    },
    {
        "name": "_index" 
    }
]
root@kotresh-T490s:~/sandbox/kotresh-ceph/ceph/build# 

The `_index` directory should be filtered.

2) The subvolume ls with internal directories as group_name lists contents inside.

# bin/ceph fs subvolume ls a --group-name=_index
*** DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
2022-05-25T12:31:07.125+0530 7f78dac3e700 -1 WARNING: all dangerous and experimental features are enabled.
2022-05-25T12:31:07.133+0530 7f78dac3e700 -1 WARNING: all dangerous and experimental features are enabled.
[
    {
        "name": "clone" 
    }
]

# bin/ceph fs subvolume ls a --group-name=_deleting
*** DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
2022-05-25T12:31:27.209+0530 7f151262e700 -1 WARNING: all dangerous and experimental features are enabled.
2022-05-25T12:31:27.217+0530 7f151262e700 -1 WARNING: all dangerous and experimental features are enabled.
[]

This should throw proper error.


Related issues 2 (0 open2 closed)

Copied to CephFS - Backport #57057: quincy: mgr/volumes: Handle internal metadata directories under '/volumes' properly.ResolvedNikhilkumar ShelkeActions
Copied to CephFS - Backport #57058: pacific: mgr/volumes: Handle internal metadata directories under '/volumes' properly.ResolvedKotresh Hiremath RavishankarActions
Actions #1

Updated by Kotresh Hiremath Ravishankar almost 2 years ago

  • Component(FS) mgr/volumes added
Actions #2

Updated by Kotresh Hiremath Ravishankar almost 2 years ago

Note that we need to also ignore `_legacy`

Actions #3

Updated by Venky Shankar almost 2 years ago

  • Category set to Correctness/Safety
  • Assignee set to Nikhilkumar Shelke
  • Target version set to v18.0.0
  • Backport set to quincy, pacific
Actions #4

Updated by Nikhilkumar Shelke almost 2 years ago

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

Updated by Venky Shankar almost 2 years ago

  • Pull request ID set to 46927
Actions #6

Updated by Venky Shankar over 1 year ago

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

Updated by Backport Bot over 1 year ago

  • Copied to Backport #57057: quincy: mgr/volumes: Handle internal metadata directories under '/volumes' properly. added
Actions #8

Updated by Backport Bot over 1 year ago

  • Copied to Backport #57058: pacific: mgr/volumes: Handle internal metadata directories under '/volumes' properly. added
Actions #9

Updated by Backport Bot over 1 year ago

  • Tags set to backport_processed
Actions #10

Updated by Venky Shankar over 1 year ago

  • Assignee changed from Nikhilkumar Shelke to Kotresh Hiremath Ravishankar
Actions

Also available in: Atom PDF