Skip to content

Commit 90aa9c3

Browse files
committedJun 20, 2014
docker integration passes end-to-end test
1 parent b8088f5 commit 90aa9c3

File tree

4 files changed

+8
-7
lines changed

4 files changed

+8
-7
lines changed
 

‎cmd/circuit/start.go

+3-1
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,11 @@ func server(c *cli.Context) {
3030
var err error
3131

3232
if c.Bool("docker") {
33-
if err = docker.Init(); err != nil {
33+
cmd, err := docker.Init()
34+
if err != nil {
3435
log.Fatalf("cannot use docker: %v", err)
3536
}
37+
log.Printf("Enabling docker elements, using %s", cmd)
3638
}
3739
// parse arguments
3840
var tcpaddr = parseAddr(c) // server bind address

‎element/docker/docker_test.go ‎element/docker/_docker_test.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import (
1515
)
1616

1717
func TestDocker(t *testing.T) {
18-
if err := Init(); err != nil {
18+
if _, err := Init(); err != nil {
1919
t.Fatalf("init: %v", err)
2020
}
2121
run := ds.Run{

‎element/docker/docker.go

-2
Original file line numberDiff line numberDiff line change
@@ -77,12 +77,10 @@ func (con *container) Wait() (_ *ds.Stat, err error) {
7777
}
7878

7979
func (con *container) Stdin() io.WriteCloser {
80-
println("*stdin")
8180
return con.stdin
8281
}
8382

8483
func (con *container) Stdout() io.ReadCloser {
85-
println("*stdout")
8684
return con.stdout
8785
}
8886

‎element/docker/util.go

+4-3
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,13 @@ import (
1111
"os/exec"
1212
)
1313

14-
func Init() (err error) {
14+
func Init() (_ string, err error) {
1515
dkr, err = exec.LookPath("docker")
1616
if err != nil {
17-
return err
17+
return "", err
1818
}
19-
return exec.Command("dkr", "version").Run()
19+
err = exec.Command(dkr, "version").Run()
20+
return dkr, err
2021
}
2122

2223
var dkr string

0 commit comments

Comments
 (0)
Please sign in to comment.