(PC-33785)[API] ci: run sandbox only after helm chart is deployed

pass-culture · Jan 6, 2025 · bdb2d9f · bdb2d9f
1 parent d1d8ee1
commit bdb2d9f
Show file tree

Hide file tree

Showing 6 changed files with 16 additions and 17 deletions.
diff --git a/.github/workflows/dev_on_workflow_deploy_pullrequests.yml b/.github/workflows/dev_on_workflow_deploy_pullrequests.yml
@@ -235,6 +235,22 @@ jobs:
             echo "| alembic upgrade post@head    |  $duration |"
           } >> $GITHUB_STEP_SUMMARY
 
+      - name: "Sync ArgoCD pcapi application"
+        uses: pass-culture/common-workflows/actions/argocd-sync@argocd-sync/v0.6.0
+        with:
+          app_name: pcapi-pr-${{ github.event.pull_request.number }}
+          argocd_app_wait: true
+          prune: true
+
+      - name: "Run sandbox"
+        run: |
+          set -e
+          VERSION="${{ inputs.app_version }}"
+          export ENVIRONMENT="${{ inputs.environment }}"
+          IMAGE="europe-west1-docker.pkg.dev/passculture-infra-prod/pass-culture-artifact-registry/pcapi"
+          export IMAGE="${IMAGE}:${VERSION}"
+          export PRID=${{ github.event.pull_request.number }}
+
           echo "=== exec: flask sandbox -n industrial"
           export DATE=$(date +"%Y-%m-%d-%H-%M-%S")
           start=$(date +%s.%N)
@@ -249,10 +265,3 @@ jobs:
           {
             echo "| flask sandbox -n industrial    |  $duration |"
           } >> $GITHUB_STEP_SUMMARY
-
-      - name: "Sync ArgoCD pcapi application"
-        uses: pass-culture/common-workflows/actions/argocd-sync@argocd-sync/v0.6.0
-        with:
-          app_name: pcapi-pr-${{ github.event.pull_request.number }}
-          argocd_app_wait: true
-          prune: true
diff --git a/.github/workflows/templates/pullrequest-db-init-01.yaml b/.github/workflows/templates/pullrequest-db-init-01.yaml
@@ -27,13 +27,11 @@ spec:
         envFrom:
         - configMapRef:
             name: testing-pcapi-api
-            optional: true
         - configMapRef:
             name: testing-pcapi-api-specific
             optional: true
         - secretRef:
             name: testing-pcapi
-            optional: true
         - secretRef:
             name: testing-pcapi-pgsql
         command:

diff --git a/.github/workflows/templates/pullrequest-db-init-02.yaml b/.github/workflows/templates/pullrequest-db-init-02.yaml
@@ -27,13 +27,11 @@ spec:
         envFrom:
         - configMapRef:
             name: testing-pcapi-api
-            optional: true
         - configMapRef:
             name: testing-pcapi-api-specific
             optional: true
         - secretRef:
             name: testing-pcapi
-            optional: true
         - secretRef:
             name: testing-pcapi-pgsql
         command:

diff --git a/.github/workflows/templates/pullrequest-db-init-03.yaml b/.github/workflows/templates/pullrequest-db-init-03.yaml
@@ -27,13 +27,11 @@ spec:
         envFrom:
         - configMapRef:
             name: testing-pcapi-api
-            optional: true
         - configMapRef:
             name: testing-pcapi-api-specific
             optional: true
         - secretRef:
             name: testing-pcapi
-            optional: true
         - secretRef:
             name: testing-pcapi-pgsql
         command:

diff --git a/.github/workflows/templates/pullrequest-db-init-05.yaml b/.github/workflows/templates/pullrequest-db-init-05.yaml
@@ -27,13 +27,11 @@ spec:
         envFrom:
         - configMapRef:
             name: testing-pcapi-api
-            optional: true
         - configMapRef:
             name: testing-pcapi-api-specific
             optional: true
         - secretRef:
             name: testing-pcapi
-            optional: true
         - secretRef:
             name: testing-pcapi-pgsql
         command:

diff --git a/.github/workflows/templates/pullrequest-db-init.yaml b/.github/workflows/templates/pullrequest-db-init.yaml
@@ -26,13 +26,11 @@ spec:
         envFrom:
         - configMapRef:
             name: testing-pcapi-api
-            optional: true
         - configMapRef:
             name: testing-pcapi-api-specific
             optional: true
         - secretRef:
             name: testing-pcapi
-            optional: true
         - secretRef:
             name: testing-pcapi-pgsql
         command: