Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

Various fixes to circleci integration tests #797

Merged
merged 4 commits into from
May 29, 2015
Merged

Various fixes to circleci integration tests #797

merged 4 commits into from
May 29, 2015

Conversation

tomwilkie
Copy link
Contributor

  • Build the weave build container if we don't have it in the circle cache. That way, when it changes, we just flush the circle cache. It can be removed from the docker hub now.
  • Add some timeouts to the test runs and GCE destroy vm function; this should help prevent stranded VMs.
  • Checkin assert.sh instead of fetching it each time. This allows us to modify it to expose test run time. Its LGPL, so it should be fine.
  • Add a simple scheduler to decide what tests run on which shard in circle. This should balance tests more accurately, and bring down total test run time down by about a minute or two.

@paulbellamy
Copy link
Contributor

Maybe a moving average, instead of all of history?

@tomwilkie
Copy link
Contributor Author

Update to use an exponential moving average.

@paulbellamy
Copy link
Contributor

Do we need total_runs anymore?

@tomwilkie
Copy link
Contributor Author

I do not, but I thought it would be nice to keep.

@paulbellamy
Copy link
Contributor

LGTM, then.

@tomwilkie
Copy link
Contributor Author

Tests on the different shards took 03:13 and 03:02 - yay for scheduler!

tomwilkie added a commit that referenced this pull request May 29, 2015
Various fixes to circleci integration tests
@tomwilkie tomwilkie merged commit 6e2bb07 into master May 29, 2015
@tomwilkie tomwilkie deleted the timeout branch May 29, 2015 12:57
@rade rade modified the milestone: next May 31, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants