Skip to content

HDDS-12208. Give ozone debug replicas verify options for OM container location cache behavior. - #11015

Merged
aryangupta1998 merged 3 commits into
apache:masterfrom
aryangupta1998:HDDS-12208_new
Aug 18, 2026
Merged

HDDS-12208. Give ozone debug replicas verify options for OM container location cache behavior.#11015
aryangupta1998 merged 3 commits into
apache:masterfrom
aryangupta1998:HDDS-12208_new

Conversation

@aryangupta1998

@aryangupta1998 aryangupta1998 commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This change adds explicit ozone debug replicas verify options to control OM container location cache behavior during key verification.

Adds --refresh-from-scm to force OM to refresh container locations from SCM before returning key info.
Keeps default behavior unchanged (no flag: OM cache path remains as-is, for faster execution).
Updates debug CLI docs and integration tests to cover both the long and short flags.

Replica verification can be affected by stale OM container location cache entries in some situations.
This option gives operators a slower but potentially more reliable verification mode when they need fresh container location data.

What is the link to the Apache JIRA

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

How was this patch tested?

Tested manually.

Comment thread hadoop-hdds/docs/content/tools/debug/DebugReplicas.md Outdated

@sadanand48 sadanand48 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 @aryangupta1998 for the patch , LGTM

@ashishkumar50 ashishkumar50 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.

@aryangupta1998 Thanks for the patch, patch looks fine to me. Please find comment wrt test.

@ashishkumar50 ashishkumar50 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.

LGTM

@aryangupta1998
aryangupta1998 merged commit 2c02f43 into apache:master Aug 18, 2026
46 checks passed
@aryangupta1998

Copy link
Copy Markdown
Contributor Author

Thanks @ashishkumar50, @ayushtkn, and @sadanand48 for the reviews!

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.

4 participants