HDDS-14517. [Recon] Include all storage report fields in CSV report for Capacity Distribution#9681
HDDS-14517. [Recon] Include all storage report fields in CSV report for Capacity Distribution#9681priyeshkaratha wants to merge 1 commit intoapache:masterfrom
Conversation
|
Hi @devabhishekpal @devmadhuu @ArafatKhan2198 can you please review the changes? |
5ce79dd to
74f3f7f
Compare
devabhishekpal
left a comment
There was a problem hiding this comment.
This change looks good to me, but it also needs frontend change.
The download button currently calls the pendingDeletion/download API endpoint.
However this changes it to storageDistribution/download so the download button won't work
Thanks @devabhishekpal for the review. calling storageDistribution/download is already part of the PR. Did I miss anything specific? |
devabhishekpal
left a comment
There was a problem hiding this comment.
Thanks for the patch, lgtm @priyeshkaratha
What changes were proposed in this pull request?
This pull request simplifies how datanode storage metrics are downloaded. Instead of having multiple endpoints, the download logic is now handled by a single improved endpoint. The old endpoint that only supported pending deletion data has been removed. The new endpoint generates a detailed CSV with all relevant storage metrics, giving a more complete view of datanode health and usage.
What is the link to the Apache JIRA
HDDS-14517
How was this patch tested?
Tested using modified testcases.