Force the DNS check to timeout after 70 seconds (#3647)

This should fix an issue that Matthew has noticed where DNS checks could hang indefinetely
superfly · Jun 18, 2024 · 37221f6 · 37221f6
1 parent 9d47fb7
commit 37221f6
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/internal/command/deploy/machines_deploymachinesapp.go b/internal/command/deploy/machines_deploymachinesapp.go
@@ -1072,6 +1072,8 @@ func (md *machineDeployment) doSmokeChecks(ctx context.Context, lm machine.Leasa
 func (md *machineDeployment) checkDNS(ctx context.Context) error {
 	ctx, span := tracing.GetTracer().Start(ctx, "check_dns")
 	defer span.End()
+	ctx, cancel := context.WithTimeout(ctx, time.Second*70)
+	defer cancel()
 
 	client := flyutil.ClientFromContext(ctx)
 	ipAddrs, err := client.GetIPAddresses(ctx, md.appConfig.AppName)