fix: make network selection deterministic (prefer bridge over random)

commit: fe25b4f885f2034b0647aa89f9686735190f994b
author: brice · 9mo ago · 2026-01-17 19:32 UTC
parent: 4912d47
signature: GPG VerifiedCD88F9E564E0102646FB6077A1A07C7C7C3E349D · arch-gpg Verified for the committer email
refs: tag: v2.6.0

1 file · +12 -1

M internal/adapters/out/docker/runtime.go +12 -1


@@ -766,17 +766,28 @@ func (r *Runtime) GetContainerNetwork(ctx context.Context, containerID string) (
766	766		return "bridge", nil
767	767		}
768	768
	769	+	// Priority: gordon-* networks first, then bridge, then any other
769	770		selected := ""
	771	+	hasBridge := false
770	772		for name := range networks.Networks {
771	773		if strings.HasPrefix(name, "gordon-") {
772	774		selected = name
773	775		break
774	776		}
775		-	if selected == "" {
	777	+	if name == "bridge" {
	778	+	hasBridge = true
	779	+	} else if selected == "" {
776	780		selected = name
777	781		}
778	782		}
779	783
	784	+	// If no gordon-* network found, prefer bridge over random network
	785	+	if selected == "" \|\| (hasBridge && !strings.HasPrefix(selected, "gordon-")) {
	786	+	if hasBridge {
	787	+	selected = "bridge"
	788	+	}
	789	+	}
	790	+
780	791		if selected == "" {
781	792		return "bridge", nil
782	793		}

 		return "bridge", nil
 	}
+	// Priority: gordon-* networks first, then bridge, then any other
 	selected := ""
+	hasBridge := false
 	for name := range networks.Networks {
 		if strings.HasPrefix(name, "gordon-") {
 			selected = name
 			break
 		}
-		if selected == "" {
+		if name == "bridge" {
+			hasBridge = true
+		} else if selected == "" {
 			selected = name
 		}
 	}
+	// If no gordon-* network found, prefer bridge over random network
+	if selected == "" || (hasBridge && !strings.HasPrefix(selected, "gordon-")) {
+		if hasBridge {
+			selected = "bridge"
+		}
+	}
 	if selected == "" {
 		return "bridge", nil
 	}