Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Deployment API (versioning 3.1) #765

Merged

Conversation

antlai-temporal
Copy link
Contributor

What was changed

This PR provide support for the Deployment API with the following commands:

temporal worker deployment delete
temporal worker deployment delete-version
temporal worker deployment describe
temporal worker deployment describe-version
temporal worker deployment list
temporal worker deployment set-current-version
temporal worker deployment set-ramping-version
temporal worker deployment update-metadata-version

It also updates the workflow update-options API to the new versioning override API, and workflow describe.

NOTE: the server hash in go.mod will be changed to v1.27.1 before merging.

Checklist

  1. Closes
    [Feature Request] Update Worker deployment commands to versioning-3.1 #744 [Feature Request] Add Ramp to worker deployment #745

- worker deployment set-current-version
- worker deployment set-ramping-version
- worker deployment delete-version
- worker deployment update-metadata-version
Copy link
Contributor

Choose a reason for hiding this comment

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

It should be update-version-metadata, right?

Copy link
Contributor Author

@antlai-temporal antlai-temporal Feb 26, 2025

Choose a reason for hiding this comment

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

I'm just keeping it consistent with the others. You can read it as "update metadata for a version"

Copy link
Member

@cretz cretz left a comment

Choose a reason for hiding this comment

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

Approving with the understanding that everything added here is not required to be stable and can change in subsequent versions.

@antlai-temporal antlai-temporal merged commit c4a5055 into temporalio:next-server Feb 26, 2025
6 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants