chore: no log messages for ipc feature

like jacob mentioned, logs regarding this feature would be confusing to
users. thanks to being able to check the state of iptables, as well as
agentd logs, we can pretty easily debug thwere this failed

Author: billyb2

pkg/machine/ipc_client.go

@@ -4,7 +4,6 @@ import (
 	"context"
 	"encoding/json"
 	"fmt"
-	"log"
 	"net"
 	"os"
 	"strings"
@@ -28,7 +27,6 @@ func AllowBuilderIPViaIPC(ctx context.Context, endpoint string) error {
 		if os.IsNotExist(err) {
 			return nil
 		}
-		log.Printf("failed to check IPC socket: %v\n", err)
 	}
 
 	client, err := ipc.StartClient("depot-agentd", &ipc.ClientConfig{
@@ -37,7 +35,6 @@ func AllowBuilderIPViaIPC(ctx context.Context, endpoint string) error {
 	if err != nil {
 		// If we can't connect to the IPC server, log it but don't fail the build
 		// The IPC server only runs if the egress filter is enabled
-		log.Println("failed to connect to IPC server", "error", err)
 		return nil
 	}
 	defer client.Close()
@@ -93,7 +90,6 @@ connected:
 				}
 
 				if success, ok := response["success"].(bool); ok && success {
-					log.Printf("Successfully added builder IP %s to allowlist", ip)
 					return nil
 				}
 

pkg/machine/machine.go

@@ -85,10 +85,8 @@ func Acquire(ctx context.Context, buildID, token, platform string) (*Machine, er
 
 			// If running in Depot GHA environment, allowlist the builder IP via IPC
 			if helpers.IsDepotGitHubActionsRunner() {
-				if err := AllowBuilderIPViaIPC(ctx, m.Addr); err != nil {
-					// Log the error but don't fail the build
-					log.Printf("Warning: failed to allowlist builder IP via IPC: %v", err)
-				}
+				// if this is failing, we can check by ssh'ing into the machine
+				_ = AllowBuilderIPViaIPC(ctx, m.Addr)
 			}
 
 			// When testing locally, we don't have TLS certs.