Skip to content
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

Fix flaky snapshot test. #768

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

dblock
Copy link
Member

@dblock dblock commented Jan 6, 2025

Description

Fix for a flaky snapshot test. The snapshot completes too fast so /_snapshot/{repository}/_status isn't returning anything.

Issues Resolved

Closes #756.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dblock dblock added the skip-changelog No need to update CHANGELOG. label Jan 6, 2025
Copy link
Contributor

github-actions bot commented Jan 6, 2025

Changes Analysis

Commit SHA: 9d6cb48
Comparing To SHA: 95ddd0b

API Changes

Summary

NO CHANGES

Report

The full API changes report is available at: https://github.com/opensearch-project/opensearch-api-specification/actions/runs/12641004461/artifacts/2392700599

API Coverage

Before After Δ
Covered (%) 616 (60.33 %) 616 (60.33 %) 0 (0 %)
Uncovered (%) 405 (39.67 %) 405 (39.67 %) 0 (0 %)
Unknown 43 43 0

Copy link
Contributor

github-actions bot commented Jan 6, 2025

Spec Test Coverage Analysis

Total Tested
546 546 (100 %)

@dblock dblock force-pushed the fix-flaky-snapshot-test branch from 2ea33d3 to 9d6cb48 Compare January 6, 2025 21:56
@Xtansia Xtansia merged commit 7ff1821 into opensearch-project:main Jan 6, 2025
30 checks passed
@dblock dblock deleted the fix-flaky-snapshot-test branch January 6, 2025 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog No need to update CHANGELOG.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] snapshot/status.yaml is flaky
2 participants