fix: return error first before accessing fields that might be nil in …

…extractLogsFromVM (Azure#4855)

Co-authored-by: Mikolaj Umanski <[email protected]>

e2e/exec.go

@@ -71,6 +71,11 @@ func extractLogsFromVM(ctx context.Context, t *testing.T, vmssName, privateIP, s
 		t.Logf("executing command on remote VM at %s of VMSS %s: %q", privateIP, vmssName, sourceCmd)
 
 		execResult, err := execOnVM(ctx, opts.clusterConfig.Kube, privateIP, podName, sshPrivateKey, sourceCmd, false)
+		if err != nil {
+			t.Logf("error executing command on remote VM at %s of VMSS %s: %s", privateIP, vmssName, err)
+			return nil, err
+		}
+
 		if execResult.stdout != nil {
 			out := execResult.stdout.String()
 			if out != "" {
@@ -84,10 +89,6 @@ func extractLogsFromVM(ctx context.Context, t *testing.T, vmssName, privateIP, s
 				result[file+".stderr.txt"] = out
 			}
 		}
-
-		if err != nil {
-			t.Logf("error executing command on remote VM at %s of VMSS %s: %s", privateIP, vmssName, err)
-		}
 	}
 	return result, nil
 }