Skip to content

HDDS-9060. [snapshot] Add unit-testcase for snapshot fs -deleteSnapshot#5101

Merged
sadanand48 merged 2 commits intoapache:masterfrom
jyotirmoy-gh:HDDS-9060
Jul 25, 2023
Merged

HDDS-9060. [snapshot] Add unit-testcase for snapshot fs -deleteSnapshot#5101
sadanand48 merged 2 commits intoapache:masterfrom
jyotirmoy-gh:HDDS-9060

Conversation

@jyotirmoy-gh
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Add UTs for fs deleteSnapshot - TestOzoneFsSnapshot#testSnapshotDeleteFailure

What is the link to the Apache JIRA

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

How was this patch tested?

Testcase file - hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/fs/ozone/TestOzoneFsSnapshot.java

mvn -l testlog.out -Dtest=TestOzoneFsSnapshot#testSnapshotDeleteFailure test
[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.hadoop.fs.ozone.TestOzoneFsSnapshot
[INFO] Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 21.999 s - in org.apache.hadoop.fs.ozone.TestOzoneFsSnapshot
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 4, Failures: 0, Errors: 0, Skipped: 0

@jyotirmoy-gh
Copy link
Copy Markdown
Contributor Author

Please review - @nilotpalnandi @sadanand48 @smengcl

@adoroszlai adoroszlai added test snapshot https://issues.apache.org/jira/browse/HDDS-6517 labels Jul 22, 2023
Copy link
Copy Markdown
Contributor

@sadanand48 sadanand48 left a comment

Choose a reason for hiding this comment

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

Thanks @jyotirmoy-gh for the patch. LGTM

@sadanand48 sadanand48 merged commit 72d793c into apache:master Jul 25, 2023
@jyotirmoy-gh jyotirmoy-gh deleted the HDDS-9060 branch July 25, 2023 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

snapshot https://issues.apache.org/jira/browse/HDDS-6517 test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants