New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HDDS-3134. Debug Tool that gives chunk location information given a key. #684
Conversation
@sadanand48 Please fix the failing CI issues in this patch. |
...-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/KeyValueContainerData.java
Outdated
Show resolved
Hide resolved
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/admin/OzoneAdmin.java
Outdated
Show resolved
Hide resolved
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/debug/ChunkKeyHandler.java
Outdated
Show resolved
Hide resolved
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/debug/ChunkKeyHandler.java
Outdated
Show resolved
Hide resolved
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/debug/ChunkKeyHandler.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Plz add a sample output for the cmd for both verbose and non verbose mode.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look good to me with some minor comments to be addressed.
...-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/KeyValueContainerData.java
Outdated
Show resolved
Hide resolved
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/debug/ChunkKeyHandler.java
Outdated
Show resolved
Hide resolved
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/debug/package-info.java
Outdated
Show resolved
Hide resolved
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/debug/ChunkKeyHandler.java
Outdated
Show resolved
Hide resolved
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/debug/ChunkKeyHandler.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look good to me but will be holding off committing this until the acceptance tests failures are fixed and have clean CI run.
Please fix the acceptance test as its related.
Start / End / Elapsed: | 20200409 13:50:46.864 / 20200409 13:50:46.865 / 00:00:00.001
KEYWORD ${result} = commonlib . Execute ozone debug chunkinfo ${protocol}${server}/${volume}/bb1/key1 | jq, '.[]'
Start / End / Elapsed:20200409 13:50:46.864 / 20200409 13:50:46.865 / 00:00:00.00113:50:46.865FAILKeyword 'commonlib.Execute' expected 1 argument, got 2.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix the acceptance test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes @sadanand48, +1 the changes look good to me.
What changes were proposed in this pull request?
Debug tool to display chunk information given a key.Apart from Chunks ,it also displays information about the container on which the chunks are present and the pipeline info
Non-Verbose output:
Verbose Output:
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-3134
How was this patch tested?
Included robot tests in ozone-shell.robot