ozone icon indicating copy to clipboard operation
ozone copied to clipboard

HDDS-8101. Add FSO repair tool to ozone CLI in read-only and repair modes.

Open DaveTeng0 opened this issue 1 year ago • 4 comments

What changes were proposed in this pull request?

Bugs like HDDS-7592 can break the FSO tree and cause data to be orphaned in the OM. We have developed a tool to identify and repair this condition in the OM and tested it on affected clusters. This jira is to contribute the tool back to the community under the ozone CLI.

What is the link to the Apache JIRA

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

How was this patch tested?

Unit test, integration test.

DaveTeng0 avatar Apr 30 '24 00:04 DaveTeng0

cc. @errose28

DaveTeng0 avatar Apr 30 '24 00:04 DaveTeng0

I think we still need to decide what the CLI for this should look like. We could do ozone {debug,repair} fso-tree or ozone repair fso-tree [--dry-run]. Also as we add more of these type of commands I think ones that are specific to a component should be under their own subcommand for organization, like ozone repair om fso-tree.

Attila also brought up the --dry-run mode. I think if the command is under repair only, then dry run would not be the expected default value. If we add the read-only invocation under debug then that becomes the equivalent of dry run and no flag is needed.

Yeah! extracted common codes between FSODebugCLI and FSORepairCLI to separated base classes FSOBaseCLI and FSOBaseTool, and make them reuse same logic.

DaveTeng0 avatar May 03 '24 00:05 DaveTeng0

Hello team! please feel free to let me know if there is any new comment~ Thanks!

DaveTeng0 avatar May 14 '24 18:05 DaveTeng0