Restrict CI and add debugging output

Author: dandavison

.github/workflows/ci.yml

@@ -88,11 +88,6 @@ jobs:
           TEMPORAL_CLIENT_CLOUD_API_KEY: ${{ secrets.TEMPORAL_CLIENT_CLOUD_API_KEY }}
           TEMPORAL_CLIENT_CLOUD_API_VERSION: 2024-05-13-00
           TEMPORAL_CLIENT_CLOUD_NAMESPACE: sdk-ci.a2dd6
-      - if: ${{ matrix.openaiTestTarget && (github.event.pull_request.head.repo.full_name == '' || github.event.pull_request.head.repo.full_name == 'temporalio/sdk-python') }}
-        run: poe test tests/contrib/openai_agents/test_openai.py ${{matrix.pytestExtraArgs}} -s  --junit-xml=junit-xml/${{ matrix.python }}--${{ matrix.os }}--openai.xml
-        timeout-minutes: 10
-        env:
-          OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
       - name: "Upload junit-xml artifacts"
         uses: actions/upload-artifact@v4
         if: always()
@@ -101,43 +96,7 @@ jobs:
           path: junit-xml
           retention-days: 14
 
-      # Confirm protos are already generated properly with older protobuf
-      # library and run test with that older version. We must downgrade protobuf
-      # so we can generate 3.x and 4.x compatible API. We have to use older
-      # Python to run this check because the grpcio-tools version we use
-      # is <= 3.10.
-      - name: Check generated protos and test protobuf 3.x
-        if: ${{ matrix.protoCheckTarget }}
-        env:
-          TEMPORAL_TEST_PROTO3: 1
-        run: |
-          uv add --python 3.9 "protobuf<4"
-          uv sync --all-extras
-          poe build-develop
-          poe gen-protos
-          poe format
-          [[ -z $(git status --porcelain temporalio) ]] || (git diff temporalio; echo "Protos changed"; exit 1)
-          poe test -s
-        timeout-minutes: 10
 
-      # Do docs stuff (only on one host)
-      - name: Build API docs
-        if: ${{ matrix.docsTarget }}
-        run: poe gen-docs
-      - name: Deploy prod API docs
-        if: ${{ github.ref == 'refs/heads/main' && matrix.docsTarget }}
-        env:
-          VERCEL_ORG_ID: ${{ secrets.VERCEL_ORG_ID }}
-          VERCEL_PROJECT_ID: ${{ secrets.VERCEL_PROJECT_ID }}
-        run: npx vercel deploy build/apidocs -t ${{ secrets.VERCEL_TOKEN }} --prod --yes
-
-      # Confirm README ToC is generated properly
-      - uses: actions/setup-node@v4
-      - name: Check generated README ToC
-        if: ${{ matrix.docsTarget }}
-        run: |
-          npx doctoc README.md
-          [[ -z $(git status --porcelain README.md) ]] || (git diff README.md; echo "README changed"; exit 1)
   test-latest-deps:
     timeout-minutes: 30
     runs-on: ubuntu-latest
@@ -173,11 +132,3 @@ jobs:
           name: junit-xml--${{github.run_id}}--${{github.run_attempt}}--latest-deps--time-skipping
           path: junit-xml
           retention-days: 14
-
-  # Runs the sdk features repo tests with this repo's current SDK code
-  features-tests:
-    uses: temporalio/features/.github/workflows/python.yaml@main
-    with:
-      python-repo-path: ${{github.event.pull_request.head.repo.full_name}}
-      version: ${{github.event.pull_request.head.ref}}
-      version-is-repo-ref: true

pyproject.toml

@@ -81,7 +81,7 @@ lint-types = [
   { cmd = "uv run mypy --namespace-packages --check-untyped-defs ."},
 ]
 run-bench = "uv run python scripts/run_bench.py"
-test = "uv run pytest"
+test = "uv run pytest -s 'tests/nexus/test_workflow_caller_cancellation_types.py::test_cancellation_type[TRY_CANCEL]' 'tests/nexus/test_workflow_caller_cancellation_types_when_cancel_handler_fails.py::test_cancellation_type[TRY_CANCEL]'"
 
 
 [tool.pytest.ini_options]

tests/nexus/test_workflow_caller_cancellation_types.py

@@ -21,6 +21,7 @@
 from temporalio.common import WorkflowIDConflictPolicy
 from temporalio.testing import WorkflowEnvironment
 from temporalio.worker import Worker
+from tests.helpers import print_interleaved_histories
 from tests.helpers.nexus import create_nexus_endpoint, make_nexus_endpoint_name
 
 
@@ -329,6 +330,9 @@ async def check_behavior_for_try_cancel(
     handler_status = (await handler_wf.describe()).status
     assert handler_status == WorkflowExecutionStatus.CANCELED
     caller_op_future_resolved = test_context.caller_op_future_resolved.result()
+
+    await print_interleaved_histories([caller_wf, handler_wf])
+
     await assert_event_subsequence(
         [
             (caller_wf, EventType.EVENT_TYPE_NEXUS_OPERATION_CANCEL_REQUESTED),

tests/nexus/test_workflow_caller_cancellation_types_when_cancel_handler_fails.py

@@ -23,6 +23,7 @@
 from temporalio.common import WorkflowIDConflictPolicy
 from temporalio.testing import WorkflowEnvironment
 from temporalio.worker import Worker
+from tests.helpers import print_interleaved_histories
 from tests.helpers.nexus import create_nexus_endpoint, make_nexus_endpoint_name
 from tests.nexus.test_workflow_caller_cancellation_types import (
     assert_event_subsequence,
@@ -301,6 +302,8 @@ async def check_behavior_for_try_cancel(
     assert result.error_cause_type == "CancelledError"
 
     caller_op_future_resolved = test_context.caller_op_future_resolved.result()
+    await print_interleaved_histories([caller_wf, handler_wf])
+
     await assert_event_subsequence(
         [
             (caller_wf, EventType.EVENT_TYPE_WORKFLOW_EXECUTION_STARTED),