Skip to content

Add compute provider validation to worker deployment version summary#810

Merged
smuneebahmad merged 2 commits into
mainfrom
muneeb/wci-validation
Jun 22, 2026
Merged

Add compute provider validation to worker deployment version summary#810
smuneebahmad merged 2 commits into
mainfrom
muneeb/wci-validation

Conversation

@smuneebahmad

@smuneebahmad smuneebahmad commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

What changed?
Added ComputeStatus to the Worker Deployment API with a ProviderValidationStatus that tracks the result of the most recent connectivity check between Temporal and a customer's compute resource. An empty error message means validation passed; a non-empty message describes what failed.

Why?
Enables surfacing the connectivity health between Temporal and a customer's compute resource (e.g. Lambda) through the Worker Deployment API, so the UI can show ongoing validations status of compute configs without additional queries.

Breaking changes
None. All additions are new optional fields; existing clients are unaffected.

Server PR
temporalio/temporal#10778

@smuneebahmad smuneebahmad force-pushed the muneeb/wci-validation branch from 5394b7f to e6d1719 Compare June 19, 2026 21:32
@smuneebahmad smuneebahmad marked this pull request as ready for review June 19, 2026 21:40
@smuneebahmad smuneebahmad requested review from a team June 19, 2026 21:40
Comment thread temporal/api/compute/v1/validation.proto Outdated
Comment thread temporal/api/compute/v1/validation.proto Outdated
Comment thread temporal/api/deployment/v1/message.proto Outdated
Comment thread temporal/api/compute/v1/validation.proto Outdated
Comment thread temporal/api/workflowservice/v1/request_response.proto Outdated
@smuneebahmad smuneebahmad changed the title Include validation status in worker deployments Add compute provider validation to worker deployment version summary Jun 22, 2026
@smuneebahmad smuneebahmad requested a review from jaypipes June 22, 2026 20:29
@smuneebahmad smuneebahmad force-pushed the muneeb/wci-validation branch from 4645ce3 to fe7a6dc Compare June 22, 2026 20:42

@jaypipes jaypipes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@smuneebahmad smuneebahmad enabled auto-merge (squash) June 22, 2026 22:29
Comment thread temporal/api/deployment/v1/message.proto

@yuandrew yuandrew left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One question for docs, but not blocking

Comment thread temporal/api/deployment/v1/message.proto
@smuneebahmad smuneebahmad merged commit 96fb607 into main Jun 22, 2026
4 checks passed
@smuneebahmad smuneebahmad deleted the muneeb/wci-validation branch June 22, 2026 22:34
smuneebahmad added a commit that referenced this pull request Jun 22, 2026
**What changed?**
Update doc strings for Worker Deployment

**Why?**
These got misplaced during an earlier commit rebase:
#810
temporal-cicd Bot pushed a commit to temporalio/api-go that referenced this pull request Jun 22, 2026
**What changed?**
Update doc strings for Worker Deployment

**Why?**
These got misplaced during an earlier commit rebase:
temporalio/api#810
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.

6 participants