Skip to content

Commit

Permalink
Merge pull request spring-projects#5059 from Vedran Pavić
Browse files Browse the repository at this point in the history
* spring-projectsgh-5059:
  Remove Docker container after launch script test execution
  • Loading branch information
wilkinsona committed Feb 8, 2016
2 parents 29c7b93 + 6540b1f commit a1fefb1
Showing 1 changed file with 23 additions and 18 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -206,24 +206,29 @@ private String doTest(String script) throws Exception {
DockerClient docker = createClient();
String imageId = buildImage(docker);
String container = createContainer(docker, imageId, script);
copyFilesToContainer(docker, container, script);
docker.startContainerCmd(container).exec();
StringBuilder output = new StringBuilder();
AttachContainerResultCallback resultCallback = docker
.attachContainerCmd(container).withStdOut(true).withStdErr(true)
.withFollowStream(true).withLogs(true)
.exec(new AttachContainerResultCallback() {

@Override
public void onNext(Frame item) {
output.append(new String(item.getPayload()));
super.onNext(item);
}

});
resultCallback.awaitCompletion(60, TimeUnit.SECONDS).close();
docker.waitContainerCmd(container).exec();
return output.toString();
try {
copyFilesToContainer(docker, container, script);
docker.startContainerCmd(container).exec();
StringBuilder output = new StringBuilder();
AttachContainerResultCallback resultCallback = docker
.attachContainerCmd(container).withStdOut(true).withStdErr(true)
.withFollowStream(true).withLogs(true)
.exec(new AttachContainerResultCallback() {

@Override
public void onNext(Frame item) {
output.append(new String(item.getPayload()));
super.onNext(item);
}

});
resultCallback.awaitCompletion(60, TimeUnit.SECONDS).close();
docker.waitContainerCmd(container).exec();
return output.toString();
}
finally {
docker.removeContainerCmd(container).exec();
}
}

private DockerClient createClient() {
Expand Down

0 comments on commit a1fefb1

Please sign in to comment.