Skip to content

Conversation

@sumitagrawl
Copy link
Contributor

@sumitagrawl sumitagrawl commented Oct 27, 2025

What changes were proposed in this pull request?

revert Ratis log metadata - ensure the termUpdate event happens for metadata changes also
HDDS-13281. Disable Ratis metadata write to Raft Log on OM & SCM

What is the link to the Apache JIRA

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

How was this patch tested?

  • CI test cases for impact

@dombizita
Copy link
Contributor

Could you please check this @octachoron? thanks!

@sumitagrawl sumitagrawl marked this pull request as ready for review October 27, 2025 13:16
Copy link
Contributor

@szetszwo szetszwo left a comment

Choose a reason for hiding this comment

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

@sumitagrawl , thanks for working on this! Just a minor comment inlined.

@ivandika3 ivandika3 changed the title HDDS-13842. revert disable ratis metadata write for SC HDDS-13842. Revert disable Ratis metadata write for SCM Oct 28, 2025
@sumitagrawl sumitagrawl requested a review from szetszwo October 28, 2025 05:46
Copy link
Contributor

@szetszwo szetszwo left a comment

Choose a reason for hiding this comment

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

+1 the change looks good.

@sumitagrawl
Copy link
Contributor Author

This revert has not fixed the issue, closing the PR.

@sumitagrawl sumitagrawl reopened this Nov 1, 2025
@sumitagrawl sumitagrawl requested a review from szetszwo November 6, 2025 12:33
@sumitagrawl sumitagrawl changed the title HDDS-13842. Revert disable Ratis metadata write for SCM HDDS-13842. Exit safemode rule quickly at follower for idle system Nov 6, 2025
@sumitagrawl sumitagrawl changed the title HDDS-13842. Exit safemode rule quickly at follower for idle system HDDS-13842. Exit safemode rule quickly at follower for idle SCM Nov 6, 2025
@sumitagrawl sumitagrawl marked this pull request as draft November 6, 2025 12:45
@sumitagrawl
Copy link
Contributor Author

@szetszwo Updated the PR revert back to original state and remaining changes will be done as separate PR, please have a look

@sumitagrawl sumitagrawl marked this pull request as ready for review November 22, 2025 11:49
Copy link
Contributor

@szetszwo szetszwo left a comment

Choose a reason for hiding this comment

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

+1 the change looks good.

@sumitagrawl sumitagrawl reopened this Dec 1, 2025
@sumitagrawl sumitagrawl merged commit 227fb6a into apache:master Dec 1, 2025
45 checks passed
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.

3 participants