Skip to content

HDDS-8378. [Snapshot] Integration test to simulate staggered snapshot creation and deletion#4969

Merged
prashantpogde merged 8 commits intoapache:masterfrom
DaveTeng0:HDDS-8378
Jun 29, 2023
Merged

HDDS-8378. [Snapshot] Integration test to simulate staggered snapshot creation and deletion#4969
prashantpogde merged 8 commits intoapache:masterfrom
DaveTeng0:HDDS-8378

Conversation

@DaveTeng0
Copy link
Contributor

What changes were proposed in this pull request?

Integration test to simulate staggered snapshot creation and deletion

What is the link to the Apache JIRA

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

How was this patch tested?

Integration test.

@DaveTeng0
Copy link
Contributor Author

@smengcl smengcl added the snapshot https://issues.apache.org/jira/browse/HDDS-6517 label Jun 24, 2023
@smengcl smengcl changed the title HDDS-8378.Integration test to simulate staggered snapshot creation and deletion HDDS-8378. [Snapshot] Integration test to simulate staggered snapshot creation and deletion Jun 24, 2023
Copy link
Contributor

@prashantpogde prashantpogde left a comment

Choose a reason for hiding this comment

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

We need to have another incremental patch on top of this as suggested in the comment.

  • verify data integrity each time a snapshot is taken
  • different test data for each key
  • content verification

I am approving this PR as base PR to prevent PR from becoming too big with further enhancements.

@prashantpogde prashantpogde merged commit df3b155 into apache:master Jun 29, 2023
@prashantpogde
Copy link
Contributor

Thank you for this PR @DaveTeng0

vtutrinov pushed a commit to Cyrill/ozone that referenced this pull request Jul 3, 2023
vtutrinov pushed a commit to Cyrill/ozone that referenced this pull request Jul 3, 2023
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants