Developer Guide: Refresh e2e testing documents for kubetest2#5045
Conversation
This change creates a new file with e2e testing documentation for the new kubetest2 test runner. Once kubetest is retired, this can drop right into the existing e2e developer guide docs.
|
Hi @pymander. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions 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. |
|
/assign @amwat |
amwat
left a comment
There was a problem hiding this comment.
Thanks a lot for creating this guide!
/ok-to-test
FYI @BenTheElder @spiffxp
amwat
left a comment
There was a problem hiding this comment.
This is really great and detailed!
This addresses some suggested changes to the kubetest2 getting started guide.
|
/assign amwat |
This change adds details on building with kubetest2 and without --legacy-mode.
amwat
left a comment
There was a problem hiding this comment.
This looks like a great start! Thanks.
We can definitely keep iterating on this as and when things change.
/lgtm
|
/assign @BenTheElder @spiffxp |
amwat
left a comment
There was a problem hiding this comment.
one optional addition: perhaps we can point out somewhere that build up down test can also be used all at once.
e.g. kubetest2 gce --build --up --down --test=ginkgo -- --focus-regex "\[Feature:Performance\]"
|
I tried combining the `--up`, `--down`, and `--test` options in various forms and couldn't get that to work right. I didn't record the results because I figured that functionality wasn't carrying over. But I'll try again and see if it works now. Do you have a working example perhaps?
|
|
yup we use it in CI: recent runs/logs here: https://testgrid.k8s.io/provider-gcp-periodics#Conformance%20-%20Cloud%20Provider%20GCP%20-%20master |
|
Wow, that is amazing! Okay, I'll try it by hand. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: pymander, spiffxp The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This change creates a new file with e2e testing documentation for the
new kubetest2 test runner. Once kubetest is retired, this can drop
right into the existing e2e developer guide docs.
Which issue(s) this PR fixes:
Fixes #