Skip to content

Validate deployment config and test worker with versioning off and custom build ID#1361

Merged
mjameswh merged 4 commits intomainfrom
worker-versioning-off-with-build-id
Mar 16, 2026
Merged

Validate deployment config and test worker with versioning off and custom build ID#1361
mjameswh merged 4 commits intomainfrom
worker-versioning-off-with-build-id

Conversation

@mjameswh
Copy link
Contributor

Summary

  • Adds runtime validation rejecting default_versioning_behavior != UNSPECIFIED when use_worker_versioning=False, preventing stuck workflows caused by server-side rejection during workflow task completion
  • Adds integration test confirming a worker with use_worker_versioning=False and a custom build ID can successfully run workflows (build ID appears in history)
  • Adds unit test for the new validation

Related: temporalio/sdk-typescript#1962

Test plan

  • test_worker_deployment_config_without_versioning — integration test passes against local server
  • test_deployment_config_rejects_versioning_behavior_without_versioning — validation test passes

🤖 Generated with Claude Code

…stom build ID

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@mjameswh mjameswh requested a review from a team as a code owner March 12, 2026 21:38
@mjameswh mjameswh enabled auto-merge (squash) March 12, 2026 22:14
@mjameswh mjameswh merged commit 610a0bb into main Mar 16, 2026
45 of 51 checks passed
@mjameswh mjameswh deleted the worker-versioning-off-with-build-id branch March 16, 2026 23:12
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.

2 participants