Fix: filesystem e2e flakiness (GoogleCloudPlatform#8362)

rogerthatdev · leahecole · web-flow · commit a18c13952a05 · 2022-09-28T11:32:27.000-04:00
* cloudbuild  docker build step

* docker push step to AR

* step 3 deploy to cloud run

* use dynamic substitution

* formatting

* happy new year

* alpha =&gt; beta

* Update run/filesystem/cloudbuild.yaml

Co-authored-by: Leah E. Cole &lt;6719667+leahecole@users.noreply.github.com&gt;
diff --git a/run/filesystem/cloudbuild.yaml b/run/filesystem/cloudbuild.yaml
@@ -10,23 +10,31 @@
 # limitations under the License.
 
 steps:
+  - id: "Docker build"
+    name: "gcr.io/cloud-builders/docker"
+    args:
+      ["build", "-t", "${_AR_REPO_URL}/filesystem-e2e-test:${BUILD_ID}", "."]
+  - id: "Docker push to Artifact Registry"
+    name: "gcr.io/cloud-builders/docker"
+    args: ["push", "${_AR_REPO_URL}/filesystem-e2e-test:${BUILD_ID}"]
   - id: "Deploy to Cloud Run"
     name: "gcr.io/cloud-builders/gcloud:latest"
-    entrypoint: /bin/bash
     args:
-      - "-c"
-      - |
-        ./retry.sh "gcloud alpha run deploy ${_SERVICE_NAME} \
-          --source . \
-          --execution-environment gen2 \
-          --vpc-connector ${_CONNECTOR} \
-          --update-env-vars FILESTORE_IP_ADDRESS=${_FILESTORE_IP_ADDRESS},FILE_SHARE_NAME=${_FILESHARE} \
-          --region ${_DEPLOY_REGION} \
-          --no-allow-unauthenticated"
+      - beta
+      - run
+      - deploy
+      - ${_SERVICE_NAME}
+      - --image=${_AR_REPO_URL}/filesystem-e2e-test:${BUILD_ID}
+      - --execution-environment=gen2
+      - --vpc-connector=${_CONNECTOR}
+      - --update-env-vars=FILESTORE_IP_ADDRESS=${_FILESTORE_IP_ADDRESS},FILE_SHARE_NAME=${_FILESHARE}
+      - --region=${_DEPLOY_REGION}
+      - --no-allow-unauthenticated
 
 substitutions:
-  _SERVICE_NAME: filesystem
   _DEPLOY_REGION: us-central1
   _FILESHARE: vol1
-  _CONNECTOR: my-run-connector
-  _FILESTORE_IP_ADDRESS: 0.0.0.0
+  _AR_REPO_URL: us-central1-docker.pkg.dev/${PROJECT_ID}/cloud-run-source-deploy
+
+options:
+  dynamic_substitutions: true
