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

Run rethink integration tests #756

Merged
merged 4 commits into from
May 26, 2016
Merged

Conversation

cyli
Copy link
Contributor

@cyli cyli commented May 24, 2016

Bump circle's docker version up to 1.10 so we can use compose v2 files. Attempting to run rethink tests as part of our integration tests, but still trying to figure out what's going wrong.

We are encountering docker/compose#3432 on CircleCI.

@cyli cyli changed the title Run rethink integration tests WIP: run rethink integration tests May 24, 2016
@cyli cyli force-pushed the run-rethink-integration-tests branch 3 times, most recently from f33e939 to 2556429 Compare May 24, 2016 23:59
cyli added 2 commits May 24, 2016 17:31
…eplica.

Fix the compose file so that 2 and 3 join 1, otherwise the servers didn't
actually seem to be clustered.

Signed-off-by: Ying Li <[email protected]>
@cyli cyli force-pushed the run-rethink-integration-tests branch 2 times, most recently from 7ee4ce4 to 8dfcc96 Compare May 25, 2016 01:01
@cyli cyli force-pushed the run-rethink-integration-tests branch from 8dfcc96 to effa539 Compare May 25, 2016 01:13
@cyli cyli changed the title WIP: run rethink integration tests Run rethink integration tests May 25, 2016
@cyli cyli force-pushed the run-rethink-integration-tests branch 11 times, most recently from a7fc0de to 6a8734b Compare May 25, 2016 20:06
@cyli cyli force-pushed the run-rethink-integration-tests branch from 6a8734b to c7fd4d8 Compare May 25, 2016 23:01
@endophage
Copy link
Contributor

LGTM

@endophage endophage merged commit de2c51d into master May 26, 2016
@cyli cyli deleted the run-rethink-integration-tests branch May 26, 2016 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants