Project

General

Profile

Feature #44025

Make it harder to set pool replica size to 1

Added by Neha Ojha about 4 years ago. Updated almost 4 years ago.

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

0%

Source:
Support
Tags:
Backport:
Reviewed:
Affected Versions:
Component(RADOS):
Pull request ID:

Description

Setting pool size to 1 is dangerous. Add an option like yes_i_really_really_mean_it, similar to what we have for pool deletion, when pool size needs to be set to 1.

History

#1 Updated by Neha Ojha about 4 years ago

Neha Ojha wrote:

Setting pool size to 1 is dangerous. Add an option like yes_i_really_really_mean_it, similar to what we have for pool deletion, when pool size needs to be set to 1.

ceph osd pool set mypool size 1 --yes-i-really-really-mean-it

#2 Updated by Neha Ojha about 4 years ago

  • Assignee set to Deepika Upadhyay

#3 Updated by Greg Farnum about 4 years ago

Pool deletion also requires a config option to be set on the monitor before it's allowed through.
I think we should do that for pool size 1 as well, and not make it easy for users to find that option?

#4 Updated by Deepika Upadhyay about 4 years ago

Greg Farnum wrote:

Pool deletion also requires a config option to be set on the monitor before it's allowed through.
I think we should do that for pool size 1 as well, and not make it easy for users to find that option?

I assume you are talking about:

To remove a pool the mon_allow_pool_delete flag must be set to true in the Monitor's configuration. Otherwise they will refuse to remove a pool.

from https://github.com/ceph/ceph/blob/master/doc/rados/operations/pools.rst#delete-a-pool
Is that correct?

#5 Updated by Greg Farnum about 4 years ago

Deepika Upadhyay wrote:

I assume you are talking about:

To remove a pool the mon_allow_pool_delete flag must be set to true in the Monitor's configuration. Otherwise they will refuse to remove a pool.

from https://github.com/ceph/ceph/blob/master/doc/rados/operations/pools.rst#delete-a-pool
Is that correct?

Yep!

#6 Updated by Neha Ojha about 4 years ago

  • Status changed from New to In Progress
  • Pull request ID set to 33281

#7 Updated by Deepika Upadhyay about 4 years ago

  • Status changed from In Progress to Pending Backport

#8 Updated by Neha Ojha about 4 years ago

  • Status changed from Pending Backport to Resolved
  • Backport deleted (nautilus)

#9 Updated by Nathan Cutler almost 4 years ago

  • Backport set to octopus

#10 Updated by Nathan Cutler almost 4 years ago

  • Status changed from Resolved to Pending Backport

#12 Updated by Nathan Cutler almost 4 years ago

  • Status changed from Pending Backport to Resolved
  • Backport deleted (octopus)

Also available in: Atom PDF