-
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 test on ssr object with controller namespace mix-ups #4521
Conversation
e39bfef
to
f13155b
Compare
f13155b
to
b576bea
Compare
b576bea
to
b4eede2
Compare
fmt.Sprintf("Failed to create an ssr object in %s namespace", VeleroCfg.VeleroNamespace)) | ||
|
||
var req *v1.ServerStatusRequestList | ||
By(fmt.Sprintf("Check ssr object in %s namespace", VeleroCfg.VeleroNamespace)) |
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.
Seems the case doesn't check the CR content of the SSR?
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.
OK
test/e2e/util/k8s/client.go
Outdated
@@ -45,6 +47,8 @@ type TestClient struct { | |||
// controller runtime framework by v2.0, it is the intent to remove all | |||
// client-go API clients. Please use the controller runtime to make API calls for tests. | |||
dynamicFactory client.DynamicFactory | |||
|
|||
VeleroClient clientset.Interface |
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.
As we are moving to kubebuilder, the generated clientset will be removed after finishing the refactoring. Let's use controller-runtime/pkg/client.Client
instead.
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.
I've changed to controller-runtime/pkg/client.Client
b4eede2
to
4f3410e
Compare
4f3410e
to
67fcc6d
Compare
Signed-off-by: Ming <mqiu@vmware.com>
67fcc6d
to
f36161e
Compare
Signed-off-by: Ming mqiu@vmware.com
Thank you for contributing to Velero!
Please add a summary of your change
It reference to #4522, it's test on ssr object.
Does your change fix a particular issue?
Fixes #(issue)
Please indicate you've done the following:
/kind changelog-not-required
as a comment on this pull request.site/content/docs/main
.