From f02a21f9530f483e426b32a9e1c05a6045ae7cc2 Mon Sep 17 00:00:00 2001
From: Neil Kakkar <neilkakkar@gmail.com>
Date: Thu, 18 Apr 2024 10:11:55 +0100
Subject: [PATCH] split out temporal

---
 .github/actions/run-backend-tests/action.yml | 16 ++++++++++++++--
 .github/workflows/ci-backend.yml             |  6 ++++++
 2 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/.github/actions/run-backend-tests/action.yml b/.github/actions/run-backend-tests/action.yml
index c3e1d6d05a3a7..44e447257ca8c 100644
--- a/.github/actions/run-backend-tests/action.yml
+++ b/.github/actions/run-backend-tests/action.yml
@@ -12,7 +12,7 @@ inputs:
         description: ClickHouse server image tag, e.g. clickhouse/clickhouse-server:latest
     segment:
         required: true
-        description: Either 'FOSS' or 'EE' segment
+        description: Either 'FOSS' or 'EE' or 'TEMPORAL' segment
     concurrency:
         required: true
         description: Count of concurrency groups
@@ -132,6 +132,7 @@ runs:
           run: bin/check_kafka_clickhouse_up
 
         - name: Wait for Temporal
+          if: ${{ inputs.segment == 'TEMPORAL' }}
           shell: bash
           run: |
               bin/check_temporal_up
@@ -157,6 +158,7 @@ runs:
                   && './posthog/clickhouse/ ./posthog/hogql/ ./posthog/queries/ ./posthog/api/test/test_insight* ./posthog/api/test/dashboards/test_dashboard.py'
                   || 'hogvm posthog'
               }} -m "not async_migrations" \
+                  --ignore=posthog/temporal
                   --splits ${{ inputs.concurrency }} --group ${{ inputs.group }} \
                   --durations=100 --durations-min=1.0 --store-durations \
                   $PYTEST_ARGS
@@ -189,9 +191,19 @@ runs:
                   --durations=100 --durations-min=1.0 \
                   $PYTEST_ARGS
 
+        - name: Run Temporal tests
+          id: run-temporal-tests
+          if: ${{ inputs.segment == 'TEMPORAL' }}
+          shell: bash
+          run: |
+              pytest posthog/temporal -m "not async_migrations" \
+                  --splits ${{ inputs.concurrency }} --group ${{ inputs.group }} \
+                  --durations=100 --durations-min=1.0 --store-durations \
+                  $PYTEST_ARGS
+
         # Post tests
         - name: Show docker compose logs on failure
-          if: failure() && (steps.run-foss-tests.outcome != 'failure' && steps.run-ee-tests.outcome != 'failure' && steps.run-decide-read-replica-tests.outcome != 'failure')
+          if: failure() && (steps.run-foss-tests.outcome != 'failure' && steps.run-ee-tests.outcome != 'failure' && steps.run-decide-read-replica-tests.outcome != 'failure' && steps.run-temporal-tests.outcome != 'failure')
           shell: bash
           run: docker compose -f docker-compose.dev.yml logs
 
diff --git a/.github/workflows/ci-backend.yml b/.github/workflows/ci-backend.yml
index c0ee8ca548d79..7841b8258f920 100644
--- a/.github/workflows/ci-backend.yml
+++ b/.github/workflows/ci-backend.yml
@@ -249,6 +249,12 @@ jobs:
                 # :NOTE: Keep concurrency and groups in sync
                 concurrency: [5]
                 group: [1, 2, 3, 4, 5]
+                include:
+                    - segment: 'TEMPORAL'
+                      person-on-events: false
+                      clickhouse-server-image: 'clickhouse/clickhouse-server:23.12.5.81-alpine'
+                      concurrency: 1
+                      group: 1
 
         steps:
             # The first step is the only one that should run if `needs.changes.outputs.backend == 'false'`.