ozone icon indicating copy to clipboard operation
ozone copied to clipboard

HDDS-6854. Implement List Snapshot API

Open chungen0126 opened this issue 2 years ago • 1 comments

What changes were proposed in this pull request?

Implement list API which will list all the snapshots for a given Bucket.

What is the link to the Apache JIRA

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

How was this patch tested?

[INFO] 
[INFO] --- maven-surefire-plugin:3.0.0-M5:test (default-test) @ ozone-manager ---
[INFO] 
[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.hadoop.ozone.om.TestOmMetadataManager
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.734 s - in org.apache.hadoop.ozone.om.TestOmMetadataManager
[INFO] 
[INFO] Results:
[INFO] 
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
[INFO] 
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  51.848 s
[INFO] Finished at: 2022-09-27T18:21:19+08:00
[INFO] ------------------------------------------------------------------------

chungen0126 avatar Sep 27 '22 12:09 chungen0126

@chungen0126 thx for the PR. Please make sure the PR passes precommit tests in your local repo before posting to the Apache Ozone github :)

jojochuang avatar Sep 28 '22 18:09 jojochuang

General comment. Would be great to add a robot test to test and verify the shell list snapshot command by creating and listing snapshots.

neils-dev avatar Nov 02 '22 22:11 neils-dev

Thanks @jojochuang @GeorgeJahad @prashantpogde @myskov @neils-dev @hemantk-12 for reviewing this.

Thanks @chungen0126 for the patch! I filed HDDS-7507 for pagination implementation.

smengcl avatar Nov 18 '22 02:11 smengcl