Merge pull request #241 from vishh/dns_lookup

Avoid compiling cadvisor statically for the docker image

Author: vmarmol

cadvisor.go

@@ -93,16 +93,23 @@ func main() {
 
 	defer glog.Flush()
 
+	errChan := make(chan error)
 	go func() {
-		glog.Fatal(containerManager.Start())
+		errChan <- containerManager.Start()
 	}()
 
 	glog.Infof("Starting cAdvisor version: %q", info.VERSION)
 	glog.Infof("About to serve on port %d", *argPort)
 
 	addr := fmt.Sprintf(":%v", *argPort)
 
-	glog.Fatal(http.ListenAndServe(addr, nil))
+	go func() {
+		errChan <- http.ListenAndServe(addr, nil)
+	}()
+	select {
+	case err := <-errChan:
+		glog.Fatal(err)
+	}
 }
 
 func setMaxProcs() {

deploy/Dockerfile

@@ -1,11 +1,8 @@
-FROM scratch
-MAINTAINER [email protected] [email protected]
-
-# NOTE: Run prepare.sh before building this Docker image.
+FROM progrium/busybox
+MAINTAINER [email protected] [email protected] [email protected]
 
 # Grab cadvisor from the staging directory.
 ADD cadvisor /usr/bin/cadvisor
 
 EXPOSE 8080
-ENTRYPOINT ["/usr/bin/cadvisor"] 
-CMD ["-log_dir", "/"]
+ENTRYPOINT ["/usr/bin/cadvisor"]

deploy/build.sh

@@ -0,0 +1,8 @@
+#!/bin/bash
+
+set -e
+set -x
+
+godep go build -a github.com/google/cadvisor
+
+sudo docker build -t google/cadvisor:test .

deploy/prepare.sh

@@ -19,6 +19,7 @@ import (
 	"fmt"
 	"sync"
 
+	"github.com/golang/glog"
 	"github.com/google/cadvisor/info"
 	"github.com/google/cadvisor/storage"
 )
@@ -104,7 +105,9 @@ func (self *InMemoryStorage) AddStats(ref info.ContainerReference, stats *info.C
 		// TODO(monnand): To deal with long delay write operations, we
 		// may want to start a pool of goroutines to do write
 		// operations.
-		self.backend.AddStats(ref, stats)
+		if err := self.backend.AddStats(ref, stats); err != nil {
+			glog.Error(err)
+		}
 	}
 	return cstore.AddStats(stats)
 }