Skip to content

HDDS-15085. Add DN and Cluster Readiness - #10757

Merged
peterxcli merged 2 commits into
apache:masterfrom
henrybear327:HDDS-15085
Jul 20, 2026
Merged

HDDS-15085. Add DN and Cluster Readiness#10757
peterxcli merged 2 commits into
apache:masterfrom
henrybear327:HDDS-15085

Conversation

@henrybear327

@henrybear327henrybear327 commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Original work was done in #10159 by https://github.com/YutaLin

  • start 1..N datanodes
  • readiness wait until cluster is actually usable
  • local-safe replication defaults
  • integration test that cluster exits safe mode
  • DN count / readiness assertions
  • restart with persistent data

What is the link to the Apache JIRA

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

How was this patch tested?

https://github.com/apache/ozone/actions/runs/29327332184

@henrybear327

Copy link
Copy Markdown
ContributorAuthor

PTAL @peterxcli , thank you!

@adoroszlaiadoroszlai added the ozone-local HDDS-14893 label Jul 14, 2026
@henrybear327
henrybear327force-pushed the HDDS-15085 branch 2 times, most recently from 3a38005 to c36b14aCompareJuly 16, 2026 06:23

@chungen0126chungen0126 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @henrybear327 for working on this.

Signed-off-by: Bolin Lin <linbolin1230@gmail.com>

@chungen0126chungen0126 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Overall LGTM. However, I think we should log some information when some critical configuration error occurs.

@henrybear327

Copy link
Copy Markdown
ContributorAuthor

Overall LGTM. However, I think we should log some information when some critical configuration error occurs.

Thanks for the review @chungen0126!

What do you have in mind as the critical configuration error? I can improve this in a follow up PR.

Thanks.

@peterxclipeterxcli left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! @henrybear327 thanks for the work! pending ci.

@peterxcli
peterxcli merged commit 0f6295e into apache:masterJul 20, 2026
45 checks passed
@henrybear327
henrybear327 deleted the HDDS-15085 branch July 20, 2026 19:11
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ozone-localHDDS-14893

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@henrybear327@peterxcli@chungen0126@adoroszlai@YutaLin