Project

General

Profile

Bug #43061

ceph fs add_data_pool doesn't set pool metadata properly

Added by Paul Emmerich 4 months ago. Updated about 1 month ago.

Status:
In Progress
Priority:
Normal
Assignee:
Category:
-
Target version:
% Done:

0%

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

Description

maybe related to https://tracker.ceph.com/issues/36028

root@ceph-dev-build ~/ceph/build $ ./bin/ceph osd pool create test 1
*** DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
2019-11-29T00:16:15.153+0100 7fa7b35ad700 -1 WARNING: all dangerous and experimental features are enabled.
2019-11-29T00:16:15.253+0100 7fa7b35ad700 -1 WARNING: all dangerous and experimental features are enabled.
pool 'test' created
root@ceph-dev-build ~/ceph/build $ ./bin/ceph osd pool application enable test cephfs
*** DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
2019-11-29T00:16:30.657+0100 7f084ca39700 -1 WARNING: all dangerous and experimental features are enabled.
2019-11-29T00:16:30.761+0100 7f084ca39700 -1 WARNING: all dangerous and experimental features are enabled.
enabled application 'cephfs' on pool 'test'
root@ceph-dev-build ~/ceph/build $ ./bin/ceph fs add_data_pool a test
*** DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
2019-11-29T00:16:54.521+0100 7f1f8f622700 -1 WARNING: all dangerous and experimental features are enabled.
2019-11-29T00:16:54.613+0100 7f1f8f622700 -1 WARNING: all dangerous and experimental features are enabled.
added data pool 3 to fsmap
root@ceph-dev-build ~/ceph/build $ ./bin/ceph osd pool application get test
*** DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
2019-11-29T00:17:11.977+0100 7fbcc4c72700 -1 WARNING: all dangerous and experimental features are enabled.
2019-11-29T00:17:12.093+0100 7fbcc4c72700 -1 WARNING: all dangerous and experimental features are enabled.
{
    "cephfs": {}
}

History

#1 Updated by Patrick Donnelly 4 months ago

  • Status changed from New to Triaged
  • Assignee set to Ramana Raja
  • Target version set to v15.0.0
  • Source set to Development
  • Backport set to nautilus
  • Component(FS) MDSMonitor added

#2 Updated by Ramana Raja 4 months ago

  • Status changed from Triaged to In Progress
$ ./bin/ceph osd pool create test 1
$ ./bin/ceph fs add_data_pool a test
$ ./bin/ceph osd pool application get test 
{
    "cephfs": {
        "data": "a" 
    }
}

`add_data_pool` sets the pool's meta data properly if the pool's application metadata isn't already set/enabled.

In the description, the `osd pool application enable test cephfs` sets the pool's application to `cephfs`. The subsequent `fs add_data_pool` command doesn't update the contents of the pool's application metadata. To get around this for now, you can, execute `osd pool application set test cephfs data a`.

#3 Updated by Patrick Donnelly 4 months ago

Ramana Raja wrote:

[...]
`add_data_pool` sets the pool's meta data properly if the pool's application metadata isn't already set/enabled.

In the description, the `osd pool application enable test cephfs` sets the pool's application to `cephfs`. The subsequent `fs add_data_pool` command doesn't update the contents of the pool's application metadata. To get around this for now, you can, execute `osd pool application set test cephfs data a`.

So if the file system is empty in the "data" field, I would suggest we could overwrite it with the file system name in `add_data_pool`. Makes sense?

#4 Updated by Patrick Donnelly about 2 months ago

Status on this Ramana?

#5 Updated by Patrick Donnelly about 1 month ago

  • Target version changed from v15.0.0 to v16.0.0
  • Backport changed from nautilus to octopus,nautilus

Also available in: Atom PDF