Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mgr: expose CRUSHMap.find_roots() #41552

Merged
merged 1 commit into from May 31, 2021
Merged

Conversation

tchaikov
Copy link
Contributor

@tchaikov tchaikov commented May 26, 2021

so mgr module could use it to enumerate all nodes without parents

See-also: https://tracker.ceph.com/issues/50971
Signed-off-by: Kefu Chai kchai@redhat.com

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

so mgr module could use it to enumerate all nodes without parents

See-also: https://tracker.ceph.com/issues/50971
Signed-off-by: Kefu Chai <kchai@redhat.com>
Copy link
Contributor

@avanthakkar avanthakkar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks @tchaikov . Btw just a question though (not related to PR), I see some difference in o/p of ceph osd tree and crush buckets list, for e.g.

 "id": -1,
          "name": "default",
          "type_id": 11,
          "type_name": "root",
            }
  
          ]
        },
        {
          "id": -2,
          "name": "default~hdd",
          "type_id": 11,
          "type_name": "root",

this is the crush bucket list, but for crush tree I don't see the name=default~hdd. So are these not considered under crush tree?

@tchaikov
Copy link
Contributor Author

not sure what do you mean by "bucket tree", but FWIW, the notation of "default~hdd" is the result of how we compose buckets with class (like "hdd" and "ssd").

@avanthakkar
Copy link
Contributor

not sure what do you mean by "bucket tree", but FWIW, the notation of "default~hdd" is the result of how we compose buckets with class (like "hdd" and "ssd").

No, "crush tree". Ok, yes but I mean to ask that I can't see these buckets (with class hdd, ssd, etc..) in the output of ceph osd tree . But I think now I am clear bcz it has no corresponding tree so it won't be shown in the crush tree.

@tchaikov tchaikov merged commit 54d02f0 into ceph:master May 31, 2021
@tchaikov tchaikov deleted the wip-mgr-find-roots branch May 31, 2021 01:40
ljflores added a commit to ljflores/ceph that referenced this pull request Apr 2, 2022
mgr: expose CRUSHMap.find_roots()

Reviewed-by: Avan Thakkar <athakkar@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants