-
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
E2E SSR test add retry mechanism and logs #4591
Conversation
✔️ Deploy Preview for fervent-minsky-ed95cc canceled. 🔨 Explore the source changes: e404601 🔍 Inspect the deploy log: https://app.netlify.com/sites/fervent-minsky-ed95cc/deploys/61f3caccd6703e0007fd886b |
60f7097
to
09ca005
Compare
Signed-off-by: Ming <mqiu@vmware.com>
if err == waitutil.ErrWaitTimeout { | ||
fmt.Printf("exceed test case deadline and failed to check ssr object in %s namespace", VeleroCfg.VeleroNamespace) | ||
} | ||
Expect(err).To(Succeed(), fmt.Sprintf("Failed to check ssr object in %s namespace", VeleroCfg.VeleroNamespace)) | ||
|
||
By(fmt.Sprintf("Check ssr object in %s namespace", testNS)) |
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.
Why repeated code with line #76?
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.
It tested in two different namespaces, one is the velero installation namespace, another is not a velero installation namespace. velero installation namespace should get server version, another should not get the server version
Signed-off-by: Ming mqiu@vmware.com
Thank you for contributing to Velero!
Please add a summary of your change
Does your change fix a particular issue?
Fixes #(issue)
it reference to #4590
Please indicate you've done the following:
/kind changelog-not-required
as a comment on this pull request.site/content/docs/main
.