-
Notifications
You must be signed in to change notification settings - Fork 37
Comparing changes
Open a pull request
base repository: temporalio/api-go
base: v1.49.1
head repository: temporalio/api-go
compare: v1.50.0
- 12 commits
- 50 files changed
- 8 contributors
Commits on May 14, 2025
-
VersionLocalState in sync with VersionSummary + Activation/Deactivati…
…on timestamps (#585) _**READ BEFORE MERGING:** All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted._ <!-- Describe what has changed in this PR --> **What changed?** - Added two new timestamps to signify when a version first/last receives traffic in the `WorkerDeploymentVersionInfo` and `WorkerDeploymentInfo`. Added missing fields from VersionLocalState that were not present in VersionSummary. <!-- Tell your future self why have you made these changes --> **Why?** - Adding the two new timestamps shall allow operators to have more visibility into when a version gets activated/deactivated. - The reason fields from `VersionLocalState` were added to the `VersionSummary` is because the UI, right now, is querying a bunch of different places to get information about a specific version. This might be annoying and the aim of this PR is to make VersionSummary the source of truth for information related to a specific version. A nice cherry on top is that this shall also reduce the number of API calls to get information about a version. <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** - None <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. --> **Server PR** - temporalio/temporal#7736)
Configuration menu - View commit details
-
Copy full SHA for aef6069 - Browse repository at this point
Copy the full SHA aef6069View commit details
Commits on May 15, 2025
-
Fix protobuf syntax that is causing an error in protobufjs (#589)
**What changed?** - Fix a double semicolon introduced in #579. **Why?** - The double semicolon sequence is causing a compilation error in `protobufjs`, even though this is technically valid according to [the official Protobuf 3 spec](https://protobuf.dev/reference/protobuf/proto3-spec/#emptystatement). This is a bug in `protobufjs`, for which there's been [an open issue ticket](protobufjs/protobuf.js#1322) since 2019, but it was never acknowledged, and a PR fixing this has been ignored for more than a year. Given that there's simply no reason to use a double semicolon sequence anyway, it appears preferable to simply avoid that in our proto definitions.
Configuration menu - View commit details
-
Copy full SHA for 546d09f - Browse repository at this point
Copy the full SHA 546d09fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 39aa8c9 - Browse repository at this point
Copy the full SHA 39aa8c9View commit details
Commits on May 16, 2025
-
Add a new workflow failure cause for oversized grpc messages (#591)
_**READ BEFORE MERGING:** All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted._ <!-- Describe what has changed in this PR --> Add a new workflow failure cause for oversized grpc messages <!-- Tell your future self why have you made these changes --> We want to be able to distinguish failures with this cause <!-- Are there any breaking changes on binary or code level? --> <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. -->
Configuration menu - View commit details
-
Copy full SHA for 55da144 - Browse repository at this point
Copy the full SHA 55da144View commit details -
Sync //Deprecated comment and [deprecated = true] attribute (#592)
**What changed?** Sync `//Deprecated` comment and `[deprecated = true]` attribute. **Why?** When field is deprecated it should be marked with both comment and attribute to get full support in all languages. **Breaking changes** No, only comments/attributes changes.
Configuration menu - View commit details
-
Copy full SHA for 6bdc773 - Browse repository at this point
Copy the full SHA 6bdc773View commit details
Commits on May 28, 2025
-
added latest_version, current_version, ramping_version summaries in L…
…istDeployment (#594) _**READ BEFORE MERGING:** All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted._ <!-- Describe what has changed in this PR --> **What changed?** - WISOTT <!-- Tell your future self why have you made these changes --> **Why?** - making more information available in one API call rather than via multiple API calls. <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. --> **Server PR** - temporalio/temporal#7802
Configuration menu - View commit details
-
Copy full SHA for 522a75a - Browse repository at this point
Copy the full SHA 522a75aView commit details -
VersionStatus for WorkerDeploymentVersions (#595)
_**READ BEFORE MERGING:** All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted._ <!-- Describe what has changed in this PR --> **What changed?** - add VersionStatus inside `WorkerDeploymentVersionInfo` + `WorkerDeploymentVersionSummary` <!-- Tell your future self why have you made these changes --> **Why?** - right now, the UI code deduces the status of a version by checking `CurrentSince`, `RampingSince` and other attributes. This can just be simplified by adding this enum and the server doing the heavy lifting instead. <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** - None <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. --> **Server PR** - [741011](temporalio/temporal#7804)
Configuration menu - View commit details
-
Copy full SHA for 8e52eb2 - Browse repository at this point
Copy the full SHA 8e52eb2View commit details -
Add flag to restore activity options to reset and update (#593)
<!-- Describe what has changed in this PR --> **What changed?** Add flags to activity reset/activity update-options requests to support "restore original options" behaviour. <!-- Tell your future self why have you made these changes --> **Why?** They will help users to "reset" their changes. <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** No
Configuration menu - View commit details
-
Copy full SHA for b08ef1e - Browse repository at this point
Copy the full SHA b08ef1eView commit details
Commits on May 30, 2025
-
inherit pinned version and override across CaN when pinned (#588)
_**READ BEFORE MERGING:** All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted._ **What changed?** Add fields to workflow started event to enable inheriting pinned version and override across CaN when the effective behavior of the WF is pinned. This inheritance will not happen if the behavior is anything else (auto-upgrade, pinned-until-continue-as-new, etc) **Why?** To differentiate pinned from trampolining <!-- Are there any breaking changes on binary or code level? --> No <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. --> **Server PR**
Configuration menu - View commit details
-
Copy full SHA for 6ac9b43 - Browse repository at this point
Copy the full SHA 6ac9b43View commit details -
Add Stats to DescribeTaskQueue (#596)
_**READ BEFORE MERGING:** All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted._ <!-- Describe what has changed in this PR --> **What changed?** Added `TaskQueueStats` field to `DescribeTaskQueueResponse`. <!-- Tell your future self why have you made these changes --> **Why?** To provide non-versioning users with relevant task queue stats. PS: The long-term plan is to deprecate the ENHANCED mod fields. <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** N/A <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. --> **Server PR** temporalio/temporal#7816
Configuration menu - View commit details
-
Copy full SHA for c6aefda - Browse repository at this point
Copy the full SHA c6aefdaView commit details -
add client identity to update-metadata request (#600)
_**READ BEFORE MERGING:** All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted._ <!-- Describe what has changed in this PR --> add client identity to update-metadata request <!-- Tell your future self why have you made these changes --> on the server side we generate a uuid identity. would be nice to allow user to specify it instead <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. --> **Server PR**
Configuration menu - View commit details
-
Copy full SHA for 68ab55b - Browse repository at this point
Copy the full SHA 68ab55bView commit details
Commits on Jun 5, 2025
-
Add worker visibility API - RecordWorkerHeartbeat and ListWorkers (#601)
<!-- Describe what has changed in this PR --> **What changed?** What changed? * 2 new APIs added: * WorkerHeartbeat * ListWorkerStatus * WorkerInfo information added to: * ShutDownWorker request * Poll(Activity|Nexus|Workflow)TaskQueue requests This is "for merge". Here is a previous PR, where we discuss this at length: temporalio/api#599 <!-- Tell your future self why have you made these changes --> **Why?** Part of the worker visibility work. Workers should be able to report their status. Users should be able to get information about workers. <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** Yes - new API is introduce. <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. --> **Server PR** [Server PR](temporalio/temporal#7870)
Configuration menu - View commit details
-
Copy full SHA for e57aa58 - Browse repository at this point
Copy the full SHA e57aa58View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v1.49.1...v1.50.0