Skip to content

Libkermit integration 😇#264

Merged
emilevauge merged 2 commits intotraefik:masterfrom
vdemeester:libkermit-integration
Mar 28, 2016
Merged

Libkermit integration 😇#264
emilevauge merged 2 commits intotraefik:masterfrom
vdemeester:libkermit-integration

Conversation

@vdemeester
Copy link
Copy Markdown
Contributor

This changes the integration tests to use libkermit. This is a start, libkermit will get more feature in over time and traefik will benefit from it first 😇.

  • It simplifies writing integration tests.
  • It uses docker/engine API, and this is the first step to migrate traefik to use engine-api as well — will come soon.

An open question (completely related to libkermit) is : should compose.Project embedded docker.Project and thus let it have all the docker.Project methods ? (this would mean, no need for the project attribute on DockerSuite and ConsulCatalogSuite for example).

🐸

Using the compose package for starting and stopping project.
@emilevauge
Copy link
Copy Markdown
Member

Well done @vdemeester ! libkermit rocks!
LGTM

@emilevauge emilevauge merged commit 6308ce2 into traefik:master Mar 28, 2016
@vdemeester vdemeester deleted the libkermit-integration branch March 28, 2016 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants