Merge pull request #56 from mageddo/feature/55

#55 fixing the gap between the containers list and the events registr…

Author: mageddo

VERSION

@@ -1 +1 @@
-2.2.2
+2.2.3

events/docker/DockerEvents.go

@@ -40,32 +40,32 @@ func HandleDockerEvents(){
 		return
 	}
 
-	for _, c := range containers {
-
-		cInspection, err := cli.ContainerInspect(ctx, c.ID)
-		logger.Infof("status=container-from-list-begin, container=%s", cInspection.Name)
-		if err != nil {
-			logger.Errorf("status=inspect-error-at-list, container=%s, err=%v", c.Names, err)
-		}
-		hostnames := getHostnames(cInspection)
-		putHostnames(defaultLogger, hostnames, cInspection)
-		logger.Infof("status=container-from-list-success, container=%s, hostnames=%s", cInspection.Name, hostnames)
-
-	}
-
 	// more about events here: http://docs-stage.docker.com/v1.10/engine/reference/commandline/events/
 	var eventFilter = filters.NewArgs()
 	eventFilter.Add("event", "start")
-
 	eventFilter.Add("event", "die")
 	eventFilter.Add("event", "stop")
 
+	// registering at events before get the list of actual containers, this way no one container will be missed #55
 	body, err := cli.Events(ctx, types.EventsOptions{ Filters: eventFilter })
 	if err != nil {
 		logger.Errorf("status=error-to-attach-at-events-handler, solver=docker, err=%v", err)
 		return
 	}
 
+	for _, c := range containers {
+
+		cInspection, err := cli.ContainerInspect(ctx, c.ID)
+		logger.Infof("status=container-from-list-begin, container=%s", cInspection.Name)
+		if err != nil {
+			logger.Errorf("status=inspect-error-at-list, container=%s, err=%v", c.Names, err)
+		}
+		hostnames := getHostnames(cInspection)
+		putHostnames(defaultLogger, hostnames, cInspection)
+		logger.Infof("status=container-from-list-success, container=%s, hostnames=%s", cInspection.Name, hostnames)
+
+	}
+	
 	dec := json.NewDecoder(body)
 	for {