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 instructions about how to bring up e2e test cluster #85836
Conversation
Hi @YangLu1031. 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. 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. |
/cc pjh |
/ok-to-test |
/kind documentation |
@@ -95,6 +95,44 @@ To teardown the cluster run: | |||
PROJECT=${CLOUDSDK_CORE_PROJECT} KUBERNETES_SKIP_CONFIRM=y ./cluster/kube-down.sh | |||
``` | |||
|
|||
#### 2b. Create a Kubernetes end-to-end (E2E) test cluster |
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'm not sure that these headings are accurate:
#### 2a. Create a regular Kubernetes cluster
#### 2b. Create a Kubernetes end-to-end (E2E) test cluster
Is there any difference in the resulting clusters? Can e2e tests be run against clusters brought up using the 2a steps?
I'm wondering if this should instead be:
#### 2. Create a Kubernetes cluster
...explain that there are two ways that work, and explain what the difference between them is...
##### Using kube-up
##### Using kubetest
What do you think?
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.
Agree, did some experiments, the main differences are
- Both e2e test clusters and e2e test scripts are setup using the config
cluster/gce/config-test.sh
, so the test scripts look for the e2e cluster only and run tests. - Only e2e test clusters enables SSH on windows nodes. Regular clusters are not. So couldn't dump logs thru scp on regular cluster windows node.
Added explanations.
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.
Also the differences between kubetest --up
& ./hack/e2e-internal/e2e-up.sh
is that kubetest has extra step at the end: ./cluster/kubectl.sh --match-server-version=false version
which seems just update the kubectl binary. And the kubectl.sh will be deprecated soon from the file descriptions.
/test ci-kubernetes-e2e-windows-node-throughput |
/retest |
/test pull-kubernetes-e2e-windows-node-throughput |
/retest |
/skip |
/test pull-kubernetes-e2e-windows-node-throughput |
/lgtm Thanks for checking on those things and updating the description. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: pjh, YangLu1031 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
@YangLu1031: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. 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. |
What type of PR is this?
/kind documentation
What this PR does / why we need it:
Add instructions about how to bring up e2e test cluster.
./cluster/kube-up.sh
can bring up a cluster but not work with e2e test script, instead./hack/e2e-internal/e2e-up.sh
works. kubetest also works, but due to some go module incompatible issues in kubetest, need to do some walkaround to update kubetest binary. Although kubetest can extract different versions of k8s binaries, but for local test, it seems not necessary. So./hack/e2e-internal/e2e-up.sh
should be the best way.Special notes for your reviewer:
Some discussions about build kubetest:
Azure/go-autorest#414
Azure/go-autorest#481
Does this PR introduce a user-facing change?: