From dd385c7a01a740711d62fb6c8866e2fc4d267adb Mon Sep 17 00:00:00 2001
From: Kegan Dougal <7190048+kegsay@users.noreply.github.com>
Date: Fri, 4 Apr 2025 11:09:36 +0100
Subject: [PATCH] Don't PublishAllPorts as we already explicitly do :8008/8448

This may break use cases which want extra ports to be exposed.
This may fix issues where Complement sporadically decides that
there are port conflicts, causing CI flakes.
---
 internal/docker/deployer.go | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/internal/docker/deployer.go b/internal/docker/deployer.go
index 2dbe4d98..6507aff5 100644
--- a/internal/docker/deployer.go
+++ b/internal/docker/deployer.go
@@ -380,8 +380,7 @@ func deployImage(
 			"complement_hs_name":   hsName,
 		},
 	}, &container.HostConfig{
-		CapAdd:          []string{"NET_ADMIN"}, // TODO : this should be some sort of option
-		PublishAllPorts: true,
+		CapAdd: []string{"NET_ADMIN"}, // TODO : this should be some sort of option
 		PortBindings: nat.PortMap{
 			nat.Port("8008/tcp"): []nat.PortBinding{
 				{