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

Custom queries

Backports
Backports: nautilus
Bug queue
Bug queue (last 30 days)
Bug triage
Clean-ups
Crash queue
Crash triage
Dashboard bugs (last 180 days)
Dashboard Bugs (NEW) in the last 30 days (except accessibility)
Dashboard CDS
Dashboard Sprint 23
Feedback
Low Hanging Fruit
Monitoring
My issues
Need Review
Pending backports
Product Backlog Scrub
Quincy: backlog
Rook - Dashboard Integration bugs
Telemetry
Test Failures

Actions

Copy link

Bug #39533

closed

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

Added by Tatjana Dehler about 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)

Related to Dashboard - Tasks #42536: mgr/dashboard: make greyed out `Delete` button for pools more straightforward

New

Stephan Müller

Actions

Copied to Dashboard - Backport #40616: nautilus: mgr/dashboard: notify the user about unset 'mon_allow_pool_delete' flag beforehand

Resolved

Tatjana Dehler

Actions

Issue # Delay: days Cancel

History
Notes
Property changes

Actions

Copy link

Updated by Tatjana Dehler about 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 about 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 about 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 about 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