-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 schedule ordered resources E2E test #4913
Add schedule ordered resources E2E test #4913
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4913 +/- ##
==========================================
- Coverage 41.70% 41.69% -0.01%
==========================================
Files 204 204
Lines 18040 18263 +223
==========================================
+ Hits 7523 7615 +92
- Misses 9956 10069 +113
- Partials 561 579 +18
Continue to review full report at Codecov.
|
4d46702
to
e5fa2c0
Compare
err := test.Init() | ||
Expect(err).To(Succeed(), err) | ||
defer func() { | ||
Expect(DeleteNamespace(test.Ctx, test.Client, test.Namespace, false)).To(Succeed(), fmt.Sprintf("Failed to delete the namespace %s", test.Namespace)) |
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.
How about define a function to deal with velero schedule command, then other test cases can reuse it.
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.
@danfengliu yeah, I've modified
e5fa2c0
to
fddbb32
Compare
Signed-off-by: Ming <mqiu@vmware.com>
fddbb32
to
6b9b137
Compare
Signed-off-by: Ming mqiu@vmware.com
Thank you for contributing to Velero!
Please add a summary of your change
it refer to #4561, E2E test on schedule ordered resources
Please indicate you've done the following:
/kind changelog-not-required
as a comment on this pull request.site/content/docs/main
.