Merge pull request #6 from segoldma/feat/persist-manifest

Feat/persist manifest
segoldma · Aug 3, 2024 · a2a23e9 · a2a23e9
2 parents eb32da3 + b7a3f5f
commit a2a23e9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 13 deletions.
diff --git a/.ci/profiles.yml b/.ci/profiles.yml
@@ -2,14 +2,13 @@ sethbox:
   target: ci
   outputs:
     ci:
-      dataset: {{ env_var('CICD_DATASET') }}
-      keyfile:  {{ env_var('CICD_SERVICE_ACCOUNT') }}
-      location: {{ env_var('CICD_LOCATION') }}
-      priority: {{ env_var('CICD_PRIORITY') }}
-      project: {{ env_var('CICD_PROJECT') }}
+      dataset: "{{ env_var('CICD_DATASET') }}"
+      keyfile: "{{ env_var('CICD_SERVICE_ACCOUNT') }}"
+      location: "{{ env_var('CICD_LOCATION') }}"
+      priority: "{{ env_var('CICD_PRIORITY') }}"
+      project: "{{ env_var('CICD_PROJECT') }}"
       type: bigquery
       method: service-account
       job_execution_timeout_seconds: 300
       job_retries: 1
       threads: 16
-
diff --git a/.github/workflows/persist-manifest.yml b/.github/workflows/persist-manifest.yml
@@ -25,18 +25,13 @@ jobs:
         pip install dbt-bigquery
 
     - name: Set up environment variables and write service account key
+      run: |
+        echo "${{ secrets.CICD_SERVICE_ACCOUNT }}" > /tmp/service_account.json
       env:
         CICD_DATASET: ${{ secrets.CICD_DATASET }}
         CICD_LOCATION: ${{ secrets.CICD_LOCATION }}
         CICD_PRIORITY: ${{ secrets.CICD_PRIORITY }}
         CICD_PROJECT: ${{ secrets.CICD_PROJECT }}
-      run: |
-        echo "${{ secrets.CICD_SERVICE_ACCOUNT }}" > /tmp/service_account.json
-        echo "Dataset: $CICD_DATASET"
-        echo "Keyfile: /tmp/service_account.json"
-        echo "Location: $CICD_LOCATION"
-        echo "Priority: $CICD_PRIORITY"
-        echo "Project: $CICD_PROJECT"
 
     - name: Create dbt profiles directory
       run: mkdir -p ~/.dbt