Bug #39533: mgr/dashboard: notify the user about unset 'mon_allow_pool_delete' flag beforehand - Dashboard - Ceph

Actions

Copy link

Bug #39533

closed

mgr/dashboard: notify the user about unset 'mon_allow_pool_delete' flag beforehand

Added by Tatjana Dehler almost 5 years ago. Updated about 3 years ago.

Status:

Resolved

Priority:

Normal

Assignee:

Tatjana Dehler

Category:

Target version:

% Done:

Source:

Development

Tags:

Backport:

nautilus

Regression:

Severity:

3 - minor

Reviewed:

Affected Versions:

ceph-qa-suite:

Pull request ID:

28260

Crash signature (v1):

Crash signature (v2):

Files

Screenshot_2019-04-30_12-00-31.png (45.3 KB) Screenshot_2019-04-30_12-00-31.png

pool deletion dialog

Tatjana Dehler, 04/30/2019 10:05 AM

Related issues 2 (1 open — 1 closed)

Actions

Copy link

Updated by Tatjana Dehler almost 5 years ago

File Screenshot_2019-04-30_12-00-31.png Screenshot_2019-04-30_12-00-31.png added
Subject changed from mgr/dashboard: to mgr/dashboard: notify the user about unset 'mon_allow_pool_delete' flag beforehand
Category set to 152
Assignee set to Tatjana Dehler
Source set to Development

If the 'mon_allow_pool_delete' flag is not set and the user tries to delete a pool, he only gets notified about it, when he checks "Yes, I am sure." and clicks the "Delete Pool" button in the deletion dialog (see attachment). It would be good to let him know he currently can't delete pools beforehand.

There are to possible options:

1. Deactivate the "Delete" button in the menu and show a tooltip containing the reason why it's not possible to delete pools. It's the favored solution (as discussed in the dashboard stand-up) but needs some investigation if it's possible to show a tooltip for deactivated components.
2. If option 1.) is not possible a modal dialog, containing the reason why deleting pools is not possible, should be shown after clicking "Delete" in the menu.

Actions

Copy link

Updated by Sebastian Wagner almost 5 years ago

Option 3: Fix mon_allow_pool_delete : This flag was never meant to introduce Dashboard bugs.
Option 4: Remove mon_allow_pool_delete altogether and replace it with something more suitable for CLI users.

Relates to https://github.com/ceph/ceph/pull/11665

Actions

Copy link

Updated by Wido den Hollander almost 5 years ago

I would be very, very cautious with this. Removing pools should not be very easy for users.

The mon_allow_pool_delete was introduced as users would remove pools by accident. Keep in mind that there are also APIs which do not ask and just do what you want them to do. You can remove a 1PB pool without the API asking twice. We do not want that to happen.

Only if we have a soft-delete or a grace period when removing a pool this might be an option. Otherwise I would always make it very difficult to remove pools.

Actions

Copy link

Updated by Lenz Grimmer almost 5 years ago

Wido den Hollander wrote:

I would be very, very cautious with this. Removing pools should not be very easy for users.

The mon_allow_pool_delete was introduced as users would remove pools by accident. Keep in mind that there are also APIs which do not ask and just do what you want them to do. You can remove a 1PB pool without the API asking twice. We do not want that to happen.

Only if we have a soft-delete or a grace period when removing a pool this might be an option. Otherwise I would always make it very difficult to remove pools.

Thanks for your feedback, Wido!

Note that this issue is primarily about making the user aware of the fact that mon_allow_pool_delete is set to false and that the delete operation will not succeed.

This provides a better user experience than letting the user proceed with the pool deletion workflow and then getting an error from the backend, that the deletion operation failed.

Actions

Copy link