Skip to content

HDDS-14558. [Recon] Show fs reserved space in datanodes page table under Storage Capacity#9709

Merged
devabhishekpal merged 2 commits intoapache:masterfrom
priyeshkaratha:HDDS-14558
Feb 17, 2026
Merged

HDDS-14558. [Recon] Show fs reserved space in datanodes page table under Storage Capacity#9709
devabhishekpal merged 2 commits intoapache:masterfrom
priyeshkaratha:HDDS-14558

Conversation

@priyeshkaratha
Copy link
Contributor

What changes were proposed in this pull request?

On the Recon Datanode page, the Storage Capacity section currently displays Filesystem Capacity and Ozone Capacity. The difference between these two values represents the reserved space, but this is not explicitly shown, which can be confusing for users.

This pull request improves clarity by explicitly exposing the reserved space value. To achieve this, the existing datanode backend endpoint has been updated to return the correct reserved space, and the UI has been enhanced to display this value alongside the existing capacity metrics.

What is the link to the Apache JIRA

HDDS-14558

How was this patch tested?

Patch is tested manually
image

@priyeshkaratha priyeshkaratha changed the title HDDS-14558. Show fs reserved space in datanodes page table under Storage Capacity HDDS-14558. [Recon] Show fs reserved space in datanodes page table under Storage Capacity Feb 4, 2026
@priyeshkaratha priyeshkaratha marked this pull request as ready for review February 5, 2026 04:04
@ArafatKhan2198
Copy link
Contributor

Hey @priyeshkaratha, thanks for the changes. Could you also document these updates in the Recon Swagger API and any other documentation that references or points to this report?

I think for the overview page as well we will have to document it.

@priyeshkaratha
Copy link
Contributor Author

Thanks @ArafatKhan2198. I have filed https://issues.apache.org/jira/browse/HDDS-14594 for documentation update in Recon.

@devmadhuu devmadhuu self-requested a review February 10, 2026 13:34
Copy link
Contributor

@devmadhuu devmadhuu left a comment

Choose a reason for hiding this comment

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

Thanks @priyeshkaratha . Changes LGTM +1

@devabhishekpal
Copy link
Contributor

devabhishekpal commented Feb 17, 2026

Thanks for the reviews @devmadhuu @ArafatKhan2198.
Thanks for the improvement @priyeshkaratha.
LGTM, +1, merging this PR

@devabhishekpal devabhishekpal merged commit 9407d8a into apache:master Feb 17, 2026
44 checks passed
@adoroszlai
Copy link
Contributor

@devabhishekpal please feel free to remove Co-authored-by for insignificant contribution (merging master into the PR branch)

@devabhishekpal
Copy link
Contributor

Sure, @adoroszlai I had missed that on this patch. We might have to revert this patch, will keep this in mind for the future.

@adoroszlai
Copy link
Contributor

We might have to revert this patch

I don't think it should be reverted for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Comments