Skip to content

Comments

HDDS-9206. testSnapshotBackgroundServices is failing most of the time in workflow run#5219

Merged
GeorgeJahad merged 2 commits intoapache:masterfrom
mladjan-gadzic:HDDS-9206
Aug 24, 2023
Merged

HDDS-9206. testSnapshotBackgroundServices is failing most of the time in workflow run#5219
GeorgeJahad merged 2 commits intoapache:masterfrom
mladjan-gadzic:HDDS-9206

Conversation

@mladjan-gadzic
Copy link
Contributor

@mladjan-gadzic mladjan-gadzic commented Aug 24, 2023

What changes were proposed in this pull request?

Assertion that times out (compaction backup files pruning) during workflow run is commented out. This is only a temporary solution, because compaction backup files pruning is just a part of larger set of snapshot background services tests which are important. Jira HDDS-9209 is created to fix this.

What is the link to the Apache JIRA

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

How was this patch tested?

currentCompactionLog);

checkIfCompactionBackupFilesWerePruned(sstBackupDir, numberOfSstFiles);
// TODO: create Jira to fix and re-enable this test
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you please create a jira and attach it here? Or use HDDS-9206? What ever you feel like?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added to PR description. Will add in code as well.

Copy link
Contributor

@hemantk-12 hemantk-12 left a comment

Choose a reason for hiding this comment

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

Thanks for the patch @mladjan-gadzic and quick fix.

@mladjan-gadzic
Copy link
Contributor Author

Thank you @hemantk-12 for a quick response and review!

@GeorgeJahad GeorgeJahad merged commit 489e2f3 into apache:master Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants