Skip to content

Conversation

@Russole
Copy link
Contributor

@Russole Russole commented Jan 24, 2026

What changes were proposed in this pull request?

Update the Containers page label from No of Keys to No of Blocks to match the correct terminology and maintain consistency with the old UI.

What is the link to the Apache JIRA

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

How was this patch tested?

All CI checks passed.

Demo_3

Copy link
Contributor

@Gargi-jais11 Gargi-jais11 left a comment

Choose a reason for hiding this comment

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

Thanks @Russole for the patch. Overall LGTM, just one minor comment.

},
{
title: 'No. of Keys',
title: 'No of Blocks',
Copy link
Contributor

Choose a reason for hiding this comment

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

It should be Number of Blocks means No. of Blocks

Suggested change
title: 'No of Blocks',
title: 'No. of Blocks',

@Russole
Copy link
Contributor Author

Russole commented Jan 25, 2026

Thanks @Gargi-jais11 for the review.

@Russole Russole requested a review from Gargi-jais11 January 25, 2026 16:27
Copy link
Contributor

@echonesis echonesis left a comment

Choose a reason for hiding this comment

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

Thanks @Russole for the patch.
LGTM!

Copy link
Contributor

@chungen0126 chungen0126 left a comment

Choose a reason for hiding this comment

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

+1 LGTM

Copy link
Contributor

@devabhishekpal devabhishekpal left a comment

Choose a reason for hiding this comment

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

Thanks for the patch @Russole.
LGTM, +1

Copy link
Contributor

@sreejasahithi sreejasahithi left a comment

Choose a reason for hiding this comment

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

Thanks @Russole for working on this , LGTM

@ArafatKhan2198 ArafatKhan2198 merged commit 73d10a4 into apache:master Jan 27, 2026
71 of 72 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants