Skip to content

HDDS-13526. [Docs] Update snapshot related configuration properties#8883

Merged
smengcl merged 3 commits intoapache:masterfrom
jojochuang:HDDS-13526
Sep 16, 2025
Merged

HDDS-13526. [Docs] Update snapshot related configuration properties#8883
smengcl merged 3 commits intoapache:masterfrom
jojochuang:HDDS-13526

Conversation

@jojochuang
Copy link
Contributor

HDDS-13526. [Docs] Update snapshot related configuration properties

What changes were proposed in this pull request?

  • A few configuration properties were missed from the last doc update. Add them according to ozone-default.xml
  • Generated-by: Google Gemini Pro 2.5 + Gemini Cli

What is the link to the Apache JIRA

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

How was this patch tested?

The new properties added are all coming from ozone-default.xml

Change-Id: I882d40181bea9a03b6659ec963a94fa0272cca5d
@jojochuang jojochuang added documentation Improvements or additions to documentation snapshot https://issues.apache.org/jira/browse/HDDS-6517 AI-gen labels Jul 31, 2025
Change-Id: Icb16c0021191c9f53cd381346df569cf7c3e9a7f
Change-Id: I6ff700873b877091a810290405588c14315738b0
@jojochuang jojochuang marked this pull request as ready for review August 25, 2025 18:35
@jojochuang
Copy link
Contributor Author

@peterxcli can you review this one? I updated the default value of the config properties and they are now correct.

<tag>OZONE, OM, MANAGEMENT</tag>
<description>
The maximum number of filesystem snapshot allowed in an Ozone Manager.
This limit is set to 65000 because the ext4 filesystem limits the number of hard links per file to 65,000.
Copy link
Member

Choose a reason for hiding this comment

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

Curious the reason why you remove this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because we ended up setting the limit to 10k.

@jojochuang jojochuang requested a review from peterxcli August 29, 2025 22:36
Copy link
Contributor

@smengcl smengcl left a comment

Choose a reason for hiding this comment

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

Thanks @jojochuang

@smengcl smengcl merged commit 9a37748 into apache:master Sep 16, 2025
47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI-gen documentation Improvements or additions to documentation snapshot https://issues.apache.org/jira/browse/HDDS-6517

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants