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

Test flake due to CPU requested in Travis CI #890

Closed
micw523 opened this issue Jul 23, 2019 · 5 comments
Closed

Test flake due to CPU requested in Travis CI #890

micw523 opened this issue Jul 23, 2019 · 5 comments
Assignees
Labels
kind/flake Categorizes issue or PR as related to a flaky test.

Comments

@micw523
Copy link
Contributor

micw523 commented Jul 23, 2019

See my personal build here.

It seems that the default config requests 2 CPUs but sometimes Travis CI only provides 1. This behavior causes Minikube to crash and the py*-functional test would fail. We should request 1 CPU from minikube instead.

@roycaihw roycaihw added the kind/flake Categorizes issue or PR as related to a flaky test. label Jul 23, 2019
@roycaihw
Copy link
Member

/assign

@roycaihw
Copy link
Member

minikube cpu usage is configurable using the command line flag. I haven't tested if it can work with 1 cpu

Before that, 2 cpus doesn't sound like a huge value and I'm surprised that travis ci cannot guarantee that. My assumption is that workers running in parallel can share the cpus, and most of the time we only have one job running. I wonder if we can configure the worker pool to ensure the cpus assigned to each job: https://docs.travis-ci.com/user/enterprise/worker-configuration/#configuring-the-number-of-concurrent-jobs

/assign @micw523

@micw523
Copy link
Contributor Author

micw523 commented Jul 30, 2019

Sounds good. I'll do some investigations when I have time. Also let me check some of our earlier builds to see what our configurations were.

@micw523
Copy link
Contributor Author

micw523 commented Jul 30, 2019

Our previous builds were all configured for using 2 CPUs. I wonder if Travis CI had put some quota system in place.

@micw523
Copy link
Contributor Author

micw523 commented Jul 31, 2019

See #906 - failure attributed to upstream Travis CI

@micw523 micw523 closed this as completed Jul 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/flake Categorizes issue or PR as related to a flaky test.
Projects
None yet
Development

No branches or pull requests

2 participants