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

doc: more details for pool deletion #10190

Merged
merged 1 commit into from Jul 10, 2016
Merged

Conversation

ktdreyer
Copy link
Member

@ktdreyer ktdreyer commented Jul 7, 2016

Prior to this change, the documention instructed administrators to delete rulesets and users, but did not go into details regarding how to do that.

Add example commands that admins may use to search for rulesets and users that might reference the to-be-deleted pool.

@badone
Copy link
Contributor

badone commented Jul 8, 2016

lgtm, not sure how a doc change fails to build/check?

@badone
Copy link
Contributor

badone commented Jul 8, 2016

retest this please

deleting those users too.
consider removing them when you no longer need your pool::

ceph osd dump | grep {pool-name}
Copy link
Contributor

Choose a reason for hiding this comment

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

could just use

ceph osd pool get {pool-name} crush_ruleset

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks! changed

Prior to this change, the documention instructed administrators to
delete rulesets and users, but did not go into details regarding how to
do that.

Add example commands that admins may use to search for rulesets and
users that might reference the to-be-deleted pool.

Signed-off-by: Ken Dreyer <kdreyer@redhat.com>
@tchaikov
Copy link
Contributor

tchaikov commented Jul 8, 2016

lgtm.

@badone
Copy link
Contributor

badone commented Jul 8, 2016

retest this please

@badone
Copy link
Contributor

badone commented Jul 8, 2016

Still lgtm

@tchaikov tchaikov merged commit 89ba840 into ceph:master Jul 10, 2016
@ktdreyer ktdreyer deleted the doc-delete-pool branch May 25, 2017 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants