From f7735d629bb3c48dc69b8f48d88068891aca8f5a Mon Sep 17 00:00:00 2001
From: Pierre Wacrenier <pierre.wacrenier@gmail.com>
Date: Tue, 13 Jan 2015 01:57:29 +0100
Subject: [PATCH 1/3] More canonical Event construction on node event handling

Signed-off-by: Pierre Wacrenier <pierre.wacrenier@gmail.com>
---
 cluster/node.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/cluster/node.go b/cluster/node.go
index ccc4eba703..13919d7808 100644
--- a/cluster/node.go
+++ b/cluster/node.go
@@ -376,9 +376,9 @@ func (n *Node) handler(ev *dockerclient.Event, args ...interface{}) {
 	}
 
 	event := &Event{
-		Node: n,
+		Node:  n,
+		Event: *ev,
 	}
-	event.Event = *ev
 
 	n.eventHandler.Handle(event)
 }

From 044b33275945199d57ae427ef333f1f393e17b2f Mon Sep 17 00:00:00 2001
From: Pierre Wacrenier <pierre.wacrenier@gmail.com>
Date: Tue, 13 Jan 2015 01:59:12 +0100
Subject: [PATCH 2/3] createRouter doesn't yield errors

Signed-off-by: Pierre Wacrenier <pierre.wacrenier@gmail.com>
---
 api/api.go      | 4 ++--
 api/api_test.go | 5 +----
 api/server.go   | 5 +----
 3 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/api/api.go b/api/api.go
index 4839e7bc63..8aecd14350 100644
--- a/api/api.go
+++ b/api/api.go
@@ -273,7 +273,7 @@ func httpError(w http.ResponseWriter, err string, status int) {
 	http.Error(w, err, status)
 }
 
-func createRouter(c *context, enableCors bool) (*mux.Router, error) {
+func createRouter(c *context, enableCors bool) *mux.Router {
 	r := mux.NewRouter()
 	m := map[string]map[string]handler{
 		"GET": {
@@ -352,5 +352,5 @@ func createRouter(c *context, enableCors bool) (*mux.Router, error) {
 		}
 	}
 
-	return r, nil
+	return r
 }
diff --git a/api/api_test.go b/api/api_test.go
index b24f1c6cb8..c453898bc8 100644
--- a/api/api_test.go
+++ b/api/api_test.go
@@ -18,10 +18,7 @@ func serveRequest(c *cluster.Cluster, s *scheduler.Scheduler, w http.ResponseWri
 		version:   "test-version",
 	}
 
-	r, err := createRouter(context, false)
-	if err != nil {
-		return err
-	}
+	r := createRouter(context, false)
 	r.ServeHTTP(w, req)
 	return nil
 }
diff --git a/api/server.go b/api/server.go
index 7a3b44f552..b3df453a5f 100644
--- a/api/server.go
+++ b/api/server.go
@@ -38,10 +38,7 @@ func ListenAndServe(c *cluster.Cluster, s *scheduler.Scheduler, hosts []string,
 		tlsConfig:     tlsConfig,
 	}
 	c.Events(context.eventsHandler)
-	r, err := createRouter(context, enableCors)
-	if err != nil {
-		return err
-	}
+	r := createRouter(context, enableCors)
 	chErrors := make(chan error, len(hosts))
 
 	for _, host := range hosts {

From 4cd7a4e8e36290d1b919fd142ee33673730cfd72 Mon Sep 17 00:00:00 2001
From: Pierre Wacrenier <pierre.wacrenier@gmail.com>
Date: Tue, 13 Jan 2015 02:09:44 +0100
Subject: [PATCH 3/3] More canonical Container creation in inspectContainer

Signed-off-by: Pierre Wacrenier <pierre.wacrenier@gmail.com>
---
 cluster/node.go | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/cluster/node.go b/cluster/node.go
index 13919d7808..ed5410de53 100644
--- a/cluster/node.go
+++ b/cluster/node.go
@@ -183,9 +183,10 @@ func (n *Node) ForceRefreshContainer(c dockerclient.Container) error {
 
 func (n *Node) inspectContainer(c dockerclient.Container, containers map[string]*Container, lock bool) error {
 
-	container := &Container{}
-	container.Container = c
-	container.Node = n
+	container := &Container{
+		Container: c,
+		Node:      n,
+	}
 
 	info, err := n.client.InspectContainer(c.Id)
 	if err != nil {