Skip to content

HDDS-12928. datanode min free space configuration #8388

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.

Sign up for GitHub

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

Merged
merged 15 commits into from
May 28, 2025
Merged

HDDS-12928. datanode min free space configuration #8388

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
b07c088
HDDS-12928. design approach for min free space configuration
sumitagrawl May 5, 2025
db7211c
HDDS-12928. min free space configuration change with default
sumitagrawl May 8, 2025
cef0da3
checkstyle and doc fix
sumitagrawl May 8, 2025
f07be0e
test case fix
sumitagrawl May 8, 2025
c268e57
checkstyle fix
sumitagrawl May 8, 2025
23f06cf
fix test case
sumitagrawl May 8, 2025
2fca212
fix test case
sumitagrawl May 8, 2025
0a215ec
fix test cases
sumitagrawl May 9, 2025
6032c71
fix review comments
sumitagrawl May 12, 2025
96a7486
check style fix
sumitagrawl May 12, 2025
914ff7f
fix review comments
sumitagrawl May 12, 2025
e3e7480
fix review comment
sumitagrawl May 13, 2025
2b74370
fix review comment for doc
sumitagrawl May 19, 2025
614acd3
Merge branch 'apache:master' into HDDS-12928
sumitagrawl May 20, 2025
c22a69c
fix test case config
sumitagrawl May 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test case fix
  • Loading branch information
@sumitagrawl
sumitagrawl committed May 16, 2025
commit f07be0effec1fd4347b5caa8e4aa6452380830a2
8 changes: 4 additions & 4 deletions ...rc/test/java/org/apache/hadoop/ozone/container/common/volume/TestReservedVolumeSpace.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -202,12 +202,12 @@ public void testMinFreeSpaceCalculator() throws Exception {
assertEquals(minSpace, conf.getObject(DatanodeConfiguration.class).getMinFreeSpace(capacity));

conf.setFloat(HDDS_DATANODE_VOLUME_MIN_FREE_SPACE_PERCENT, 0.01f);
// When both are set, minSpace will be used
// When both are set, max(minSpace, %cent), minSpace will be used
assertEquals(minSpace, conf.getObject(DatanodeConfiguration.class).getMinFreeSpace(capacity));

// capacity * 1% = 10
conf.unset(HDDS_DATANODE_VOLUME_MIN_FREE_SPACE);
assertEquals(10, conf.getObject(DatanodeConfiguration.class).getMinFreeSpace(capacity));
conf.setFloat(HDDS_DATANODE_VOLUME_MIN_FREE_SPACE_PERCENT, 1f);
// When both are set, max(minSpace, %cent), hence %cent will be used
assertEquals(1000, conf.getObject(DatanodeConfiguration.class).getMinFreeSpace(capacity));
}

private long getExpectedDefaultReserved(HddsVolume volume) {
Expand Down