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: additional fixes to block pages #20941
Conversation
@dillaman this is great! Thanks for fixing the iSCSI part of the dashboard. |
Indeed - thanks a lot for your help! |
no worries |
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.
You left an unused import on iscsi.component.ts
file. Apart from that, this PR Lgtm.
Nice improvements / fixes, thanks!
@@ -1,5 +1,6 @@ | |||
import { Component } from '@angular/core'; | |||
|
|||
import { CellTemplate } from '../../../shared/enum/cell-template.enum'; | |||
import { CephShortVersionPipe } from '../../../shared/pipes/ceph-short-version.pipe'; | |||
import { DimlessBinaryPipe } from '../../../shared/pipes/dimless-binary.pipe'; |
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.
Now that we are no longer using the dimlessBinaryPipe
in this file, we can remove this import and the corresponding line on the class constructor (line 25).
Label rates w/ "/s" suffix and switch throughput back to the sparkline graph from dashboard v1. Signed-off-by: Jason Dillaman <dillaman@redhat.com>
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
02f920b
to
875f0bd
Compare
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.
lgtm
No description provided.