Skip to content

Conversation

@sreejasahithi
Copy link
Contributor

What changes were proposed in this pull request?

Improve the documentation for Container Replica Debugger Tool (ContainerLogParser.md) and add Chinese version of documentation.

What is the link to the Apache JIRA

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

How was this patch tested?

https://github.com/sreejasahithi/ozone/actions/runs/15540002838

@sreejasahithi sreejasahithi marked this pull request as ready for review June 10, 2025 04:27
@sreejasahithi
Copy link
Contributor Author

@ChenSammi and @Tejaskriya could you please review this PR.

@sreejasahithi
Copy link
Contributor Author

@ChenSammi and @Tejaskriya could you please review this PR.

@Tejaskriya I have addressed the reviews you had given in #8485

Copy link
Contributor

@Tejaskriya Tejaskriya 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 making the changes @sreejasahithi . The english version LGTM!

@Tejaskriya Tejaskriya added the documentation Improvements or additions to documentation label Jun 10, 2025
@ivandika3
Copy link
Contributor

@ChenSammi @xichen01 Can kindly take a look on the Chinese version when you have time? Thanks.


`ozone debug log container --db=<path to db> list --lifecycle=<state> --length=<limit>`
- 按状态列出所有集装箱,覆盖行限制:

Copy link
Contributor

Choose a reason for hiding this comment

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

集装箱 -> 容器

@sreejasahithi sreejasahithi requested a review from ChenSammi July 9, 2025 05:07
@ChenSammi ChenSammi merged commit bf9cdd0 into apache:master Jul 9, 2025
14 checks passed
@ChenSammi
Copy link
Contributor

Thanks @sreejasahithi, and @Tejaskriya @ivandika3 for the review.

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.

4 participants