Skip to content
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 tests should allow multiple concurrent runs #3132

Closed
roberthbailey opened this issue Dec 23, 2014 · 7 comments
Closed

e2e tests should allow multiple concurrent runs #3132

roberthbailey opened this issue Dec 23, 2014 · 7 comments
Labels
area/reliability area/test-infra priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done.

Comments

@roberthbailey
Copy link
Contributor

Refactor the e2e tests to allow multiple concurrent runs (assuming it is supported by the cloud provider). For GCE, this will require that the kubectl client can authenticate to multiple clusters (#1755). For GKE, this is already supported in gcloud.

@roberthbailey roberthbailey added priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. area/reliability labels Dec 23, 2014
@zmerlynn
Copy link
Member

Actually, you can mostly do this on gce in a ghetto way by bonking HOME and
overriding the parameters I added. But I need to add the CIDR to that set,
and do more testing.

On Tue, Dec 23, 2014, 13:19 roberthbailey notifications@github.com wrote:

Refactor the e2e tests to allow multiple concurrent runs (assuming it is
supported by the cloud provider). For GCE, this will require that the
kubectl client can authenticate to multiple clusters (#1755
#1755). For
GKE, this is already supported in gcloud.


Reply to this email directly or view it on GitHub
#3132.

@lavalamp
Copy link
Member

Can you clarify? You want to stand up multiple e2e clusters and run a complete set of tests on each cluster, right? That's already possible in the way @zmerlynn says, and will be less hacky in the future when we solve the general problem of keeping auth to multiple clusters around.

Another possibility is to just run all the tests in parallel. The downside to this is that they might interfere with each other, and that it might make failures harder to understand. But I think that it might be worth it for the extra speed.

@zmerlynn
Copy link
Member

I believe the intent of this bug came out of my angst over things like
Jenkins being able to do multiple things in parallel, since it's a limiter
right now.

On Tue, Dec 23, 2014 at 3:00 PM, Daniel Smith notifications@github.com
wrote:

Can you clarify? You want to stand up multiple e2e clusters and run a
complete set of tests on each cluster, right? That's already possible in
the way @zmerlynn https://github.com/zmerlynn says, and will be less
hacky in the future when we solve the general problem of keeping auth to
multiple clusters around.

Another possibility is to just run all the tests in parallel. The downside
to this is that they might interfere with each other, and that it might
make failures harder to understand. But I think that it might be worth it
for the extra speed.


Reply to this email directly or view it on GitHub
#3132 (comment)
.

@roberthbailey
Copy link
Contributor Author

It would also increase our confidence in a release if we had multiple e2e test runs passing for the release (since that will help detect flakes in the tests as well as the code). Being able to parallelize a bunch of e2e tests (e.g. ./run100e2etests.sh ) would be nice.

@zmerlynn
Copy link
Member

I actually intend to setup a --times=10 long job on Jenkins, but the
stability of the main e2e jobs is ... lacking. With the half hour drumbeat,
it's pretty easy to see how bad we're doing today. :( But yes, we will need
to parallelize after we do better.

On Tue, Dec 23, 2014 at 3:20 PM, roberthbailey notifications@github.com
wrote:

It would also increase our confidence in a release if we had multiple e2e
test runs passing for the release (since that will help detect flakes in
the tests as well as the code). Being able to parallelize a bunch of e2e
tests (e.g. ./run100e2etests.sh ) would be nice.


Reply to this email directly or view it on GitHub
#3132 (comment)
.

@roberthbailey roberthbailey added priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done. area/test-infra and removed priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels Feb 11, 2015
@ikehz
Copy link
Contributor

ikehz commented Sep 2, 2015

We now have PARALLEL test suites in hack/jenkins/e2e.sh.

Proposal to close this issue in favor of a work-issue to track moving as many e2es into the parallel suite as possible.

@zmerlynn
Copy link
Member

zmerlynn commented Sep 8, 2015

Yup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/reliability area/test-infra priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done.
Projects
None yet
Development

No branches or pull requests

4 participants