Skip to content

feat(service): Temporal visibility for pipeline workflows#278

Open
tonyxiao wants to merge 2 commits intov2from
feat/pipeline-temporal-visibility
Open

feat(service): Temporal visibility for pipeline workflows#278
tonyxiao wants to merge 2 commits intov2from
feat/pipeline-temporal-visibility

Conversation

@tonyxiao
Copy link
Copy Markdown
Collaborator

Summary

Temporal visibility for pipeline workflows (search attributes + status query). See description in commits.

- Upsert keyword search attributes SyncEnginePipelineStatus and
  SyncEnginePipelineDesiredStatus when status or desired_status changes
  (register both on the namespace for UI columns and list filters).
- Add status workflow query returning pipeline snapshot for drill-down.
- Register the same search attributes in local TestWorkflowEnvironment
  via a shared test helper; add @temporalio/common devDependency for keys.

Made-with: Cursor
Committed-By-Agent: cursor
@tonyxiao tonyxiao force-pushed the feat/pipeline-temporal-visibility branch from f17be91 to 57687bd Compare April 13, 2026 23:06
Self-hosted Temporal (compose + GHA service containers) does not define the
Keyword search attributes used by pipelineWorkflow upsertSearchAttributes.
Without them, workflow tasks fail and pipeline e2e polls time out.

- Add one-shot temporal-search-attributes in compose.dev.yml only (base compose.yml
  is used by the test job without this overlay; avoids Compose env interpolation
  mangling shell variables in inline commands).
  Gate service and worker on it via service_completed_successfully.
- Run scripts/register-temporal-pipeline-search-attributes.sh after Temporal is
  ready in e2e_stripe and e2e_service CI jobs.

Made-with: Cursor
Committed-By-Agent: cursor
Made-with: Cursor
Committed-By-Agent: cursor
@tonyxiao tonyxiao force-pushed the feat/pipeline-temporal-visibility branch from 57687bd to bb4f4ad Compare April 13, 2026 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant