Project

General

Profile

Bug #36742

mgr/dashboard: Block >> Images: images in pools without "rbd" label not shown

Added by Alfonso Martínez over 5 years ago. Updated almost 3 years ago.

Status:
Won't Fix
Priority:
Normal
Assignee:
-
Category:
Component - RBD
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Page "Block >> Images" is not listing
images for pools without "rbd" application label

Expected behaviour:
As per previous conversation in IRC ceph-dashboard channel,
this page should list all images.

History

#1 Updated by Alfonso Martínez over 5 years ago

  • Status changed from New to In Progress

#2 Updated by Alfonso Martínez over 5 years ago

Jason Dillaman wrote:

The purpose for the application tags were to enable optimizations like this (i.e. we don't need to scan >>all the pools for an "rbd_directory" file). Your cluster should be in a health warn state if it has >>populated pools without an application tag associated with it. Also, at least the latest version of the >>dashboard allows you to manipulate the application tags. Therefore, if there is some confusion, perhaps >>we just need to add a pool drop-down selector to the table? The RBD pool drop-down already filters out >>non-RBD pools.

#3 Updated by Alfonso Martínez over 5 years ago

As per our conversation in daily standup meeting,

it is decided to go with the current behaviour and add a checkbox for showing all RBD images.

#4 Updated by Ernesto Puerta over 5 years ago

Why allowing dashboard users to take the wrong way? If pool app tags were created exactly for this purpose, the only thing a user has to do is properly tagging their pools as rbd (something that can be done from the dashboard itself).

By providing any 'tricky' checkbox, we are accustoming dashboard users to keep up with poor operating practices and overriding intended behaviours. If we fear most rbd users don't know about the need of tagging their rbd pools, let's add some warning message in case no rbd/rgw/cephfs is found due to missing tags.

#5 Updated by Ernesto Puerta over 5 years ago

And, btw, cluster would be on HEALTH_WARN due to pools without proper app tagging, so I think it's better to foster users towards HEALTH_OK clusters rather than providing work-arounds.

#6 Updated by Ricardo Marques over 5 years ago

I'm also in for of adding some kind of help text to guide the user, instead of adding a checkbox that overrides the expected behavior.

#7 Updated by Lenz Grimmer over 5 years ago

I think the actual problem is that it's possible to add RBDs on the command line on pools that don't have the "rbd" application tag. The Dashboard already prevents that from happening by only offering Pools that have the appropriate tag. Maybe the CLI should abort with an error or print at least a warning message that the pool is missing the tag?

#8 Updated by Lenz Grimmer over 5 years ago

#9 Updated by Sebastian Wagner almost 5 years ago

  • Category set to 139

#10 Updated by Lenz Grimmer over 4 years ago

  • Tags set to usability
  • Status changed from In Progress to Won't Fix
  • Assignee deleted (Alfonso Martínez)
  • Target version deleted (v14.0.0)

Resolving this as "won't fix" for now. Ceph issues a health warning if RBDs are created in an unlabeled pool already. As soon as the "rbd" label is added, the RBDs in that pool will appear in the list.

#11 Updated by Ernesto Puerta almost 3 years ago

  • Project changed from mgr to Dashboard
  • Category changed from 139 to Component - RBD

Also available in: Atom PDF