Skip to content
This repository was archived by the owner on Aug 27, 2025. It is now read-only.

Conversation

@springmeyer
Copy link
Contributor

@springmeyer springmeyer commented Jan 6, 2018

This adds support to mapbox-tile-copy for running test on circle ci. Their are several motivations for this PR:

  • Travis has been flaky recently, not only due to some flapping tests (Tests are horribly blippy #77) but also several types of travis failures to launch.
  • I wanted to learn circleci
  • I've heard circleci is faster

This tests mapbox-tile-copy 3 ways:

  • On OSX
  • On ubuntu
  • Inside docker, also running ubuntu

Compared to travis:

  • Builds start instantly
  • Jobs run in 3-3.5 minutes, which is about a 1 minute faster than avg travis jobs

/cc @kkaefer for review
/cc @GretaCB for visibility

@springmeyer
Copy link
Contributor Author

How appropriate, this PR is failing on travis with:

Err:22 http://ppa.launchpad.net/chris-lea/redis-server/ubuntu trusty/main amd64 Packages
  Unable to connect to ppa.launchpad.net:http

Going to merge and run them together for a bit, then plan to disable travis soon.

@springmeyer springmeyer merged commit d735937 into master Jan 6, 2018
@springmeyer springmeyer deleted the circleci branch January 6, 2018 02:14
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.

1 participant