Make mount order predictable - fixes docker tests

Author: kvaster

internal/services/executor/driver/docker.go

@@ -249,6 +249,8 @@ func (d *DockerDriver) NewPod(ctx context.Context, podConfig *PodConfig, out io.
 			}
 		}
 
+		sort.Sort(MountSlice(container.Mounts))
+
 		seenIndexes[cIndex] = struct{}{}
 		count++
 	}
@@ -468,7 +470,7 @@ func (d *DockerDriver) GetPods(ctx context.Context, all bool) ([]Pod, error) {
 			}
 		}
 
-		// overwrite containers with the right order
+		sort.Sort(MountSlice(container.Mounts))
 
 		// add labels from the container with index 0
 		if cIndex == 0 {
@@ -736,3 +738,11 @@ func makeEnvSlice(env map[string]string) []string {
 
 	return envList
 }
+
+type MountSlice []dockertypes.MountPoint
+
+func (p MountSlice) Len() int { return len(p) }
+func (p MountSlice) Less(i, j int) bool {
+	return strings.Compare(p[i].Destination, p[j].Destination) < 0
+}
+func (p MountSlice) Swap(i, j int) { p[i], p[j] = p[j], p[i] }

internal/services/executor/driver/docker_test.go

@@ -17,7 +17,7 @@ package driver
 import (
 	"bytes"
 	"context"
-	"io/ioutil"
+	"io"
 	"os"
 	"testing"
 	"time"
@@ -64,7 +64,7 @@ func TestDockerPod(t *testing.T) {
 				},
 			},
 			InitVolumeDir: "/tmp/agola",
-		}, ioutil.Discard)
+		}, io.Discard)
 		if err != nil {
 			t.Fatalf("unexpected err: %v", err)
 		}
@@ -82,7 +82,7 @@ func TestDockerPod(t *testing.T) {
 				},
 			},
 			InitVolumeDir: "/tmp/agola",
-		}, ioutil.Discard)
+		}, io.Discard)
 		if err != nil {
 			t.Fatalf("unexpected err: %v", err)
 		}
@@ -121,7 +121,7 @@ func TestDockerPod(t *testing.T) {
 				},
 			},
 			InitVolumeDir: "/tmp/agola",
-		}, ioutil.Discard)
+		}, io.Discard)
 		if err != nil {
 			t.Fatalf("unexpected err: %v", err)
 		}
@@ -175,7 +175,7 @@ func TestDockerPod(t *testing.T) {
 				},
 			},
 			InitVolumeDir: "/tmp/agola",
-		}, ioutil.Discard)
+		}, io.Discard)
 		if err != nil {
 			t.Fatalf("unexpected err: %v", err)
 		}
@@ -196,7 +196,7 @@ func TestDockerPod(t *testing.T) {
 				},
 			},
 			InitVolumeDir: "/tmp/agola",
-		}, ioutil.Discard)
+		}, io.Discard)
 		if err != nil {
 			t.Fatalf("unexpected err: %v", err)
 		}
@@ -232,7 +232,7 @@ func TestDockerPod(t *testing.T) {
 				},
 			},
 			InitVolumeDir: "/tmp/agola",
-		}, ioutil.Discard)
+		}, io.Discard)
 		if err != nil {
 			t.Fatalf("unexpected err: %v", err)
 		}
@@ -278,7 +278,7 @@ func TestDockerPod(t *testing.T) {
 				},
 			},
 			InitVolumeDir: "/tmp/agola",
-		}, ioutil.Discard)
+		}, io.Discard)
 		if err != nil {
 			t.Fatalf("unexpected err: %v", err)
 		}
@@ -324,7 +324,7 @@ func TestDockerPod(t *testing.T) {
 				},
 			},
 			InitVolumeDir: "/tmp/agola",
-		}, ioutil.Discard)
+		}, io.Discard)
 		if err != nil {
 			t.Fatalf("unexpected err: %v", err)
 		}
@@ -382,7 +382,7 @@ func TestDockerPod(t *testing.T) {
 				},
 			},
 			InitVolumeDir: "/tmp/agola",
-		}, ioutil.Discard)
+		}, io.Discard)
 		if err != nil {
 			t.Fatalf("unexpected err: %v", err)
 		}
@@ -421,7 +421,7 @@ func TestDockerPod(t *testing.T) {
 				},
 			},
 			InitVolumeDir: "/tmp/agola",
-		}, ioutil.Discard)
+		}, io.Discard)
 		if err != nil {
 			t.Fatalf("unexpected err: %v", err)
 		}
@@ -468,7 +468,7 @@ func TestDockerPod(t *testing.T) {
 				},
 			},
 			InitVolumeDir: "/tmp/agola",
-		}, ioutil.Discard)
+		}, io.Discard)
 		if err != nil {
 			t.Fatalf("unexpected err: %v", err)
 		}
@@ -513,7 +513,7 @@ func TestDockerPod(t *testing.T) {
 				},
 			},
 			InitVolumeDir: "/tmp/agola",
-		}, ioutil.Discard)
+		}, io.Discard)
 		if err != nil {
 			t.Fatalf("unexpected err: %v", err)
 		}