Skip to content

Conversation

@Gargi-jais11
Copy link

@Gargi-jais11 Gargi-jais11 commented Jan 9, 2026

What changes were proposed in this pull request?

Migrate the v1 doc "Snapshot Configuration Properties" https://ozone.apache.org/docs/edge/feature/snapshot-configuration-properties.html to v2 doc site.

Sugested location: Administrator Guide -> Operations -> Snapshots -> Configuration Properties https://ozone-site-v2.staged.apache.org/docs/administrator-guide/operations/snapshots

What is the link to the Apache Jira?

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

How was this patch tested?

Check off which of the following tests were done on this change. If additional testing was done, please elaborate here as well.

  • The CI checks on my fork are passing
  • I verified the rendered content using a local preview
  • I manually verified the steps provided in this change work as described

Github Actions CI - https://github.com/Gargi-jais11/ozone-site/actions/runs/20849173064

|----------|---------------|-------------|
| `ozone.om.fs.snapshot.max.limit` | 10000 | Max snapshots per bucket. Safety limit. |
| `ozone.om.ratis.snapshot.dir` | ratis-snapshot under OM DB dir | The directory where OM Ratis snapshots are stored. |
| `ozone.om.ratis.snapshot.max.total.sst.size` | 100000000 | The maximum total size of SST files to be included in a Ratis snapshot. |
Copy link
Contributor

Choose a reason for hiding this comment

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

TODO: this value is too small for production clusters. We need to bump up the default value: https://issues.apache.org/jira/browse/HDDS-14403

Copy link
Contributor

@jojochuang jojochuang left a comment

Choose a reason for hiding this comment

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

Went through each and every configuration property and double checked the value. This is looking good now.

@jojochuang jojochuang merged commit 1ffc081 into apache:HDDS-9225-website-v2 Jan 9, 2026
11 checks passed
@jojochuang
Copy link
Contributor

Merged. Thanks @Gargi-jais11

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.

2 participants