fix: do not interrupt tests (#187)

camunda · Nov 18, 2024 · 6b60788 · 6b60788
1 parent 21d5f64
commit 6b60788
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/.github/workflows/test-gha-eks.yml b/.github/workflows/test-gha-eks.yml
@@ -145,6 +145,8 @@ jobs:
               timeout-minutes: 45
               uses: ./.github/actions/eks-manage-cluster
               id: create_eks_cluster
+              # Do not interrupt tests; otherwise, the Terraform state may become inconsistent.
+              if: always() && success()
               with:
                   cluster-name: ${{ steps.commit_info.outputs.cluster_name }}
                   aws-region: ${{ env.AWS_REGION }}
@@ -174,7 +176,8 @@ jobs:
               timeout-minutes: 20
               uses: ./.github/actions/aurora-manage-cluster
               id: create_aurora_cluster
-              if: env.CREATE_DB == 'true'
+              # Do not interrupt tests; otherwise, the Terraform state may become inconsistent.
+              if: env.CREATE_DB == 'true' && always() && success()
               with:
                   cluster-name: ${{ steps.commit_info.outputs.cluster_name }}
                   username: ${{ steps.commit_info.outputs.db_username }}
@@ -194,7 +197,8 @@ jobs:
             - name: Deploy OpenSearch Domain
               uses: ./.github/actions/opensearch-manage-cluster
               id: deploy_opensearch_domain
-              if: env.CREATE_OPENSEARCH == 'true'
+              # Do not interrupt tests; otherwise, the Terraform state may become inconsistent.
+              if: env.CREATE_OPENSEARCH == 'true' && always() && success()
               with:
                   domain-name: ${{ steps.commit_info.outputs.cluster_name }}-opensearch
                   aws-region: ${{ env.AWS_REGION }}

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -142,6 +142,8 @@ jobs:
 
             - name: Launch test
               timeout-minutes: 125
+              # Do not interrupt tests; otherwise, the Terraform state may become inconsistent.
+              if: always() && success()
               run: |
                   export TESTS_CLUSTER_ID="${{ needs.configure-tests.outputs.cluster_id }}"
                   export TESTS_CLUSTER_REGION="${{ env.AWS_REGION }}"