-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
add e2e s3 test #7833
add e2e s3 test #7833
Conversation
fmt.Println(a) | ||
Expect(err).NotTo(HaveOccurred()) | ||
}) | ||
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need additional checks to insure that the s3 snapshot can successfully be listed or queried.
926c546
to
be79534
Compare
You need to go fmt your files for CI to pass. |
Signed-off-by: Ian Cardoso <osodracnai@gmail.com>
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #7833 +/- ##
==========================================
+ Coverage 47.22% 51.52% +4.29%
==========================================
Files 143 143
Lines 14509 14524 +15
==========================================
+ Hits 6852 7483 +631
+ Misses 6571 5855 -716
- Partials 1086 1186 +100
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Ian Cardoso <osodracnai@gmail.com> (cherry picked from commit 9e33415)
Signed-off-by: Ian Cardoso <osodracnai@gmail.com> (cherry picked from commit 9e33415)
Signed-off-by: Ian Cardoso <osodracnai@gmail.com> (cherry picked from commit 9e33415)
Signed-off-by: Ian Cardoso <osodracnai@gmail.com> (cherry picked from commit 9e33415)
Proposed Changes
adds e2e test checking s3-etcd-snapshot functionality, closes #7441
Types of Changes
Verification
Testing
go test -timeout=15m ./tests/e2e/s3/...
Linked Issues
User-Facing Change
Further Comments