Always check for Docker when using containers

Even in dry run mode we should check that Docker is installed, because
dry run mode uses docker commands too.

Author: gpanders

packages/wrangler/src/deploy/deploy.ts

@@ -770,6 +770,23 @@ See https://developers.cloudflare.com/workers/platform/compatibility-dates for m
 		let workerBundle: FormData;
 		const dockerPath = getDockerPath();
 
+		// lets fail earlier in the case where docker isn't installed
+		// and we have containers so that we don't get into a
+		// disjointed state where the worker updates but the container
+		// fails.
+		if (config.containers) {
+			// if you have a registry url specified, you don't need docker
+			const hasDockerfiles = config.containers.some((container) =>
+				isDockerfile(
+					container.image ?? container.configuration?.image,
+					config.configPath
+				)
+			);
+			if (hasDockerfiles) {
+				await verifyDockerInstalled(dockerPath, false);
+			}
+		}
+
 		if (props.dryRun) {
 			if (config.containers) {
 				for (const container of config.containers) {
@@ -792,23 +809,6 @@ See https://developers.cloudflare.com/workers/platform/compatibility-dates for m
 		} else {
 			assert(accountId, "Missing accountId");
 
-			// lets fail earlier in the case where docker isn't installed
-			// and we have containers so that we don't get into a
-			// disjointed state where the worker updates but the container
-			// fails.
-			if (config.containers) {
-				// if you have a registry url specified, you don't need docker
-				const hasDockerfiles = config.containers?.some((container) =>
-					isDockerfile(
-						container.image ?? container.configuration?.image,
-						config.configPath
-					)
-				);
-				if (hasDockerfiles) {
-					await verifyDockerInstalled(dockerPath, false);
-				}
-			}
-
 			if (getFlag("RESOURCES_PROVISION")) {
 				await provisionBindings(
 					bindings,