-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
mgr/dashboard: Cleanup of OSD list methods #24823
Conversation
f8f95a2
to
a05ed0c
Compare
jenkins retest this please |
It would be good if you briefly mentioned that this is a cleanup PR, which doesn't fix a bug or implements a feature. |
src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/osd/osd-list/osd-list.component.spec.ts
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you change the commit title to something like Cleanup of OSD List's methods
?
I don't think interface
is the best word to use here.
Some test changes were required, too. Fixes: https://tracker.ceph.com/issues/36615 Signed-off-by: Stephan Müller <smueller@suse.com>
a05ed0c
to
ff64ffd
Compare
Addressed all comments |
jenkins test dashboard |
As discussed #24606 (comment) , the interface methods should be removed for a cleanup RFC PR.
Fixes: https://tracker.ceph.com/issues/36615
Signed-off-by: Stephan Müller smueller@suse.com