ensureServiceIsConnectedToCloudSql only need to call backend when schemaValidation==NONE

src/dataconnect/schemaMigration.ts

@@ -26,7 +26,7 @@
 import { logLabeledBullet, logLabeledWarning, logLabeledSuccess } from "../utils";
 import * as errors from "./errors";
 
 export async function diffSchema(
   schema: Schema,
   schemaValidation?: SchemaValidation,
 ): Promise<Diff[]> {
@@ -53,8 +53,8 @@
     } else {
       logLabeledSuccess("dataconnect", `Database schema is compatible.`);
     }
   } catch (err: any) {
     if (err?.status !== 400) {
       throw err;
     }
     const invalidConnectors = errors.getInvalidConnectors(err);
@@ -82,8 +82,8 @@
       logLabeledBullet("dataconnect", `generating schema changes, including optional changes...`);
       await upsertSchema(schema, /** validateOnly=*/ true);
       logLabeledSuccess("dataconnect", `no additional optional changes`);
     } catch (err: any) {
       if (err?.status !== 400) {
         throw err;
       }
       const incompatible = errors.getIncompatibleSchemaError(err);
@@ -105,7 +105,7 @@
   return diffs;
 }
 
 export async function migrateSchema(args: {
   options: Options;
   schema: Schema;
   /** true for `dataconnect:sql:migrate`, false for `deploy` */
@@ -130,8 +130,8 @@
   try {
     await upsertSchema(schema, validateOnly);
     logger.debug(`Database schema was up to date for ${instanceId}:${databaseId}`);
   } catch (err: any) {
     if (err?.status !== 400) {
       throw err;
     }
     // Parse and handle failed precondition errors, then retry.
@@ -183,8 +183,8 @@
     setSchemaValidationMode(schema, validationMode);
     try {
       await upsertSchema(schema, validateOnly);
     } catch (err: any) {
       if (err.status !== 400) {
         throw err;
       }
       // Parse and handle failed precondition errors, then retry.
@@ -543,6 +543,7 @@
         {
           postgresql: {
             database: databaseId,
+            schemaValidation: "NONE",
             cloudSql: {
               instance: instanceId,
             },
@@ -566,7 +567,8 @@
       `Switching connected Postgres database from ${postgresql?.database} to ${databaseId}`,
     );
   }
-  if (!postgresql || postgresql.schemaValidation === "STRICT") {
+  if (!postgresql || postgresql.schemaValidation !== "NONE") {
+    // Skip provisioning connectvity if it is already connected.
     return;
   }
   postgresql.schemaValidation = "STRICT";