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: capacity card #47396
mgr/dashboard: capacity card #47396
Conversation
5b057be
to
176649c
Compare
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved |
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually 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.
Great work!
...d/mgr/dashboard/frontend/src/app/ceph/new-dashboard/dashboard-pie/dashboard-pie.component.ts
Outdated
Show resolved
Hide resolved
...d/mgr/dashboard/frontend/src/app/ceph/new-dashboard/dashboard-pie/dashboard-pie.component.ts
Outdated
Show resolved
Hide resolved
...d/mgr/dashboard/frontend/src/app/ceph/new-dashboard/dashboard-pie/dashboard-pie.component.ts
Outdated
Show resolved
Hide resolved
...d/mgr/dashboard/frontend/src/app/ceph/new-dashboard/dashboard-pie/dashboard-pie.component.ts
Show resolved
Hide resolved
...d/mgr/dashboard/frontend/src/app/ceph/new-dashboard/dashboard-pie/dashboard-pie.component.ts
Outdated
Show resolved
Hide resolved
src/pybind/mgr/dashboard/frontend/src/app/ceph/new-dashboard/dashboard/dashboard.component.ts
Outdated
Show resolved
Hide resolved
src/pybind/mgr/dashboard/frontend/src/app/ceph/new-dashboard/dashboard/dashboard.component.ts
Outdated
Show resolved
Hide resolved
src/pybind/mgr/dashboard/frontend/src/app/ceph/new-dashboard/dashboard/dashboard.component.ts
Outdated
Show resolved
Hide resolved
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved |
739675f
to
2b70b4f
Compare
@epuertat Could you review it again? I added your suggested changes and had some questions with some of them. Anyways, let me know |
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.
Great job, @Pegonzal ! LGTM. Just a couple of comments (the i18n should be fixed).
...d/mgr/dashboard/frontend/src/app/ceph/new-dashboard/dashboard-pie/dashboard-pie.component.ts
Outdated
Show resolved
Hide resolved
2b70b4f
to
67cc207
Compare
jenkins test make check |
jenkins test api |
jenkins test dashboard |
...d/mgr/dashboard/frontend/src/app/ceph/new-dashboard/dashboard-pie/dashboard-pie.component.ts
Outdated
Show resolved
Hide resolved
jenkins test make check |
jenkins test api |
not sure but maybe the feature branch needs to rebased against the main for all these test to pass. |
mgr/dashboard: fix variable names and using Math.round() now mgr/dashboard cluster endpoint changes mgr/dashboard: refactoring mgr/dashboard: string concatenation to interpolation and typedict to namedtuple mgr/dashboard: change function order Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
67cc207
to
569e5f1
Compare
Capacity card for the new dashboard page.
Currently working with two rings. One outter ring for the osd-nearFullRatio and osd-FullRatio and other ring for the capacity usage display.
Signed-off-by: Pedro Gonzalez Gomez pegonzal@redhat.com
Contribution Guidelines
To sign and title your commits, please refer to Submitting Patches to Ceph.
If you are submitting a fix for a stable branch (e.g. "pacific"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test classic perf
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test dashboard cephadm
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox
jenkins test windows