use implicit DOCKER_HOST instead

Author: nicktrn

apps/supervisor/.env.example

@@ -7,7 +7,7 @@ MANAGED_WORKER_SECRET=managed-secret
 # Point this at the webapp in prod
 TRIGGER_API_URL=http://localhost:3030
 
-# Point this at the OTel collector in prod
+# Point this at the webapp or an OTel collector in prod
 OTEL_EXPORTER_OTLP_ENDPOINT=http://localhost:3030/otel
 # Use this on macOS
 # OTEL_EXPORTER_OTLP_ENDPOINT=http://host.docker.internal:3030/otel

apps/supervisor/src/env.ts

@@ -44,7 +44,6 @@ const Env = z.object({
 
   // Used by the workload manager, e.g docker/k8s
   DOCKER_NETWORK: z.string().default("host"),
-  DOCKER_SOCKET_PATH: z.string().default("/var/run/docker.sock"),
   OTEL_EXPORTER_OTLP_ENDPOINT: z.string().url(),
   ENFORCE_MACHINE_PRESETS: z.coerce.boolean().default(false),
   KUBERNETES_IMAGE_PULL_SECRETS: z.string().optional(), // csv

apps/supervisor/src/workloadManager/docker.ts

@@ -13,9 +13,7 @@ export class DockerWorkloadManager implements WorkloadManager {
   private readonly docker: Docker;
 
   constructor(private opts: WorkloadManagerOptions) {
-    this.docker = new Docker({
-      socketPath: env.DOCKER_SOCKET_PATH,
-    });
+    this.docker = new Docker();
 
     if (opts.workloadApiDomain) {
       this.logger.warn("⚠️ Custom workload API domain", {