Skip to content

Comments

HDDS-12036. Add storage indicators when reaching capacity#7663

Merged
tanvipenumudy merged 3 commits intoapache:masterfrom
devabhishekpal:HDDS-12036
Jan 10, 2025
Merged

HDDS-12036. Add storage indicators when reaching capacity#7663
tanvipenumudy merged 3 commits intoapache:masterfrom
devabhishekpal:HDDS-12036

Conversation

@devabhishekpal
Copy link
Contributor

@devabhishekpal devabhishekpal commented Jan 8, 2025

What changes were proposed in this pull request?

HDDS-12036. Add storage indicators when reaching capacity

Please describe your PR in detail:

  • Currently in Recon we do not have any quick indication for whether the storage is reaching it's capacity or not. This followed by the fact that Ozone storage is indicated by green colour is misleading.
  • As a part of this we make two main changes:
    • On the Overview page we are adding a red border
    • In the tables, the "storage bar" progress-bar and percentage text will turn red when storage is almost full.
    • The limit for the indication is set to above 80% i.e. when the storage is above 80% full, the above indicators will be shown.
  • We also remove the older separation of colors for the progress-bar for different category of capacity usage and instead formatting the tooltip to clearly show the data

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-12036

How was this patch tested?

Patch was tested manually
Screenshot 2025-01-08 at 17 30 57
Screenshot 2025-01-08 at 17 30 41

@devabhishekpal devabhishekpal marked this pull request as draft January 8, 2025 11:35
@devabhishekpal devabhishekpal marked this pull request as ready for review January 8, 2025 12:29
@ArafatKhan2198 ArafatKhan2198 self-requested a review January 8, 2025 13:08
@ArafatKhan2198
Copy link
Contributor

Thanks for the patch @devabhishekpal
I verified the changes on my local.

LGTM +1

image

Copy link
Contributor

@tanvipenumudy tanvipenumudy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @devabhishekpal for working on the patch, and @ArafatKhan2198 for the review!

@tanvipenumudy tanvipenumudy merged commit 76ac396 into apache:master Jan 10, 2025
15 checks passed
@devabhishekpal devabhishekpal deleted the HDDS-12036 branch January 10, 2025 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants