Merge pull request #56 from j3-signalroom/github_issue-55

See #55. Fixed issue.
j3-signalroom · Sep 6, 2024 · 9379a5a · 9379a5a
2 parents d042ebe + 69e51d2
commit 9379a5a
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 2 deletions.
diff --git a/.github/actions/aws-environment-info/action.yml b/.github/actions/aws-environment-info/action.yml
@@ -18,9 +18,9 @@ inputs:
   aws_region:
     required: true
   snowflake_warehouse:
-    required: false
+    required: true
   service_account_user:
-    required: false
+    required: true
   day_count:
     required: false
 outputs:

diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml
@@ -92,6 +92,7 @@ jobs:
           TF_VAR_aws_access_key_id: ${{ steps.aws-credentials.outputs.aws-access-key-id }}
           TF_VAR_aws_secret_access_key: ${{ steps.aws-credentials.outputs.aws-secret-access-key }}
           TF_VAR_aws_session_token: ${{ steps.aws-credentials.outputs.aws-session-token }}
+          TF_VAR_snowflake_warehouse: ${{ github.event.inputs.snowflake_warehouse }}
           TF_VAR_service_account_user: ${{ github.event.inputs.service_account_user }}
           TF_VAR_day_count: ${{ github.event.inputs.day_count }}
 
@@ -104,5 +105,6 @@ jobs:
           TF_VAR_aws_access_key_id: ${{ steps.aws-credentials.outputs.aws-access-key-id }}
           TF_VAR_aws_secret_access_key: ${{ steps.aws-credentials.outputs.aws-secret-access-key }}
           TF_VAR_aws_session_token: ${{ steps.aws-credentials.outputs.aws-session-token }}
+          TF_VAR_snowflake_warehouse: ${{ github.event.inputs.snowflake_warehouse }}
           TF_VAR_service_account_user: ${{ github.event.inputs.service_account_user }}
           TF_VAR_day_count: ${{ github.event.inputs.day_count }}
diff --git a/.github/workflows/undeploy.yml b/.github/workflows/undeploy.yml
@@ -25,6 +25,10 @@ on:
           - us-east-2
           - us-west-1
           - us-west-2
+      snowflake_warehouse:
+        type: string
+        description: Snowflake Warehouse
+        required: true
       service_account_user:
         type: string
         description: Snowflake Service Account User
@@ -49,6 +53,8 @@ jobs:
             aws_test_account_id: ${{ vars.AWS_TEST_ACCOUNT_ID }}
             aws_prod_account_id: ${{ vars.AWS_PROD_ACCOUNT_ID }}
             aws_region: ${{ github.event.inputs.aws_region }}
+            snowflake_warehouse: ${{ github.event.inputs.snowflake_warehouse }}
+            service_account_user: ${{ github.event.inputs.service_account_user }}
 
         - uses: hashicorp/setup-terraform@v3
           with:
@@ -79,6 +85,7 @@ jobs:
             TF_VAR_aws_access_key_id: ${{ steps.aws-credentials.outputs.aws-access-key-id }}
             TF_VAR_aws_secret_access_key: ${{ steps.aws-credentials.outputs.aws-secret-access-key }}
             TF_VAR_aws_session_token: ${{ steps.aws-credentials.outputs.aws-session-token }}
+            TF_VAR_snowflake_warehouse: ${{ github.event.inputs.snowflake_warehouse }}
             TF_VAR_service_account_user: ${{ github.event.inputs.service_account_user }}
 
         - name: Terraform Destroy Apply
@@ -90,5 +97,6 @@ jobs:
             TF_VAR_aws_access_key_id: ${{ steps.aws-credentials.outputs.aws-access-key-id }}
             TF_VAR_aws_secret_access_key: ${{ steps.aws-credentials.outputs.aws-secret-access-key }}
             TF_VAR_aws_session_token: ${{ steps.aws-credentials.outputs.aws-session-token }}
+            TF_VAR_snowflake_warehouse: ${{ github.event.inputs.snowflake_warehouse }}
             TF_VAR_service_account_user: ${{ github.event.inputs.service_account_user }}
 
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,10 @@ All notable changes to this project will be documented in this file.
 
 The format is base on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.34.00.000] - 2024-09-06
+### Fixed
+- Passing `snowflake_warehouse` variable to the Terraform configuration in the GitHub Workflows.
+
 ## [0.33.00.000] - 2024-09-06
 ### Fixed
 - Passing `service_account_user` variable to the Terraform configuration in the GitHub Workflows.