Skip to content

Conversation

@szetszwo
Copy link
Contributor

What changes were proposed in this pull request?

Document how to set arbitrary Ratis configuration properties with prefixes in Ozone.

What is the link to the Apache JIRA

HDDS-14433

How was this patch tested?

NA

@szetszwo szetszwo requested a review from jojochuang January 14, 2026 19:51
Copy link
Contributor

@ivandika3 ivandika3 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. Left one comment.

Also should we remove "[Website V2]" from the PR title?

@ivandika3 ivandika3 added the documentation Improvements or additions to documentation label Jan 15, 2026
@szetszwo szetszwo requested a review from ivandika3 January 15, 2026 18:37
@jojochuang
Copy link
Contributor

| Ozone Manager (OM) | `ozone.om.ha` |
| Storage Container Manager (SCM) | `ozone.scm.ha` |
| Datanode | `hdds.ratis` |
| Ozone Client | `hdds.ratis` |
Copy link
Contributor

Choose a reason for hiding this comment

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

please add an example. For example: hdds.ratis.raft.server.log.appender.buffer.byte-limit is for raft.server.log.appender.buffer.byte-limit in ozone client.

Copy link
Contributor Author

@szetszwo szetszwo Jan 16, 2026

Choose a reason for hiding this comment

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

Thanks for reviewing this! Just pushed a commit to add the example from #8668 (comment)

@szetszwo szetszwo changed the title HDDS-14433. [Website v2] [Docs] [Core Concepts] Ratis configurations. HDDS-14433. [Docs] Advanced Ratis configuration. Jan 16, 2026
@szetszwo
Copy link
Contributor Author

Also should we remove "[Website V2]" from the PR title?

Sure, just removed it.

@szetszwo szetszwo merged commit e5d3f04 into apache:master Jan 16, 2026
16 checks passed
@szetszwo
Copy link
Contributor Author

@ivandika3 , @jojochuang , thanks for reviewing this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants