-
Notifications
You must be signed in to change notification settings - Fork 127
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
[WIP]add_scrnario_OCP-71253 #1192
base: master
Are you sure you want to change the base?
Conversation
8537ff5
to
79e4830
Compare
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: geliu2016 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@@ -47,4 +52,13 @@ test-e2e: GO_TEST_FLAGS += -v | |||
test-e2e: GO_TEST_FLAGS += -timeout 2h | |||
test-e2e: GO_TEST_FLAGS += -p 1 | |||
test-e2e: test-unit | |||
test-e2e: | |||
go test \ |
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.
Thanks @geliu2016 - dunno if you saw this already, but @Elbehery added a very similar test in test/e2e/backup_script_test.go already. I believe you could reuse almost everything in there to write a testcase for that bug.
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.
Thank you, @tjungblu Yes, I saw there is similar test in backup_script_test.go already, and it checking the etcd backup feature with go function mode, and QE's test are focus on more complex scenarios, and QE using ginkgo mod to design test case, as I reviewed other sub component teams' e2e, they are also organize e2e test with go functions(Dev's prefer) and ginkgo(QE's prefer), it could co-exist, and I wish this backup case is a beginning, we will contribute QE's scenarios with this ginkgo framework.
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 prefer to have ginkgo tests in origin: https://github.com/openshift/origin/tree/master/test/extended/etcd
Otherwise this causes too much confusion and I don't want to maintain two test frameworks in here.
go.mod
Outdated
@@ -3,31 +3,34 @@ module github.com/openshift/cluster-etcd-operator | |||
go 1.20 | |||
|
|||
require ( | |||
github.com/davecgh/go-spew v1.1.1 | |||
github.com/cert-manager/cert-manager v1.14.0 |
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.
what on earth do we need cert manager for?
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.
aha, correct it.
test/e2e/etcd_scenario_test.go
Outdated
By("Wait for debug pod to be in Running phase.") | ||
err = wait.PollUntilContextTimeout(context.Background(), time.Second, 30*time.Minute, true, func(ctx context.Context) (done bool, err error) { | ||
pods, err := clientSet.CoreV1Interface.Pods(debugNamespace).List(ctx, metav1.ListOptions{}) | ||
if err != nil { |
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.
if err != nil && !apierrors.IsNotFound(err) {
return false, err
}
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.
Updated, thanks
79e4830
to
74e1ea4
Compare
@geliu2016: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
Stale issues rot after 30d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle rotten |
@tjungblu I added qe test scenario to e2e test, could you please take a look?
cc: @sandeepknd