Skip to content

Conversation

@jojochuang
Copy link
Contributor

@jojochuang jojochuang commented Jul 3, 2025

What changes were proposed in this pull request?

HDDS-13373. [Docs] Add leader transfer command and service role listing command in the "OM High Availability" page

Please describe your PR in detail:

  • Update OM HA and SCM HA user doc to include command lines for transfer leadership and get service roles, both are useful in the context of HA.
  • Generated-by: Google Gemini Cli: Gemini 2.5 Flash; Prompt:

Read the jira https://issues.apache.org/jira/browse/HDDS-13373. Implement it.

Number of tokens used:

Input Tokens 361,864
Output Tokens 2,336
Thoughts Tokens 2,308
─────────────────────────────
Total Tokens 366,508
Total duration (API) 1m 17s

What is the link to the Apache JIRA

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

How was this patch tested?

Draft prepared by Gemini Cli; Reviewed manually.

…ng command in the "OM High Availability" page

Change-Id: I397296679415e7908c1a218b4a4783ccba8aef8d
@jojochuang jojochuang added documentation Improvements or additions to documentation AI-gen labels Jul 3, 2025
Copy link
Member

@peterxcli peterxcli left a comment

Choose a reason for hiding this comment

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

Other LGTM.

@jojochuang
Copy link
Contributor Author

Good catch! Updated.

@jojochuang jojochuang requested a review from peterxcli July 3, 2025 03:53
Change-Id: I91f667f66525e0df7def9f333aa566a187fd8dde
@jojochuang jojochuang requested a review from ivandika3 July 7, 2025 00:03
Change-Id: I05d091e0afc02c80701348e53230fe5c0217b57a
@ivandika3 ivandika3 changed the title HDDS-13373. [Docs] Add leader transfer command and service role listing command in the "OM High Availability" page HDDS-13373. [Docs] Add leader transfer command and service role listing command in the High Availability pages Jul 10, 2025
Change-Id: Iea0316735f96dd35a708e3f4606cb4400f83c8e5
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 update. LGTM +1.

@ivandika3 ivandika3 merged commit a9755a0 into apache:master Jul 10, 2025
14 checks passed
jojochuang added a commit to jojochuang/ozone that referenced this pull request Jul 31, 2025
…ng command in the High Availability pages (apache#8730)

Co-authored-by: Peter Lee <[email protected]>
Generated-by: Google Gemini Cli: Gemini 2.5 Flash
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI-gen documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants